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

Badge must have a different look when disabled #2977

Closed
michaelmairegger opened this issue May 25, 2017 · 5 comments
Closed

Badge must have a different look when disabled #2977

michaelmairegger opened this issue May 25, 2017 · 5 comments
Assignees
Milestone

Comments

@michaelmairegger
Copy link
Contributor

What steps will reproduce this issue?

The Badge has no diffent appearance if the object itself or any parent visual element are disabled:

Expected outcome

Badge should look different if disabled

current proposed
badge-currentbehavior badge-newbehavior

Implementation

https://github.com/xxMUROxx/MahApps.Metro/tree/feature/Badge-Disabled-Style

If desired I will create a PR with the above linked branch.

@thoemmi
Copy link
Collaborator

thoemmi commented May 26, 2017

Can a badge be disabled? It's not interactive anyway. The number shown on the badge is still relevant.

Anyway, if a different appearance is desired, I wouldn't change the opacity but the color. The button shining through the badge in your proposal looks strange. I would prefer a solid badge background, something gray-ish maybe.

@amkuchta
Copy link
Contributor

I'm with @thoemmi (I'm glad I wasn't the only one who took issue with the button showing through). I think a grey would be awesome, but it would have to be a different grey than the control that the badge adorns

@michaelmairegger
Copy link
Contributor Author

michaelmairegger commented May 29, 2017

@thoemmi since a badge has no command, yes it is not interactive. But it looks strange if the parent control is disabled and the badge has the bright color (accented if default).
I like your and @amkuchta's idea.

badgebackgroundgray

@michaelmairegger
Copy link
Contributor Author

@punker76 what do you think?

@punker76 punker76 added this to the 1.6.0 milestone Sep 8, 2017
@punker76 punker76 self-assigned this Sep 8, 2017
@punker76
Copy link
Member

@xxMUROxx I've added your changes from your branch. thx! 👍 I only renamed the brush to MahApps.Metro.Brushes.Badged.DisabledBackgroundBrush

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

No branches or pull requests

4 participants