Inline edit input kbc-ui -> indigo-ui

[ujovlado]

Cielom je dostat inline edit input do indigo ui, tak aby sa to dalo pouzivat v kbc-ui ako "externa" komponenta.

Jedna sa o tento input, ktory ma 2 stavy: staticky a editacny.

Momentalne su tam nejake problemy, co by sa mali fixnut:

• Napr. pri dlhsom nazve, je text skryty pod "cancel" buttonom.
• su tam inline styles a pod.

Je to tato komponenta: https://github.com/keboola/kbc-ui/blob/303e6666b62af836d4fdd07d16b683874fe0802e/src/scripts/react/common/InlineEditTextInput.coffee

Nemal by to byt len prepis to jsx ale aj nejake upratanie. Bolo by fajn ak by sa nam podarilo +- zachovat API, nech sa to da lahko pouzit.

[natocTo]

Dnes bych rád udělal toto.

Ono je v dlouhodobějším záměru převést více těchto common komponent pod Indigo-ui? Třeba tady se používá také Tooltip a Input, který by se asi v tom případě mohl přehodit první. Abych ho mohl použít u této komponenty. Nebo pokud tyto komponenty do Indigo-UI nepůjdou, můžu použít přímo react-bootstrap, ale aby jsem tu neměl zbytečné duplikace.

Potom tu jsou definované tři komponenty v jednou souboru. Asi bude křičet eslint. Ale jsou to malé komponenty určené asi jen sem a myslím že by to šlo spojit v jednu. Pokud by to pak vypadalo blbě tak bych vymyslel něco jiného.

EDIT: aha, tak eslint tu není koukám, tak křičet nebude :)

[ujovlado]

Ono je v dlouhodobějším záměru převést více těchto common komponent pod Indigo-ui?

ano

Třeba tady se používá také Tooltip a Input, který by se asi v tom případě mohl přehodit první. Abych ho mohl použít u této komponenty. Nebo pokud tyto komponenty do Indigo-UI nepůjdou, můžu použít přímo react-bootstrap, ale aby jsem tu neměl zbytečné duplikace.

rovno pouzi react bootstrap - je to ako peerDependency, malo by to byt ok.

Potom tu jsou definované tři komponenty v jednou souboru. Asi bude křičet eslint. Ale jsou to malé komponenty určené asi jen sem a myslím že by to šlo spojit v jednu. Pokud by to pak vypadalo blbě tak bych vymyslel něco jiného.

kludne to spoj v jednu

EDIT: aha, tak eslint tu není koukám, tak křičet nebude :)

nejaky tam myslim je built-in :)

[janmichek]

Ono je v dlouhodobějším záměru převést více těchto common komponent pod Indigo-ui? Třeba tady se používá také Tooltip a Input, který by se asi v tom případě mohl přehodit první. Abych ho mohl použít u této komponenty. Nebo pokud tyto komponenty do Indigo-UI nepůjdou, můžu použít přímo react-bootstrap, ale aby jsem tu neměl zbytečné duplikace.

Jo, to je to kam smerujeme. Udelat knihovnu komponent a aplikace skladat z indigo-ui komponent. Nuti nas to i na jakousi "ui first" dev workflow. Developeri pak nemusi stylovat s komponentami kdyz delaji na svoji issue, naopak si vyzadaji ui podklady.

Do jaky miry dedit komponenty z komponent, atomizovat, hierarchizovat zatim nevim (napr Input -> SearchBar -> SearchResults) . Prozatim bude vitezstvi kdyz budeme mit vetsi ci mensi UI komponenty zdokumentovany v Indigu.