styles for select project dropdown

[janmichek]

@ujovlado diky za feedback. Opravim to. S tim zpetne kompatibilnim resenim jsem si myslel ze to tak bude potreba udelat, no prob, udelam to.

Jenom me napadlo ze kdyz uz budu delat docasny reseni, jestli rovnou neodstranit .kbc-* classy a ty styly diky ti zmene taky nejsou uplne v nejlepsim stavu, prijde mi to zbytecne slozity. Co myslis?

[ujovlado]

Kludne, ale preferoval by som to nemixovat - t.j. tie kbc-* classes az v dalsom kolecku. Doriesme teraz nech to tu je BC a cim skor sa to nasadi

[janmichek]

Ok, souhlas

[ujovlado]

Potom este detail - ked to budes kontrolovat, tak su dve miesta:

• yarn start
• yarn start-projects-list

^ ProjectList komponenta sa pouziva na viac miestach ;)

[ujovlado]

@janmichek chces s tym pomoct? pozeram, ze v tom poslednom commite pribudli haldy kodu. Bolo to rozbite? Stacilo myslim len poriesit tie 2 veci co som pisal a pre istotu to znovu skontrolovat.

[janmichek]

Asi ne hele, dekuju. Ja vim co mam delat, jenom to potrebuju vsechno zkontrolovat pro kombinace noveho markupu stareho css, ... Obcas to chvili trva nez se to zbuildi. Jenom trpelivost .]

[janmichek]

@ujovlado Myslim, ze to mam pripraveno. Ten posledni commit jsem pridelaval prechodny reseni + jsem objevil ten project list o kterym jsem nevedel. Ty upravy nejsou kdovijak systematicky, nemyslim si ze to ma cenu mit hezky, kvuli par hodinam/dnim prerodu. Nicmene to funguje :)

Vyzkousel jsem kombinaci 'nove styly + stary markup' i 'nove styly + novy markup' a vse vypada jak ma. Takze ted kdyz releasneme toto https://github.com/keboola/kbc-ui/pull/1852 do produkce nebude nic rozpadleho. Pomoz mi to prosim jeste checknout, ale mam za to, ze jsem na nic nezapomel.

[janmichek]

@kacurez super, dik za check. Myslim ze ted pro to nemuzes nic udelat. Pripadne muzes checknout kombinaci 'nove styly + stary markup' ale neni to nutny

[ujovlado]

@janmichek kukam to zle alebo preco pribudlo v tom shame.less 12k riadkov?

[ujovlado]

[janmichek]

xDDD to je blbe. moment podivam se na to

[ujovlado]

no na to som sa vcera pytal ... "haldy kodu" :)

[janmichek]

aha, ja myslel ze i ty ostatni zmeny ti prisli jako halda. Fixnul jsem to Snad uz ok

[ujovlado]

Este pls rebase :) medzicasom som releasol to "purge styles" issue

[janmichek]

ok

[janmichek]

ehhh, tak jeste fixnu ten test

[janmichek]

Koukam ze je zapnutej linter na produkci, to jsem ani nevedel ze je .] Rekl bych ze v indigu jsou vetsi prohresky nez nepritomnost codestyle. Nevim jak mam udelat prechodny reseni aniz bych duplikoval selektory. Respektive vim, a je to hodne vysrany reseni (pres !important). Coz znamena to opet predelat, kontrolovat.

Pral bych si aby ty zmeny/sjednoceni sli snadno aplikovat. Tech zmen bude vyhledove hodne. Uz tak je to dost mentalne narocny + obchazeni autodeploye + obchazeni linteru.

Nejaky napady co s tim?

[ujovlado]

No ja som tam ten linter nedaval :) Kukni historiu, riesili ste to tusim s Tomasom.

Nie je to az tak mentalne narocne. Staci mysliet na spatnu kompatibilitu uz pri vyvoji/fixovani a nie az ked sa na to spytam v PR. Viac usilia to stoji preto, lebo to chceme robit bezvypadkovo. BTW, spomen si, ked sa s kazdym releasom indigo-ui (este kbc-bootstrap), musel robit release backednu. Ide to pomaly ale postupne to vylepsujueme. Casom to registrovanie na backend nebude potrebne.

[ujovlado]

K tomu linteru:

• no bud to oprav aby tam neboli tie chyby
• alebo zakazeme linter (a nechame ho tam len pre manualne spustanie)

[janmichek]

Ok, zatim mi pripada nejlepsi ten linter vypnout. O codestylu zatim nemuze byt rec, kdyz tam jsou vetsi prasarny :| mrknu se na to

[ujovlado]

Ja ten linter vypnem v mastri. Skor fixni ten konflikt v shame.less

[ujovlado]

Skoro som zabudol. Stylelint sa da predsa vypnut pre konkretne riadky - pozri ten commit. Pre taketo docasne veci je to vhodne ;)

[janmichek]

cool, ani jsem nevedel, ze mam takovou moznost. Takovy reseni mi staci, dik.

[janmichek]

Jeste to naposledy checknu po poslednich zmenach

[ujovlado]

ok, napis potom sem az to bude ready.

[janmichek]

@ujovlado za me je to ready k nasazeni. Jeste jsem opravil posledni styly a checknul sidebar i project list pro kombinaci 'nove styly + stary markup' a 'nove styly + novy markup'

[ujovlado]

ten rebase tuna bol celkom porod :) skusme to robit priebezne ...