[#407] add scss selector for feedback tooltip #408
Conversation
geophyzik
changed the title
| .toolbar-feedback { | ||
| display: block; | ||
| min-width: 150px; | ||
| } |
There was a problem hiding this comment.
здесь вы задаете display-block а внизу убираете d-block?
Это же про одно и то же да?
т.е. я вижу, что здесь добавляется min-width. Выглядит так что это кастомный класс, который мы можем добавить в className, назвать его типа x-mw-150 и внутри только min-width использовать.
Просто вы добавляете кастомный который прям привязывается к конкретному инпуту и его никак нельзя переиспользовать. Мы стараемся наоборот все кастомы убрать и использовать дефолтные компоненты бутстрапа.
Но иногда требуется вводить кастомные классы, например в hexlet-basics, hexlet-sicp такие классы бывают.
There was a problem hiding this comment.
Это же про одно и то же да?
ага
Выглядит так что это кастомный класс, который мы можем добавить в className, назвать его типа x-mw-150 и внутри только min-width использовать.
d-block перекинул в scss чтобы дополнительно разгрузить строку
Мы стараемся наоборот все кастомы убрать и использовать дефолтные компоненты бутстрапа.
min-width: 150px как ни крути вроде над закинуть в кастом, а d-block на Ваше усмотрение, можн и обратно вернуть
ну кстати да я вроде понял зачем d-block не стоило убирать, а min-width: 150px закинуть в x-mw-150 и потом его проще переиспользовать т.к. нет лишнего d-blocka, с другой стороны 150px тоже слишком ситуативный селектор
There was a problem hiding this comment.
с другой стороны 150px тоже слишком ситуативный селектор
но все равно это лучше, когда css в css, а не в html, иначе очень быстро инлайн стили начинают размножаться по коду и это превращается в большую, плохопахнущую помойку. А это открытый проект, лучше чтобы распространялось хорошее)
There was a problem hiding this comment.
x-mw-150
я бы сделал чуть иначе название, сейчас очень общее, тем более mw в бутстрапе читается как max-width
наверное явное x-min-w-150px будет лучше
There was a problem hiding this comment.
в бутстрапе в примерах переодически используют инлайн стили, но если можно от них отказаться и сделать код чище - то почему бы и не попробовать)
|
@geophyzik Привет. Как успехи, нужна помощь с доработками? Если не хватает времени - другие ребята могут продолжить работу. |
|
возьму в работу |
Убрал класс у инпута, добавил для фидбека чтобы снова воссоединить карандаш с названием сниппета
fixes #407