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

fix: added scroll to navigation menu #1585 #1615

Merged
merged 6 commits into from
May 8, 2024
Merged

Conversation

afedoroff
Copy link
Contributor

@afedoroff afedoroff commented Apr 18, 2024

feat(component/navigation-menu): added scroll to navigation menu #1585
fix(components/scrollbar): theme switching in scrollbar works incorrect #1657

resolved #1585
resolved #1657

@afedoroff afedoroff added this to the 4.3.0 milestone Apr 18, 2024
Copy link

github-actions bot commented Apr 18, 2024

Visit the preview URL for this PR (updated for commit 460db1e):

https://prizm-v4--pr1615-fix-navigation-menu-68edbsv4.web.app

(expires Thu, 02 May 2024 10:20:31 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: 7c62ed8dbabf5e2d6b2084ca9e107cc206d30dbd

Copy link
Collaborator

@ickisIckis ickisIckis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link

@alexhawkins94 alexhawkins94 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

при переключении темы на витрине скролл может не подтягивать нужные цвета.
неправильный внешний вид скролла:
chrome_K6eYhzScgy

правильный вид скролла:
chrome_qcvT1oKsJV

Copy link

@alexhawkins94 alexhawkins94 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Если выполнить раскрытие групп в количестве, достаточном для отображения скролла, и выполнить скроллинг, то возможно появления лагов и ощущение замедления прокрутки навигационного меню.
При обновлении страницы ситуация становится лучше.

@ickisIckis
Copy link
Collaborator

Если выполнить раскрытие групп в количестве, достаточном для отображения скролла, и выполнить скроллинг, то возможно появления лагов и ощущение замедления прокрутки навигационного меню. При обновлении страницы ситуация становится лучше.

не воспроизводится, надо перепроверить

@ZurabDev ZurabDev merged commit cbe94bf into main May 8, 2024
5 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment