bratislava / bratislava/component-library
Button pre bratislava.sk
- Dominant language
- TypeScript
- Stars
- 1
- Forks
- 0
- PR merge metrics
- No merged PRs in 30d
Description
Predtým ako sa pustíme do prenesenia buttonu do knižnice by sme ho chceli mať implementovaný “dobre” v rámci jedného z projektov. Najbližšie k správnej implementácii je tento variant z konta - https://github.com/bratislava/konto.bratislava.sk/blob/master/next/components/forms/simple-components/ButtonNew.tsx. Čo by sme s ním chceli spraviť je dostať ho do projektu bratislava.sk a následne ho dokončiť do požadovaného stavu. Každý z bodov nižšie ideálne ako separátny PR po ktorom zmeny mergneme / otestujeme / deployneme:
- [x] vložiť “ButtonNew” z linku vyššie do bratislava.sk, ako náhradu za [dnešnú veľmi podobnú implementáciu](https://github.com/bratislava/bratislava.sk/blob/master/next/components/forms/simple-components/Button.tsx)
- [x] 🔧 updatnuť verziu react-aria-components na bratislava.sk na najnovšiu (`yarn update-react-aria` - vsetky react-aria libky musia byt synchronizovane co sa tyka verzie, inak spolu nefunguju)
- [x] prepísať Button do `react-aria-components` (alebo zakomponovat [ButtonContext](https://react-spectrum.adobe.com/react-aria/Button.html#hooks)). Zároveň nahradiť naše custom data atribúty tými z react arie (pozri [states](https://react-spectrum.adobe.com/react-aria/styling.html#states), a ako [tailwinde plugin](https://react-spectrum.adobe.com/react-aria/styling.html#tailwind-css))
- [ ] Button musi fungovat v roznych scenaroch, ako: link = button s hrefom, button s onPress funkciou, icon button (required aria label), nenastylovany button, custom stylovany button, v hlavnom menu, v kontextovom menu, v modali ako triger na otvorenie modalu (toto teraz nefunguje, lebo chyba ten ButtonContext)
- [ ] vyriešiť všetky TODOs a ľubovolné problémy ktoré v rámci jeho implementácie nájdete
- [ ] zbaviť sa ostatných Button implementácii v bratislava.sk, všade použiť iba tento nový komponent
- [ ] následne vytiahnuť komponent do tejto knižnice a nahradiť ho za knižničný v rámci bratislava.sk
v pripade potreby konzultovat s @radoslavzeman
Contributor guide
No contributing guide indexed for this repository
Assessment
This issue has not been assessed yet.