Skip to content

Feat/button toggle #101

Merged
merged 9 commits into from
Apr 5, 2019
Merged

Feat/button toggle #101

merged 9 commits into from
Apr 5, 2019

Conversation

roll314
Copy link
Contributor

@roll314 roll314 commented Apr 1, 2019

Peter Kornuishin added 2 commits April 2, 2019 12:00
@lskramarov
Copy link
Contributor

Фокус не отображается при нажатом состоянии.

т.е. табом выбираем кнопку, затем нажимаем space и фокус пропадает.

@lskramarov
Copy link
Contributor

В остальном все отлично, правим эти два замечания и вливаем.

Peter Kornuishin added 2 commits April 4, 2019 10:43
@roll314
Copy link
Contributor Author

roll314 commented Apr 4, 2019

фокус поправлен

@mikeozornin
Copy link
Contributor

А что чинится? Что проверять?

На вид ок кроме отступов внутри кнопок слева и справа.

http://screenshots.ptsecurity.com/mozornin-2019-04-04_19-45-43.png
У отдельного тоггла на вид ок, а у тех, что по несколько штуки слева-справа маленькие отступы.

@roll314
Copy link
Contributor Author

roll314 commented Apr 4, 2019 via email

@roll314
Copy link
Contributor Author

roll314 commented Apr 4, 2019 via email

@pimenovoleg
Copy link
Member

А что чинится? Что проверять?

На вид ок кроме отступов внутри кнопок слева и справа.

http://screenshots.ptsecurity.com/mozornin-2019-04-04_19-45-43.png
У отдельного тоггла на вид ок, а у тех, что по несколько штуки слева-справа маленькие отступы.

укажи какие должны быть отступы по умолчанию.

@mikeozornin
Copy link
Contributor

По отступам кнопки и тоглы идентичны.

@lskramarov
Copy link
Contributor

По отступам:
после мержа нужно будет перенести логику отступов из mc-icon-button в mc-button и удалить mc-icon-button + удалить стайлеры для кнопок с разными размерами.

На данный момент специально не стали тянуть сюда знания о mc-icon-button, поэтому такие отступы.

Задача есть в итерации.

@pimenovoleg pimenovoleg merged commit 37abf64 into positive-js:master Apr 5, 2019
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.

4 participants