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 dropdown-menu. issue #485 #490

Merged
merged 1 commit into from
Aug 17, 2020
Merged

Conversation

AlexeyShobanov
Copy link
Contributor

Добавил кастомный класс css

.gitignore Outdated Show resolved Hide resolved
@@ -18,3 +18,7 @@
.x-min-w-0 {
min-width: 0px !important;
}

.x-z-index-dropdown {
Copy link
Collaborator

Choose a reason for hiding this comment

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

У нас ведь был кастомный класс для таких штук. Он не подойдет?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

По поводу кастомного класса не понял
Наложение идет потому, что для меню навигации используется класс sticky-top с z-index 1020
Класс dropdown-menu имеет z-index 1000
Чтобы это исправить, по-моему, нужно внести кастомный класс с z-index > 1020
или задавать другие значения для переменных
$zindex-dropdown: 1000 !default;
$zindex-sticky: 1020 !default;
$zindex-fixed: 1030 !default;
$zindex-modal-backdrop: 1040 !default;
$zindex-modal: 1050 !default;
$zindex-popover: 1060 !default;
$zindex-tooltip: 1070 !default;

Copy link

Choose a reason for hiding this comment

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

кажется нейминг не верный, скорее должно быть что-то типа x-z-index-2000

@AlexeyShobanov
Copy link
Contributor Author

AlexeyShobanov commented Aug 16, 2020 via email

@fey
Copy link
Collaborator

fey commented Aug 16, 2020

можешь сделать ребейз - в этой ветке удалит ькоммиты, которая относятся к лендингу. Потом форспуш

@fey
Copy link
Collaborator

fey commented Aug 16, 2020

Можно попробовать git pull --rebase origin master Тогда твоя ветка с фиксом станет типа продолжением мастера

@AlexeyShobanov
Copy link
Contributor Author

все почистил, решение пока не менял

@fey fey requested a review from amshkv August 16, 2020 13:29
@fey fey merged commit 52a84c6 into Hexlet:master Aug 17, 2020
@AlexeyShobanov AlexeyShobanov deleted the dropdown-menu branch August 17, 2020 18:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants