fix(popover): margin before footer (#UIM-602)

positive-js / mosaic

Angular components ✨ 📦🐣 ✨

https://mosaic.ptsecurity.com

MIT License

142 stars 40 forks source link

Closed VSiardziuk closed 2 years ago

VSiardziuk commented 2 years ago

Перенесен margin в mc-popover__content при наличии footer

lskramarov commented 2 years ago

@VSiardziuk на макетах этот токен UX задали специально:

@agalaktionov там действительно он нужен ?

agalaktionov commented 2 years ago

@VSiardziuk на макетах этот токен UX задали специально:

@agalaktionov там действительно он нужен ?

Этот токен заводили, чтобы содержимое в небольших поповерах с футерами, где нет скролла, выглядело более композиционно аккуратно.

Можем ли мы как-то отслеживать, есть скролл или нет, чтобы этот токен использовать или нет?

lskramarov commented 2 years ago

Договорились оставить токен, использоваться он будет в content и только если у поповера есть футер.

VSiardziuk commented 2 years ago

Договорились оставить токен, использоваться он будет в content и только если у поповера есть футер.

вернула токен. добавила отступ у content