positive-js / mosaic

Angular components ✨ 📦🐣 ✨
https://mosaic.ptsecurity.com
MIT License
142 stars 40 forks source link

feat(docs) : new popover docs (#UIM-861) #795

Closed bogdanhpk closed 2 years ago

PositiveJS commented 2 years ago

Preview docs changes for 58e8b90 at https://positive-js.github.io/mosaic-previews/pr795-58e8b90/

agalaktionov commented 2 years ago

предлагаю вместо серых алертов использовать голубые, как например в валидации https://mosaic.ptsecurity.com/validation/overview

PositiveJS commented 2 years ago

Preview docs changes for cb9d932 at https://positive-js.github.io/mosaic-previews/pr795-cb9d932/

vercel[bot] commented 2 years ago

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/ptsecurity/mosaic/9cWXKNFpVwFxaiDT944BD2Sewjxt
✅ Preview: Failed

[Deployment for 7fff937 failed]

PositiveJS commented 2 years ago

Preview docs changes for 8c2c56b at https://positive-js.github.io/mosaic-previews/pr795-8c2c56b/

PositiveJS commented 2 years ago

Preview docs changes for 2dcc39d at https://positive-js.github.io/mosaic-previews/pr795-2dcc39d/

PositiveJS commented 2 years ago

Preview docs changes for a29bcd7 at https://positive-js.github.io/mosaic-previews/pr795-a29bcd7/

PositiveJS commented 2 years ago

Preview docs changes for 059f7c0 at https://positive-js.github.io/mosaic-previews/pr795-059f7c0/

oburdasov commented 2 years ago

@agalaktionov Привет. Сейчас в разделе поведение при скролле пример не соответствует содержанию.

Этот пример о том, что можно управлять видимостью поповера вручную. По-моему нужно либо добавить соответствующий раздел, и положить этот пример туда, либо удалить его.

А если для раздела про скролл нужен пример, то возможно стоит добавить такой, в котором показать, как сделать, чтобы поповер пропадал при скролле

agalaktionov commented 2 years ago

@oburdasov если пример неправильный, давай уберем его.

И сделаем пример в котором по кнопке будем вызывать поповер, который по умолчанию будет скролится со страницей, и под кнопкой добавим чекбокс «Скрывать при скроле» при установке, которого поповер будет скрываться

PositiveJS commented 2 years ago

Preview docs changes for 7ab5bcc at https://positive-js.github.io/mosaic-previews/pr795-7ab5bcc/

oburdasov commented 2 years ago

@agalaktionov @lskramarov Добавил пример для скролла https://positive-js.github.io/mosaic-previews/pr795-7ab5bcc/popover/overview

Можно смотреть финальный результат

PositiveJS commented 2 years ago

Preview docs changes for 7fff937 at https://positive-js.github.io/mosaic-previews/pr795-7fff937/

PositiveJS commented 2 years ago

Preview docs changes for f3cdcd5 at https://positive-js.github.io/mosaic-previews/pr795-f3cdcd5/

PositiveJS commented 2 years ago

Preview docs changes for ca4f931 at https://positive-js.github.io/mosaic-previews/pr795-ca4f931/

PositiveJS commented 2 years ago

Preview docs changes for ddef79b at https://positive-js.github.io/mosaic-previews/pr795-ddef79b/

lskramarov commented 2 years ago

@agalaktionov добавил исправления от техрайтеров, посмотри еще раз, если все ок, поставь апрув.

sonarcloud[bot] commented 2 years ago

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

PositiveJS commented 2 years ago

Preview docs changes for 6599952 at https://positive-js.github.io/mosaic-previews/pr795-6599952/