Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixes #38

Closed
3 tasks done
Jagerente opened this issue May 20, 2023 · 10 comments
Closed
3 tasks done

Fixes #38

Jagerente opened this issue May 20, 2023 · 10 comments
Assignees
Labels
bug Something isn't working enhancement Fixes or optimizations frontend Frontend related issues

Comments

@Jagerente
Copy link
Member

Jagerente commented May 20, 2023

  • Не отображать объект новости и таблиц, если название пустое.
  • Меню не отображается в 769px<ЭТОМ<992px диапазоне
    image
  • Добавить target _blank к редиректам таблиц.
@Jagerente Jagerente added the enhancement Fixes or optimizations label May 20, 2023
@Jagerente Jagerente added this to the Public Preview milestone May 20, 2023
@Bobrokrot
Copy link
Contributor

Bobrokrot commented May 21, 2023

  • После выбора характеристики дродауну требуется целых полсекунды, чтобы исчезнуть.
    image

@Jagerente
Copy link
Member Author

Jagerente commented May 21, 2023

  • Accept Languages не отправляется.

@Bobrokrot
Copy link
Contributor

Bobrokrot commented May 21, 2023

  • У некоторых таблиц нет иконок и нет описаний. Надо, чтобы они соответствующе отображались.
    image

@Jagerente
Copy link
Member Author

Jagerente commented May 21, 2023

  • Добавить аналитику
<!-- Google tag (gtag.js) -->
<script async src="https://www.googletagmanager.com/gtag/js?id=G-0L5E763TLC"></script>
<script>
  window.dataLayer = window.dataLayer || [];
  function gtag(){dataLayer.push(arguments);}
  gtag('js', new Date());

  gtag('config', 'G-0L5E763TLC');
</script>

@Bobrokrot
Copy link
Contributor

Bobrokrot commented May 21, 2023

  • С мобилки не вижу ни в каком виде смены языка. У меня при нажатии на пространство слева от меню получается сменить язык (голубым выделяется).
    image

@Bobrokrot
Copy link
Contributor

Bobrokrot commented May 21, 2023

  • На странице таблиц вместо Кэ Цин должна быть вот эта.
    image

@Jagerente
Copy link
Member Author

Jagerente commented May 22, 2023

  • Стрелочка при наведении сливается с фоном

Image

@Jagerente Jagerente added the bug Something isn't working label May 22, 2023
@Jagerente
Copy link
Member Author

Jagerente commented May 22, 2023

  • Меню не локализовано
    image

@Jagerente Jagerente added the frontend Frontend related issues label May 22, 2023
@Jagerente
Copy link
Member Author

Jagerente commented May 22, 2023

  • p > h5 ? 😿 Исправить на h1 > p

Image

  • h2 > p во всех карточках

Image

  • href сместить на уровень выше. Внутренний заменить на span.

Image

  • Добавить aria-label

Image

@Jagerente Jagerente self-assigned this May 25, 2023
@Jagerente
Copy link
Member Author

Jagerente commented Jun 4, 2023

  • Исправить адаптивность на мобилках

В рамках #66 ждём уменьшения элементов.

Текущие косяки:

  • Меню уехало
  • Текст сжимается

Image

  • Выбор статов в 1 столбец, а не 2

Image

Jagerente pushed a commit that referenced this issue Nov 19, 2023
* -- pre mobile adaptation

* fix: Правки отображения, курсора и заголовков на мобилке

* feat: Оптимизация запроса на героев

Получать героев только один раз, после нажатия кнопки "Оценить"

* feat: Оптимизация запроса на новости и лоадер

Получение новостей только когда пользователь до скролит до блока новостей. Показывать лоадер, пока новости грузятся.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement Fixes or optimizations frontend Frontend related issues
Projects
None yet
Development

No branches or pull requests

3 participants