feat: add LMS layout#405
Conversation
| @@ -1,6 +1,15 @@ | |||
| $navbar-mobile-height: 48px; | |||
| $navbar-submenu-height: 40px; | |||
There was a problem hiding this comment.
Ни в коем случае не смешиваем реализации для внешнего сайта и внутреннего, потом устанем разделять их.
| @import "pages/about"; | ||
| @import "pages/principles"; | ||
|
|
||
| @import "lms/_main"; |
There was a problem hiding this comment.
Разделяем целиком стили lms и внешнего сайта. В будущем у нас будет 3 репозитория: компоненты, внешний сайт, lms. Последние 2 будут переиспользовать только компоненты, но наследовать код у друг-друга точно не будут.
mikhailovSvyat
force-pushed
the
feature-add-lms-layout
branch
from
42483dc to
7aff9ef
Compare
| lg: 992px, // XXX: Keep in sync with values from `v2/js/utils.js` | ||
| xl: 1260px | ||
| xl: 1260px, | ||
| lms: 1222px // $grid-breakpoints(lg) + $sidebar-width |
There was a problem hiding this comment.
Есть несколько переменных только для lms, вынести их в отдельный файл lms/variables.scss ?
| $zindex-popover: 1080; | ||
| $zindex-tooltip: 1090; | ||
| $zindex-close: 1100; | ||
| $zindex-sidebar: 1050; |
There was a problem hiding this comment.
Вот это тоже только для lms
| @@ -0,0 +1,43 @@ | |||
| $sidebar-width: 230px; | |||
There was a problem hiding this comment.
Тоже наверное в lms/variables.scss нужно положить?
| @@ -0,0 +1,194 @@ | |||
| $navbar-submenu-height: 40px; | |||
There was a problem hiding this comment.
Это правильная реализация независимого меню для lms?
2 participants
Переполнение
