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

Investigate enabling brush transitions on new styles that were disabled by accent brush update #3913

Closed
ranjeshj opened this issue Jan 9, 2021 · 3 comments
Labels

Comments

@ranjeshj
Copy link
Contributor

ranjeshj commented Jan 9, 2021

Accent brush PR #3909 is removing brush animations since we hit an issue, This issue is to track investigating a way to enable that back.

@ghost ghost added the needs-triage Issue needs to be triaged by the area owners label Jan 9, 2021
@mdtauk
Copy link
Contributor

mdtauk commented Jan 9, 2021

Could it be an issue with the ColorAnimationByKeyFrames

What about something like this

<ColorAnimation Storyboard.TargetName="ControlPartColorBrush"
                Storyboard.TargetProperty="Color" Duration="0:0:0.083" 
                By="{ThemeResource ControlColorChange}"/>

Maybe @JustinXinLiu could offer an insight, he has done a lot of Xaml animation stuff

Essentially, what is needed is for the system provided Accent Colour to be given to WinUI - and for that to generate the needed AccentColor and AccentAAColor variations as both brushes and colours in the WinUI ThemeResource Dictionaries.

At the moment there are System Colours provided, I think as Brushes - but I believe @chigy mentioned some experiments with the Accent Colour generation - so maybe this is something that will be changing in the future? Will that be WinUI 3 only, or could that change make it to WinUI 2.X also?

@karkarl
Copy link
Contributor

karkarl commented Jan 11, 2021

Could it be an issue with the ColorAnimationByKeyFrames

What about something like this

<ColorAnimation Storyboard.TargetName="ControlPartColorBrush"
                Storyboard.TargetProperty="Color" Duration="0:0:0.083" 
                By="{ThemeResource ControlColorChange}"/>

Unfortunately, this syntax also requires the TargetProperty to be a colour, not a brush, so also does not work (Docs). And you are right, some investigation is needed to re-enable the colour transitions.

@ranjeshj ranjeshj added area-Styling team-Controls Issue for the Controls team labels Jan 12, 2021
@ranjeshj ranjeshj added this to the WinUI 2.6 milestone Jan 12, 2021
@ranjeshj ranjeshj removed the needs-triage Issue needs to be triaged by the area owners label Jan 12, 2021
@ranjeshj ranjeshj removed this from the WinUI 2.6 milestone Mar 22, 2021
@github-actions
Copy link

This issue is stale because it has been open 180 days with no activity. Remove stale label or comment or this will be closed in 5 days.

@github-actions github-actions bot closed this as not planned Won't fix, can't repro, duplicate, stale Aug 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants