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

fix: display of square badge with small content size #111

Open
wants to merge 9 commits into
base: dev
Choose a base branch
from

Conversation

sawel24
Copy link
Contributor

@sawel24 sawel24 commented Apr 13, 2023

Connection with issue(s)

Resolve issue #110

Change the display of the badge with the content of small size, without changing all the functions while maintaining the proportions

Screenshots or Videos

Badge before changes :
image

Badge after changes :
image

sawel24 and others added 4 commits April 6, 2023 18:28
* feat: add triangle badge shape

* Add widget tests for triangle badge shape
* Update custom painter for all custom shapes to set fixed-size proportions

* Return default value of child content for verified account widgets

* Resize the shape depending on the child widget in the same proportions

* Fix git check errors

* Resize all types of shapes depending on the size child widget

* Add method for calculate default badge content padding value

* Change method for calculate default badge content padding value

* Add unit tests for calculateBadgeContentPadding method
@sawel24 sawel24 requested a review from yadaniyil April 13, 2023 16:25
@sawel24 sawel24 self-assigned this Apr 13, 2023
@86linksrain
Copy link

it worked,so i'll close this issues ,but i found if you chang '123' to '1' ,it able to recurrent(just in hot reload,when you restart it's ok),i don't know why.....
13

@sawel24
Copy link
Contributor Author

sawel24 commented Apr 14, 2023

@86linksrain I fixed this problem can you check with the new changes now?

@86linksrain
Copy link

👍🏻👍🏻👍🏻👍🏻 perfect

# Conflicts:
#	CHANGELOG.md
#	README.md
#	example/.flutter-plugins-dependencies
#	pubspec.yaml
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants