Skip to content

feat(modal): Component Modal #30

Merged
merged 8 commits into from
Sep 4, 2018
Merged

feat(modal): Component Modal #30

merged 8 commits into from
Sep 4, 2018

Conversation

pimenovoleg
Copy link
Member

No description provided.

@mikeozornin
Copy link
Contributor

Возможно сейчас что-то было проверять рано, я не знал что, проверил побольше.

http://d.mikeozornin.ru/sN3uLw
Давайте нигде не использовать неправильный шрифт, даже в примерах. Вообще не делать примеры неаккуратными. Это создает ощущение, что фреймворк сделан немного на коленке, хотя это же не так. Он заботливо и с любовью сделан. Хочется подобного ощущения от доков, даже если их видят только разработчики.

http://d.mikeozornin.ru/PRtdxk
Тут кнопки слиплись.

http://d.mikeozornin.ru/jW4VWG
В подтверждениях верхний отступ 16 пк, а надо 24 пк. Если сделать 16пк (как в модалке с заголовком), то текст подтверждения очень близко к краю, неаккуратно выглядит.

В модальных окнах не хватает навигации ← → по терминальным кнопкам. Пункт есть в чеклисте:

По окнам подтверждения дополнительно можно ходить ← → по терминальным кнопкам.

http://d.mikeozornin.ru/8bFsx3
Yes, No в примере надо заменить на Delete, Cancel, чтобы сразу показывать, что кнопок Yes, No быть не должно. Если Yes No примелькается, то потом будет ощущение «где-то видел».

http://d.mikeozornin.ru/0KAiXn
Этот пример тоже надо аккуратно сверстать: нужный шрифт, кегли, стили, регистр текста. Кнопку внутри лучше сделать секондари, чтобы праймари была одна на странице.

http://d.mikeozornin.ru/PSBRjJ
Опечатка в title

http://d.mikeozornin.ru/PkCXj9
Здесь тоже Yes, No.
Тоже типографика внутри.
Неправильные отступы сверху и снизу от края браузера до модального окна.

Во всех примерах модальное окно показывается не на той высоте.

Последний пример совершенно странный. Какие-то модальные окна в фоне, у них сломаны кнопки. Не понял как и что.

http://d.mikeozornin.ru/rFqfgZ
Модальное окно может создавать много множество оверлеев, хотя оверлей должен быть не больше одного.
Воспроизведение: попасть фокусом на кнопку «Template», нажать пробел 10 раз.
Это происходит ещё и от того, что модальное окно не перехватывает фокус. Появление модального окна не сбрасывает фокус с кнопки под ним и даже при показывающемся модальном окне ту кнопку под модальным окном можно нажать. Надо как-то блокировать ввод и фокус на элементы управления под модальным окном.

Анимация пропадания окна должна быть обратная к появлению. Сейчас пропадает мгновенно. Возможно надо сделать немного разную на условую отмену и условный сабмит.

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

http://d.mikeozornin.ru/I4TjVx
Если сжать экран по высоте, то модальное окно сильно сжимается

Вместо mc-modal-mask мне кажется лучше назвать его mc-modal-overlay.
Оверлей 0.5 опасити, а не 0.65.
Ещё у него в кода зашит цвет 0, 0, 0. Не знаю, может это не ок.

@pimenovoleg pimenovoleg force-pushed the feature/modal branch 2 times, most recently from 43571af to 001fd59 Compare August 31, 2018 11:03
Squashed commit:

[e19991f] chore: fixed paddings

[04be966] chore: added close icon

[e0fcadf] updated autoFocus and unit tests

[c4c16f6] added ESC

[8d04868] added theme

[3b222b9] update examples

[fda416e] update examples (+7 squashed commit)

Squashed commit:

[5367c8e] update check and dev

[94edb15] update examples

[e1a05bb] updated unit tests

[0df30b6] update styles

[29593a3] update styles (+11 squashed commit)

Squashed commit:

[06bbe01] remove InputBoolean

[f460201] updated code style

[dd9b61a] updated wallaby config

[6da021f] fix unit

[7a54c1e] fix unit

[5a169ef] fix build

[455cff9] updated modal confirm

[7735fdf] added example with component

[93c0299] added mc-buttons

[78cad3c] added animation

[205e841] added some styles

[fe2143d] added some styles

[2c17514] added some styles

[9686fe0] added modal
@pimenovoleg pimenovoleg changed the title [WIP] feat(modal): component Modal [WIP] feat(modal): Component Modal Sep 1, 2018
@pimenovoleg pimenovoleg changed the title [WIP] feat(modal): Component Modal feat(modal): Component Modal Sep 1, 2018
@pimenovoleg pimenovoleg merged commit 2971f19 into master Sep 4, 2018
@pimenovoleg pimenovoleg deleted the feature/modal branch September 4, 2018 11:39
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.

2 participants