moves styles from outdated branch

janmichek commented 6 years ago

Je tady jeste uvedeny stary nazev ikony 'Applications'. Po nasazeni https://github.com/keboola/indigo-ui/pull/202 lze prejmenovat na 'Application'

ujovlado commented 6 years ago

S tymto pockam az doriesime #202

ujovlado commented 6 years ago

Uz sa moze pouzit icon application.

janmichek commented 6 years ago

ok, checknu to jeste a upravim

ujovlado commented 6 years ago

Este mi napada, vidim, ze sa tu nejake css menia a nieco dokonca vyhadzuje. Ked nasadime tento PR prvy a az potom ten z kbc-ui, nebude to rozbite? :)

janmichek commented 6 years ago

Myslel jsem ze to pujde tak, ze releasnes novou verzi (5.x), potom k tomuto PR https://github.com/keboola/kbc-ui/pull/1813 uvedeme v packages.json ze ma pouzit prave predpripravenou verzi indiga (5.x) a nemel by byt zadny vypadek? Novy markup v kbc si hned veme novy styly, nebo?

ujovlado commented 6 years ago

No praveze nie, nefunguje to tak, preto sa radsej pytam. Riesime to uz dost dlho, ze by sa to malo robit spatne kompatibilne. :)

Indigo-ui samotne je funkcne. Cize ano, kludne sa moze pripravit verzia 5.x. My tam mame ale este pozostatok - csska sa registruju do KBC (repozitar indigo-ui-kbc).

T.j. ak to chceme "bezvypadkovo" releasuje sa to takto (uvazujme, ze nova verzia je 5.1):

pripravi sa zmena (idealne pridanie) cssiek v indigo-ui (kludne sa moze otagovat ako 5.1)
v indigo-ui-kbc sa updatne indigo-ui na 5.1
po otagovani v indigo-ui-kbc su csska deploynute do connection
!!! tu si v situacii, ze mas stary markup a nove csska, preto by to malo fungovat pre oba markupy
potom mozes v kbc-ui zmenit/vyhodit markup
nasledne sa mozu stare csska vyhodit a opat spravit release (napr. 5.1.1) a updatnut na tuto verziu v indigo-ui-kbc

Chapem, ze je to mrdacka naviac ale stava sa nam to pomerne zriedkavo.

Dolezite je si uvedomit, ze csska z indigo-ui sa neloaduju prostrednictvom kbc-ui - tam z toho vyuzivame len komponenty ale pomocou indigo-ui-kbc.

ujovlado commented 6 years ago

Cele toto sa odstrani s vyhodenim server render stranok z connection (na com pomaly zaciname robit), pretoze uz nebude nutne registrovat tie csska tam ale budu sa loadovat do kbc-ui.

janmichek commented 6 years ago

ok, to me uplne neakceleruje, ale budiz.

Nerozumim jen jedny veci. Proc se to uvadi sem? https://github.com/keboola/kbc-ui/blob/6989c1f6ef5afb504ecd895ff30302065c2f9a17/package.json#L7

ujovlado commented 6 years ago

Ako pisem, bavime sa o tom uz dlho - ak si pamatas niektore PRs sa uz kvoli tomu prerabali. Druha moznost je releasnut to nejako "naraz" (ale nikdy sa netrafis a niektori useri by to mali rozbite az do dalsieho reloadu).

Nerozumim jen jedny veci. Proc se to uvadi sem? https://github.com/keboola/kbc-ui/blob/6989c1f6ef5afb504ecd895ff30302065c2f9a17/package.json#L7

Aby sa v kbc-ui dali pouzit komponenty z indigo-ui - to je hlavny dovod, inac by to neslo a ostane to tak aj do buducnosti
A aby si videl tie csska aj lokalne (Ak uz je vsak napr. regnuta nova verzia v connection, v produkcii uz tie css su)

ujovlado commented 6 years ago

Toto teda riesit nejdem. Suvisiaci PR sa v kbc-ui zavrel.

keboola / indigo-ui

moves styles from outdated branch #210