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.

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

Implement new SmallButton type widget #119

Merged
merged 10 commits into from
Aug 12, 2024
Merged

Conversation

JvnSlv
Copy link
Contributor

@JvnSlv JvnSlv commented Aug 6, 2024

Overview

This PR implements a new SmallButton type, SmallButtonType.icon, for displaying Icon buttons

Checklist

Implementation

  • Implementation matches ticket acceptance criteria and technical notes
  • Manually tested against Acceptance Criteria
  • UI checked in Light / Dark mode

Stability

  • Checked if changes affect any features and verified affected features work as expected
  • Added unit tests for new code and verified existing tests work as expected

Code quality

  • Updated CHANGELOG.md, README.md and package versions in pubspec.yaml
  • Dependencies are updated to latest versions or new tickets are created if there are breaking changes or deprecations
  • If an unrelated part of the codebase needs to be updated or refactored, create tickets with proposed changes

@JvnSlv JvnSlv requested a review from StanevPrime August 6, 2024 11:23
@JvnSlv JvnSlv self-assigned this Aug 6, 2024
@JvnSlv JvnSlv requested a review from StanevPrime August 6, 2024 14:30
@JvnSlv JvnSlv requested a review from StanevPrime August 9, 2024 07:28
Copy link
Member

@StanevPrime StanevPrime left a comment

Choose a reason for hiding this comment

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

Please add golden tests with a two colors in light and dark mode

@JvnSlv JvnSlv merged commit f11a3af into develop Aug 12, 2024
4 of 5 checks passed
@JvnSlv JvnSlv deleted the feature/small-button-icon-type branch August 12, 2024 11:07
@JvnSlv JvnSlv assigned RomanovaPrime and unassigned JvnSlv Aug 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Published
Development

Successfully merging this pull request may close these issues.

3 participants