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 Utilities page design to align with designer feedback #2905

Merged
merged 6 commits into from
May 15, 2024

Conversation

dabhattimsft
Copy link
Contributor

@dabhattimsft dabhattimsft commented May 14, 2024

Summary of the pull request

This PR updates design of utilities page to remove the split button and add a toggle for admin launch. This change also updates descriptions to strings written by the content writer and links to go links instead of aka.ms.

image

References and relevant issues

Detailed description of the pull request / Additional comments

Validation steps performed

Manually validated

PR checklist

  • Closes #xxx
  • Tests added/passed
  • Documentation updated

HorizontalAlignment="Center"
IsOn="{x:Bind ViewModel.LaunchAsAdmin, Mode=TwoWay}"
Visibility="{x:Bind ViewModel.LaunchAsAdminVisibility, Mode=OneWay}"
OnContent="Launch as administrator"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Both On and Off use the same content? This tells me that the program will still launch as admin even when toggled off.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

To me, it is like - turn it on to launch as admin. i.e. to me, the text doesn't represent the current state of the switch but what would happen if turn it on.

@krschau krschau added this to the Dev Home v0.14 milestone May 14, 2024
@dabhattimsft dabhattimsft force-pushed the user/dabhatti/utilitiesDesign branch from f921f07 to a9c6053 Compare May 15, 2024 20:16
@dabhattimsft dabhattimsft merged commit 49b67fb into main May 15, 2024
4 checks passed
EricJohnson327 pushed a commit that referenced this pull request May 16, 2024
* strings and url

* visibility

* pr feedback

* pr feedback

* string and icon

* merge conflict

---------

Co-authored-by: Darshak Bhatti <[email protected]>
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.

6 participants