ui_kit/feat/button-components #134

Merged
hugo merged 35 commits from ui_kit/feat/button-components into master 2023-02-16 08:58:03 +00:00
Owner
  • FlatButton (plain/outlined)
  • SymbolButton (selectable)
  • SimpleIconButton (force icon child)
  • FileSelectionButton (can be invalid)
- [x] FlatButton (plain/outlined) - [x] SymbolButton (selectable) - [x] SimpleIconButton (force icon child) - [x] FileSelectionButton (can be invalid)
hugo force-pushed ui_kit/feat/button-components from c5e9259adc to 15ad037d47 2023-02-10 16:02:57 +00:00 Compare
hugo force-pushed ui_kit/feat/button-components from 881c8c9190 to 4695cf0618 2023-02-14 10:58:01 +00:00 Compare
hugo added 2 commits 2023-02-14 11:50:55 +00:00
Author
Owner

Move carousel_controller in another issue/pr because of his anatomy:

  • multiple buttons
  • controller (not only UI)
Move carousel_controller in another issue/pr because of his anatomy: - multiple buttons - controller (not only UI)
hugo changed title from WIP: ui_kit/feat/button-components to ui_kit/feat/button-components 2023-02-14 11:52:14 +00:00
hugo changed title from ui_kit/feat/button-components to WIP: ui_kit/feat/button-components 2023-02-14 11:52:50 +00:00
Author
Owner
  • need to add ThemeExtension support
  • enhance example with light/dark mode support
- need to add ThemeExtension support - enhance example with light/dark mode support
hugo added 1 commit 2023-02-14 13:21:44 +00:00
hugo force-pushed ui_kit/feat/button-components from 1703e5b4d0 to 198d83b663 2023-02-14 13:48:19 +00:00 Compare
hugo added 1 commit 2023-02-14 13:49:46 +00:00
hugo added 5 commits 2023-02-15 14:58:24 +00:00
hugo added 1 commit 2023-02-15 19:14:42 +00:00
hugo added 3 commits 2023-02-15 20:45:58 +00:00
hugo added 2 commits 2023-02-15 21:36:34 +00:00
hugo changed title from WIP: ui_kit/feat/button-components to ui_kit/feat/button-components 2023-02-15 21:37:18 +00:00
hugo added 1 commit 2023-02-15 21:40:26 +00:00
style(ui_kit): fix padding file selection
Some checks failed
continuous-integration/drone/push Build is failing
3a7b0abc58
malo approved these changes 2023-02-16 08:46:23 +00:00
Owner

Too many files to comment in the review :

  • Abstract text_resolver is very cool 👍 👍
  • Unless I'm mistaken, the style function override loses the styles calculated in the previous function. The abstract logic seems good, maybe review the switch case in the style_resolver implementations ?
  • Resolve function should be private in screen files ?
  • Export bloc should be seperate widget instead of method helper in mixin ?
Too many files to comment in the review : - Abstract text_resolver is very cool 👍 👍 - Unless I'm mistaken, the style function override loses the styles calculated in the previous function. The abstract logic seems good, maybe review the switch case in the style_resolver implementations ? - Resolve function should be private in screen files ? - Export bloc should be seperate widget instead of method helper in mixin ?
hugo merged commit 3a7b0abc58 into master 2023-02-16 08:58:03 +00:00
Sign in to join this conversation.
No Reviewers
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Wyatt-FOSS/wyatt-packages#134
No description provided.