Skip to content

Project feeedback #1

Description

@Haaxor1689
  • Styling bug, orezany vrch labelu input fieldu na vrchu dialogu
  • Bolo by fajn pridat daky gap medzi horizontalny stack inputov ako su napr. filtre taskov alebo duration/unit v edite category
  • Zvlastne stylovanie indikatora "aktivnych filtrov", nebolo mi hend jasne co ma robit. Taketo icon buttony si zasluzia nejaky tooltip urcite.
  • Po zmazani kategorie, tasky k nem priradene ostanu v trochu rozbitom stave
  • Planned/In progress/Finished filter na vrchu stranky je tiez dost neintuitivny, na toto sa pouziva checkbox group
  • Nechapem moc pointu Icons.tsx, preco ste nepouzili MUI icons?
  • Na home page nacitavate data pre oba taby naraz aj ked niesu hned potrebne. Lepsie by bolo nacitat kategorie az vtedy ked uzivatel naozaj otvori ten tab.
  • Namiesto chainovania viacej filter callov v Home.tsx:144 by bolo lepsie spravit len jedne filter a spojit tieto podmienky bool operatorom.
  • Neviem preco defaultCategoryDict inicializujete v tele componenty na prazdny objekt a potom manualne pridavate jednotlive atributy. Toto moze byt complet static object mimo tela componenty.
  • TaskStatus by som nepovedal ze patri do components folderu. Taktiez ani CalculateDuration.tsx
  • TaskPreview.tsx:64 ste mohli pouzit toLocalizedDateString alebo nejaku formatovaciu funkciu z dayjs
  • Import dayjs/locale/en-gb staci mat urcite len raz v roote aplikacie. Tak isto aj LocalizationProvider ma byt urcite len jeden v celej appke.
  • Namiesto toho aby ste z ModeContext museli exportovat mode, mozete pouzivat useTheme a z neho precitat aky mode ma appka zapnuty.
  • EditCategory a EditTask su uplne zbytocne componenty, mozete piramo pouzit tie Add verzie.
  • V StatusFilter.tsx:24 namiesto capitalize pomocou js, ste mohli pouzit bud tailwind uppercase helper.
  • Aj ked je super ze ste si vyskusali trochu tailwind, kedze uz v projekte bol pouzity mui/styled, vo vysledku to asi neprinieslo tolko co by mohlo pouzitie vyhradne jedneho z nich.

*23

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions