Add AccentToggleButton#58
Conversation
|
I have also incorporated the button in the sample app |
|
Thanks for contributing and welcome on board! Currently I don't have the opportunity to test your changes, but I hope I can get to it within the next few days. Until then I can only look at the code on GitHub. So what I noticed is that you produced an immense amount of formatting changes within your commit. It is kind of hard for me to spot the actual changes you did because they are surrounded by ~300 lines of changes where just a single space (' ') has been added. There are also changes like these in there: <!-- from -->
Margin="{adonisUi:Space 1, 0, 0, 0}"
<!-- to -->
Margin="{adonisUi:Space 1,
0,
0,
0}"... which actually looks worse in my opinion. I don't really mind if you want your lines of xaml to end with space like I hope I do not sound too harsh here. Your changes are really welcome in general and adding an AccentToggleButton makes totally sense. Okay, so how do you get rid of these formatting changes as they are already commited? I think the best option would be to reset you HEAD using Hope this helps. |
Currently the togglebutton checked state is hard to see. Instead of changing the existing design I decided to createthe accented toggle button as a new named style.
This style is based on the current toggle button - but for that to work properly I had to split the base style into two separate styles. The second one only has the triggers which I do not want in the accented style because the accent has no layers.