Skip to content

Improve controls#260

Closed
mikhailovSvyat wants to merge 1 commit intomasterfrom
improve-controls
Closed

Improve controls#260
mikhailovSvyat wants to merge 1 commit intomasterfrom
improve-controls

Conversation

@mikhailovSvyat
Copy link
Copy Markdown
Collaborator

*Добавил анимацию
*Подрихтовал внешний вид
*Реализацию посмотрел у яндекса
*Заенкодил иконку галочки вот тут https://yoksel.github.io/url-encoder/

@pacahon
Copy link
Copy Markdown
Contributor

pacahon commented Jul 31, 2019

Галочка не везде используется, поэтому давай без base64 для галки, у нас и так хватает чем раздуть размер css файла.
Мне очень сложно оценить, что и на что влияет, поэтому просьба каждую отдельную фичу отдельным PR.
P.S. Я не знаю какая реализация у яндекса, у них каждая команда свой огород делает как правило. Просьба выкладывать референсные ссылки сразу, если это важно.


//Error

.grouped.error .ui.option.radio:not(.disabled) > .control__indicator::before,
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

по идее .grouped здесь не должно быть. Если у нас уже так сделано, то это ошибка.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

#298 Поправил

@mikhailovSvyat
Copy link
Copy Markdown
Collaborator Author

Разбил на #299 и #300

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants