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

Add new helper class - kbc-break-word #307

Closed natocTo closed 5 years ago

natocTo commented 5 years ago

Fixes: #183

related https://github.com/keboola/kbc-ui/issues/1354

Jde o to: že 

word-break: break-word;

není cross browser podporované. Firefox toto ignoruje (alespoň test v novém Firefoxu). V KBC-UI to je použito ale na dosti místech (viz i issue). Bude lepší asi použít tuto třídu a v případě jako fallback použít "break-all", než aby se třeba ovládací tlačítka posunuli mimo obrazovku.

Jinak break-word je použito i na jiných místěch zde v Indigo-ui. Tak by asi stálo za to to projet a případně taky přidat fallbeck nebo nějak jinak vyřešit.

natocTo commented 5 years ago

A není škoda nevyužít že Chrome to umí zalomcovat lépe? Viz ten "break-word". V Kbc-ui to je použito na dosti místech. Podle mě sám Chrome pokud neví co s tím tak přejde na "break-all", ale pokud to dokáže líp, tak mi přijde fajn mu to "dovolit".

janmichek commented 5 years ago

Jo, klidne. Nijak zasadne na tom netrvam, spis mi prislo lepsi ocekavat stejne chovani napric browsery.

natocTo commented 5 years ago

Jo. Já přemýšlel tak (aniž bych to více testoval), že třeba ten "break-word" neumí pouze Firefox, Tak pro co nejvíce prohlížečů zachována současné chování (předejte se změně chování) a jen tomu Firefoxu (připadně dalším pokud to neumí) řekneme ať použije to další aby to neuteklo mimo web jako v tom issue.

janmichek commented 5 years ago

za me ok

ujovlado commented 5 years ago

Este mi napadlo, ci by sme nepouzili overflow-wrap: https://developer.mozilla.org/en-US/docs/Web/CSS/overflow-wrap, vyzera, ze je to podporovane vcelku dobre.

natocTo commented 5 years ago

overflow-wrap jsem testoval hned na začátku, respektive jsem taky googlil co to šlo než jsem to tu poslal. Nevím zda jsem dělal něco blbě ale vůbec mi to nefungovalo.

ujovlado commented 5 years ago

pridam tam obe a uvidime.

ujovlado commented 5 years ago

Len doplnim, ten overflow-wrap mi fical len na komponentach, ktore maju sirku. Tak mozno si to niekde opodstatnenie najde.

Hore sa este spominal "fallback" - to sa moze (ak bue treba), poriesit tak, ze sa tam daju obidve tie triedy naraz.

natocTo commented 5 years ago

To je načasování, zrovna jsem šel toto zpracovat do kbc. Dík za info.