Skip to content

Conversation

@RobertGlobant20
Copy link
Contributor

Purpose

During the Quality Testing some issues were found in the Preferences window behavior, so the next fixes were done:

  • When a specific tab in the left tab is selected the blue color was updated to use a lighter one (#ADE4DE).
  • The functionality of highlighting in blue the Tab when is selected was removed.
  • When a tab is selected the Font weight is Bold otherwise will be Regular.

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

Reviewers

@QilongTang

FYIs

@Astul-Betizagasti

I added a new workflow (issue_type_predicte.yaml) that is using the ML.NET model for predicting the issue type (source repo), so If a new issue is created in the Dynamo repo this workflow will run and will predict if is a Wishlist issue or not. If is a Wishlist issue it will be labeled as "Wishlist" and then another workflow will move the issue to the DynamoWishlist repo. If the issue is incomplete or is not valid the label "NotMLEvaluated" will be added to the issue.

Also I added two scripts more, one will return the issue body in a json string and the other one will clan the issue body removing sections not used like "Dynamo Version" or "Stack Trace"
When testing the issue predicter workflow the issues labeled as Wishlist were not moved to the DynamoWishlist repo due that the "Move Issue by labels" workflow failed. There was a problem with the PAT used to label the issue, I was using the wrong one (no triggers actions).
During the Quality Testing some issues were found in the General tab, so the next fixes were done:

- When a specific tab in the left tab is selected the blue color was updated to use a lighter one (#ADE4DE).
- The functionality of highlighting in blue the Tab when is selected was removed.
- When a tab is selected the Font weight is Bold otherwise will be Regular.
@RobertGlobant20 RobertGlobant20 changed the title Dyn 3518 preferences general tab3 Dyn 3518 preferences window - color fixes Mar 26, 2021
@RobertGlobant20
Copy link
Contributor Author

This is a gif of how the screen tab colors looks:
general_tab_tab_colors

@QilongTang QilongTang merged commit 624c84a into DynamoDS:master Mar 29, 2021
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