search

Alexiy1984 / core_kit

0 stars 0 forks source link

Alert #14

Closed kamelot43 closed 3 years ago

kamelot43 commented 3 years ago

не знаю, на сколько критично, но элемент в макете называется notifications. Возможно название компонента alert будет сбивать с толку.

kamelot43 commented 3 years ago

Screenshot at Nov 17 16-02-54

просто предложение. можно попробовать выровнять текст в блоке на мобилке

kamelot43 commented 3 years ago

Screenshot at Nov 17 16-03-45

этот элемент уже встречается в макете. можно попробовать создать этот компонент, а потом его уже переиспользовать в других компонентах.

kamelot43 commented 3 years ago

Screenshot at Nov 17 16-06-09

этот элемент очень похож на верхний компонент. элемент можно попробовать вынести в компонент и переиспользовать его

kamelot43 commented 3 years ago
Screenshot at Nov 17 16-04-42

классы , которые отвечают за анимацию можно попробовать не привязывать к alert, а вынести в миксин. этот же миксин можно будет использовать и для модальных окон.

kamelot43 commented 3 years ago

Screenshot at Nov 17 16-08-18 классы так как сделаны сейчас вроде работают, но похоже, что есть дублирование кода. по идее элемент alertlargewarning должен наследовать два класса : alertlarge alertwarning . alertlargewarning при такой записи нам приходится каждый раз объявлять стили . сейчас нет наследования классов, если мы используем два и более модификаторов.

kamelot43 commented 3 years ago

alert_danger по идее этот класс должен наследовать два класса : alert и alert_danger. типо alert содержит все общие стили , a alert_danger меняет только цвет. alert_danger при таком объявлении классов наследование вроде на работает, хотя все выглядит вроде норм.

kamelot43 commented 3 years ago

обсудили вопросы по alert