Skip to content

Feature/dark theme in select #74

Merged
merged 9 commits into from
Nov 30, 2018
Merged

Conversation

lskramarov
Copy link
Contributor

No description provided.

@mikeozornin
Copy link
Contributor

mikeozornin commented Nov 30, 2018

  1. http://d.mikeozornin.ru/aOe3fL
    Эррор + фокус какой-то неясный. Там две красных рамки разного цвета.

  2. http://d.mikeozornin.ru/LIwDu4
    При хувере иконка тега становится менее контрастной, а лучше бы более контрастной. Мы вроде делали для иконок? Если сейчас все иконки вне кнопок так, то можно пропустить.

  3. В дропдауне у линка странный хувер. Если линки ещё не адаптировали к темной, то это не вопрос дропдауна.
    http://d.mikeozornin.ru/zjy3MU

Пункт 1 точно отсюда, 2 и 3 могут быть не из селекта или дропдауна.

@lskramarov
Copy link
Contributor Author

  1. http://d.mikeozornin.ru/aOe3fL
    Эррор + фокус какой-то неясный. Там две красных рамки разного цвета.
  2. http://d.mikeozornin.ru/LIwDu4
    При хувере иконка тега становится менее контрастной, а лучше бы более контрастной. Мы вроде делали для иконок? Если сейчас все иконки вне кнопок так, то можно пропустить.
  3. В дропдауне у линка странный хувер. Если линки ещё не адаптировали к темной, то это не вопрос дропдауна.
    http://d.mikeozornin.ru/zjy3MU

Пункт 1 точно отсюда, 2 и 3 могут быть не из селекта или дропдауна.

  1. Поправил
  2. Поправил, но там все сложнее, позже возможно вернемся
  3. Будет исправлено, когда будем делать темную тему в линках

@mikeozornin
Copy link
Contributor

Если поправил, то я за влитие.

@pimenovoleg pimenovoleg merged commit 0232021 into master Nov 30, 2018
@lskramarov lskramarov deleted the feature/dark-theme-in-select branch December 5, 2018 12:28
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