Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

feat: UIKit chip component #568

Merged
merged 38 commits into from
Jun 24, 2024
Merged

feat: UIKit chip component #568

merged 38 commits into from
Jun 24, 2024

Conversation

dtscalac
Copy link
Contributor

@dtscalac dtscalac commented Jun 24, 2024 •
edited
Loading

Description

Implements a chip component from Figma.

Using a material provided Chip from flutter framework would be incorrect semantically, i.e. in VoicesChip we need a leading and trailing, in flutter's Chip it's called avatar and deleteIcon. Not all widgets we'll use for trailing will be for deleting.

Related Issue(s)

Closes #567

Screenshots

Screenshot 2024-06-24 at 14 53 48

Please confirm the following checks

  • My code follows the style guidelines of this project
  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • Any dependent changes have been merged and published in downstream module

coire1 and others added 30 commits June 11, 2024 09:15
@coire1 @stevenj
feat: Add VoicesAppBar (#531)
fed4da8 
* feat: Add new colors and definitions to Catalyst Theme.

* feat: Add VoicesAppBar and sub-actions.

* feat: Add CatalystVoicesIcons set.

* feat: Update actions components to use CatalystVoices icons.

* feat: Add VoicesAppBar widget.

* docs: Add docs for actions widgets.

* feat: Add search button.

* feat: Update unlock icon to use CatalystVoices icon set.

* feat: Add SearchButton to VoicesAppBar.

* feat: Restore `flutter_adaptive_scaffold` version and add `catalyts_voices_brands`.

* feat: Move icons to right for action buttons.

* chore: Removed unused import.

* Fix: Point `flutter-ci` to main branch.

* chore: Change typedef of button callbacks.

* chore: Rename chevron icons.

---------

Co-authored-by: Steven Johnson <[email protected]>
@dtscalac
fix: Fix .earthlyignore excluded directories/files (#536)
7bb6c8d 
* Update .earthlyignore

* chore: ignore Earthfile in .earthfileignore
@dtscalac
fix: catalyst cardano support for NFTs (#537)
9054387 
* feat: add catalyst_cardano wallet api

* feat: add error handling

* fix: tests

* docs: update toc

* docs: fix readme

* fix: decrement package version

* feat: add missing import

* fix: assets path

* fix: add missing dependency

* feat: add example

* feat: add error case

* refactor: rename getCardanoWallets

* fix: method name

* chore: ignore flutter plugins generated files

* feat: cardano wallet api example (#523)

* feat: add missing import

* fix: assets path

* fix: add missing dependency

* feat: add example

* fix: method name

* fix: balance with multi asset tokens

* feat: parse value from utxo

* fix: utxo formatting

* feat: update example

* feat: code adjust

* fix: balance

* feat: parse multi assets

* feat: add multi assets arithmetics

* feat: adjust code to latest changes

* fix: assets cleanup

* feat: support native tokens in utxos

* fix: multi asset change calculation

* fix: change calculation

* refactor: cleanup code

* Update .earthlyignore

* chore: ignore Earthfile in .earthfileignore

* docs: update example

* docs: update readme

* refactor: cleanup code

* docs: readme

* docs: improve grammar

* refactor: rename value to balance

* docs: fix

---------

Co-authored-by: Dominik Toton <[email protected]>
@dtscalac
feat: cardano wallet cip 95 (#538)
04f5c3a 
* feat: add catalyst_cardano wallet api

* feat: add error handling

* fix: tests

* docs: update toc

* docs: fix readme

* fix: decrement package version

* feat: add missing import

* fix: assets path

* fix: add missing dependency

* feat: add example

* feat: add error case

* refactor: rename getCardanoWallets

* fix: method name

* chore: ignore flutter plugins generated files

* feat: cardano wallet api example (#523)

* feat: add missing import

* fix: assets path

* fix: add missing dependency

* feat: add example

* fix: method name

* fix: balance with multi asset tokens

* feat: parse value from utxo

* fix: utxo formatting

* feat: update example

* feat: code adjust

* fix: balance

* feat: parse multi assets

* feat: add multi assets arithmetics

* feat: adjust code to latest changes

* fix: assets cleanup

* feat: support native tokens in utxos

* fix: multi asset change calculation

* fix: change calculation

* refactor: cleanup code

* Update .earthlyignore

* chore: ignore Earthfile in .earthfileignore

* docs: update example

* docs: update readme

* refactor: cleanup code

* docs: readme

* feat: add cip 95

* fix: cip95 deserialization

* refactor: formatting

* docs: improve grammar

* refactor: rename value to balance

* docs: fix

---------

Co-authored-by: Dominik Toton <[email protected]>
@dtscalac
docs: add common issues docs (#548)
77c5222 
* docs: add common issues docs

* docs: fix readme formatting
@dtscalac
feat: use theme builder
8b8c4fd
@dtscalac
chore: enable earthly files
91a0034
@dtscalac
chore: enable disabled earthly target
06f18be
@dtscalac
Merge branch 'feat/frontend' into feat/navigation-drawer
d2826c7
@dtscalac
chore: rerun target without cache
723ef89
@dtscalac
chore: no-cache earthly target
a3f010f
@dtscalac
chore: trigger rebuild
667966d
@dtscalac
Merge branch 'main' into feat/frontend
2381dd6 
# Conflicts:
#	catalyst-gateway/Earthfile
@dtscalac
Merge branch 'main' into feat/frontend
6a41db4
@stevenj
Merge branch 'main' into feat/frontend
9b71b00
@dtscalac
Merge branch 'feat/frontend' into feat/navigation-drawer
e2c2442
@dtscalac
refactor: extract widget
97027bd
@minikin
Merge branch 'main' into feat/frontend
458f783
@dtscalac
Merge branch 'main' into feat/navigation-drawer
ff9fe21
@dtscalac
Merge branch 'main' into feat/frontend
8648ad2
@dtscalac
Merge branch 'main' into feat/frontend
09169cf
@dtscalac
Merge branch 'feat/frontend' into feat/navigation-drawer
a09074e
@dtscalac
feat: uikit example
dbfafab
@dtscalac
Merge branch 'main' into feat/navigation-drawer
42d491a
@dtscalac
Merge branch 'main' into feat/navigation-drawer
324deda
@dtscalac
Merge branch 'main' into feat/navigation-drawer
4882988
@dtscalac
feat: voices drawer
6a37793
@dtscalac
feat: finish nav drawer
b276705
@dtscalac
feat: add docs and voices list tiles
43264a4
@dtscalac
feat: update to named navigation to preserve route when hot restarting
af7035c
@dtscalac dtscalac added review me PR is ready for review dart Pull requests that update Dart code labels Jun 24, 2024
@dtscalac dtscalac added this to the M4: Voting & Delegation milestone Jun 24, 2024
@dtscalac dtscalac self-assigned this Jun 24, 2024
@dtscalac dtscalac requested a review from minikin as a code owner June 24, 2024 12:57
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Jun 24, 2024 •
edited
Loading

✅ Test Report | ${\color{lightgreen}Pass: 91/91}$ | ${\color{red}Fail: 0/91}$ |

Base automatically changed from feat/navigation-drawer to main June 24, 2024 13:07
@minikin minikin enabled auto-merge (squash) June 24, 2024 13:36
minikin
minikin approved these changes Jun 24, 2024
View reviewed changes
Copy link
Collaborator

@minikin minikin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@minikin minikin merged commit cd064ad into main Jun 24, 2024
27 checks passed
@minikin minikin deleted the feat/uikit-chips branch June 24, 2024 13:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@minikin minikin minikin approved these changes

Assignees

@dtscalac dtscalac

Labels
dart Pull requests that update Dart code review me PR is ready for review
Projects
Catalyst
Archived in project
Milestone
M4: Voting & Delegation
Development

Successfully merging this pull request may close these issues.

🛠️ [TASK] : UI kit chip component
4 participants
@dtscalac @minikin @coire1 @stevenj