Skip to content

feat(input, icon) added dark-theme #62

Merged
merged 5 commits into from
Nov 2, 2018

Conversation

lskramarov
Copy link
Contributor

No description provided.

@mikeozornin
Copy link
Contributor

В инпутах есть неведомый баг, текст в дизебленном состоянии текст показывается светлей, чем надо. Видимо это из-за смешивания цветов или чего-то такого. Даже если вписать нужный цвет, браузер показывает неправильно. Это только сафари. Если это ошибка в сафари (такое может быть), то забить. Я не смог понять, почему оно так показывается.
http://d.mikeozornin.ru/tGmw7F

Инпуты остальное норм.

Про иконки у меня пока в голове вот такая картина:

Серые иконки бывают двух цветов:

  • default-icon
  • less-contrast-icon

Дефолтные серые иконки default-icon: лупа в поле, иконки дропдаунов v, все иконки вне полей (типа такого: http://d.mikeozornin.ru/otqTyR).
Облегченные серые иконки less-contrast-icon нужны тогда, когда действие, которое они делают не дефолтное и не самое важное. Не используются вне инпутов или селектов.

В дизебленном состоянии цвет default-icon становится less-contrast-icon, а less-contrast-icon сама по себе дизебленной не бывает. Если сам контрол задизеблен, то ту иконку, что была less-contrast-icon показывать просто не надо.

Хувер у них одинаковый можно сделать.
Для отдельно стоящих иконок можно подумать про хувер фоном (как в ангуляр материале).

@pimenovoleg pimenovoleg changed the title feat(input, icon) added dark-theme [WIP] feat(input, icon) added dark-theme Nov 1, 2018
@lskramarov lskramarov force-pushed the feature/dark-theme-in-input branch from 83b7387 to e360636 Compare November 2, 2018 07:23
@lskramarov lskramarov force-pushed the feature/dark-theme-in-input branch from e360636 to c2b9b0d Compare November 2, 2018 07:25
@pimenovoleg pimenovoleg merged commit 2a8cbf4 into master Nov 2, 2018
@pimenovoleg pimenovoleg changed the title [WIP] feat(input, icon) added dark-theme feat(input, icon) added dark-theme Nov 2, 2018
@lskramarov lskramarov deleted the feature/dark-theme-in-input branch November 6, 2018 14:03
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants