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

Adding "open new window" icon to "Create Dev Drive" button (#1334) #2062

Merged

Conversation

gregalantiu2
Copy link
Contributor

@gregalantiu2 gregalantiu2 commented Dec 16, 2023

Summary of the pull request

A request was made to clarify that clicking the "Create Dev Drive" button in the "Introducing Dev Home" page view takes you outside the app. @cinnamon-msft suggested that the same icon used in the "Machine Configuration" page should be added to convey this idea.

References and relevant issues

#1334

Detailed description of the pull request / Additional comments

In WhatsNewPage.xaml, some refactoring of the button element occurs. Child elements are added in order to accommodate both text and icon inside the button.

Additionally, I added a boolean property to the WhatsNewCard.cs called ShouldShowIcon which is used to toggle the Visibility property of the icon since currently the "Create Dev Drive" button is the only button we want the icon showing for.

In WhatsNewPage.xaml.cs, logic was added to set ShouldShowIcon to false except for the "Create Dev Drive" button.

Validation steps performed

Manually verified UI renders as expected
image

PR checklist

@gregalantiu2
Copy link
Contributor Author

@gregalantiu2 please read the following Contributor License Agreement(CLA). If you agree with the CLA, please reply with the following information.

@microsoft-github-policy-service agree [company="{your company}"]

Options:

  • (default - no company specified) I have sole ownership of intellectual property rights to my Submissions and I am not making Submissions in the course of work for my employer.
@microsoft-github-policy-service agree
  • (when company given) I am making Submissions in the course of work for my employer (or my employer has intellectual property rights in my Submissions by contract or applicable law). I have permission from my employer to make Submissions and enter into this Agreement on behalf of my employer. By signing below, the defined term “You” includes me and my employer.
@microsoft-github-policy-service agree company="Microsoft"

Contributor License Agreement

@microsoft-github-policy-service agree

@gregalantiu2
Copy link
Contributor Author

Hi @cinnamon-msft , does the image of the updated button look good (spacing, size, location, etc.)? If so, I'll publish this PR. Thanks!

@cinnamon-msft
Copy link
Contributor

I would suggest making the icon a bit smaller to match the font size, but otherwise I think this looks great!

@gregalantiu2
Copy link
Contributor Author

gregalantiu2 commented Dec 19, 2023

image

Yea, that's a great idea - it looks a lot better. I made it the same size (FontSize="12") as the same icon that appears on the "Feedback" settings page
image.

More than happy to make more adjustments if need be, just let me know 😄


NOTE: Just a heads up, I wanted to give clarity to this:
image

I amended a commit message which I then had to force-push

@gregalantiu2 gregalantiu2 marked this pull request as ready for review December 19, 2023 02:00
@EricJohnson327 EricJohnson327 added the Needs-Second Pull request that needs another approval label Jan 11, 2024
Copy link
Contributor

@vineeththomasalex vineeththomasalex left a comment

Choose a reason for hiding this comment

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

LGTM!

@EricJohnson327 EricJohnson327 merged commit ab5e6d6 into microsoft:main Jan 17, 2024
4 checks passed
@krschau krschau removed the Needs-Second Pull request that needs another approval label Jan 24, 2024
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.

Clarify information and behavior of Dev Drive card on "Introducing Dev Home" page
5 participants