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

Update toolbar_widget.dart #339

Open
wants to merge 8 commits into
base: master
Choose a base branch
from
Open

Conversation

PROGrand
Copy link

@PROGrand PROGrand commented Oct 26, 2022

int.parse(colorList[1]!, radix: 16) fails in some cases. For example: colorList[1] == 'transparent'.

@tneotia
Copy link
Owner

tneotia commented Nov 9, 2022

That makes sense, thanks! Is it possible that other string based colors could be added? Not sure - but what if the color is set as 'blue' or 'red' for example.

@vmurzik
Copy link

vmurzik commented Apr 10, 2023

@tneotia I think there is more work to be done on this front related to supporting additional colors, but at the same time merging this PR fixes an immediate issue. If we could merge this now and open a ticket for a future improvement for more color support I'd be happy to pick it up once I get more time.

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.

4 participants