search

microsoft / microsoft-ui-xaml

Windows UI Library: the latest Windows 10 native controls and Fluent styles for your applications
MIT License
6.27k stars 674 forks source link

Proposal: Update hover visual #953

Closed sravya03 closed 5 years ago

sravya03 commented 5 years ago

Summary

Update the hover visual to align more with the depth model. When item is hovered over, the item is lifted. When you get closer to the item, the color becomes lighter.

Rationale

Important Notes

Here are the scoped changes proposed. Ones with asterisk are places where change is proposed. image

Controls impacted:

Buttons Hyperlinks

Buttons Hyperlinks - Dark Theme

mdtauk commented 5 years ago

Fluent Web uses this elevate on hover design. Take a look at the Microsoft Store app pages

mdtauk commented 5 years ago

buttons

Here is a look at 4 different Microsoft buttons using Fluent Design.

And on the right is an idea how you could blend those ideas into a new design

chigy commented 5 years ago

Status update

This proposal is being closed in favor of the another proposal (#1054) that was created that better defines the scope of the ask.

BTW, I do believe the other proposal does cover the suggestion @mdtauk mentioned in one of the feedback.

@mdtauk , @sravya03 FYI

chigy commented 5 years ago

Status update

Re-opening this because more systematic update to the button (#1054) was tabled but we still want to do the visual update to remove the border during hover state in WinUI2.2 to bring the design more in line to be more coherent and familiar. This change and #839 should be done together to align the design amongst the set of controls.

mdtauk commented 5 years ago

@chigy Is there a rationale why the Slider hover is the Theme Base High colour, and not Accent Light 1/2 like with the hover on the Accent coloured Button/Checkbox/RadioButton/ToggleSwitch?

chigy commented 5 years ago

@mdtauk , you should open it as a proposal or idea. For this change, we are minimizing impact by not touching controls other than button related since we are trying to target WinUI2.2 release which is coming up shortly.

mdtauk commented 5 years ago

@chigy #1193 Done

msft-github-bot commented 5 years ago

:tada:This issue was addressed in #1171, which has now been successfully released as Microsoft.UI.Xaml v2.2.190822001-prerelease.:tada:

Handy links: