search

keboola / indigo-ui

Indigo UI library, its CSS and React components. Also Styleguide app showing usages of this library.
https://indigo.keboola.com
MIT License
5 stars 0 forks source link

Unify alerts #354

Open ujovlado opened 5 years ago

ujovlado commented 5 years ago
ujovlado commented 5 years ago

screenshot_2019-01-02_16-53-03

ujovlado commented 5 years ago

problem je ze kazdy si to spravil tak nejak po svojom ... pritom stacilo drzat sa bootstrapu. :)

natocTo commented 5 years ago

Teď to vidím tak že jsou dvě komponenty. Indigo-ui AlertBlock a bootstrap Alert. Z toho bude teda jedna? AlertBlock by pak musel mít teda něco jako "size" props abych mohl nastavit něco menšího + k tomu dodělat css.

Pokud zůstanou dvě tak stačí v KBC-UI vyhodit použití tříd "alert" a využít všude react-bootstrap komponentu. Pak ale budu mít právě "jednobarevný" text apod. Což nevím zda je žádoucí.

Teď to beru tak že obyčejný Alert je takové malé hlášení nebo varování. A AlertBlock je obecně důležitější zpráva. Spíš použitý někde na overview nebo indexu stránky. Přitom malý alert třeba u formulářů, nebo v modalu atd.

Další varianta je ponechat obě možnosti jako do teď. Jen upravit CSS pro obyč alerty aby nebyl ten stejně barevný text.