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

Border to PreferenceView #14078

Merged

Conversation

dnenov
Copy link
Collaborator

@dnenov dnenov commented Jun 15, 2023

Purpose

A visual update to the Preferences View adding a white (WorkspaceBackgroundHomeBrush) 2px border. This should increase the visibility of the control and make it easier for users to make out the actual window when overlapping with other UI elements.

Screenshot of changes

image

Declarations

Check these if you believe they are true

  • The codebase is in a better state after this PR
  • Is documented according to the standards
  • The level of testing this PR includes is appropriate
  • User facing strings, if any, are extracted into *.resx files
  • All tests pass using the self-service CI.
  • Snapshot of UI changes, if any.
  • Changes to the API follow Semantic Versioning and are documented in the API Changes document.
  • This PR modifies some build requirements and the readme is updated
  • This PR contains no files larger than 50 MB

Release Notes

  • added white border to preference view to increase visibility

Reviewers

@Amoursol
@reddyashish
@QilongTang

FYIs

- added white border to preference view to increase visibility
@QilongTang
Copy link
Contributor

The reported regression is sporadic failure
DynamoCoreWpfTests.CoreUserInterfaceTests.WorkspaceContextMenu_TestIfOpenOnRightClick

<Grid x:Name="mainGrid"
Background="{StaticResource PreferencesWindowBackgroundColor}">
<Border x:Name="mainBorder"
BorderBrush="{StaticResource WorkspaceBackgroundHomeBrush}"
Copy link
Contributor

Choose a reason for hiding this comment

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

I guess you happen to found out this color from other places is white?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I got this one from the NodeView border

BorderBrush="{StaticResource WorkspaceBackgroundHomeBrush}"
as instructed by @Amoursol. The only thing is the borderthickness, which for the Nodes is 1px, and for the PreferencesView is now set at 2px. I will double-check with @Amoursol to see if there is preference for each maybe.

Copy link
Contributor

Choose a reason for hiding this comment

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

Thanks, will merge it for now

@QilongTang QilongTang added this to the 2.19.0 milestone Jun 15, 2023
@QilongTang QilongTang merged commit 3daf6ed into DynamoDS:master Jun 15, 2023
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.

2 participants