Skip to content

DYN-8136 - Extensions Tab Icon #16771

Merged
zeusongit merged 1 commit into
masterfrom
DYN-8136
Dec 12, 2025
Merged

DYN-8136 - Extensions Tab Icon #16771
zeusongit merged 1 commit into
masterfrom
DYN-8136

Conversation

@CustomBuildingConfigurators-Balaji
Copy link
Copy Markdown
Contributor

@CustomBuildingConfigurators-Balaji CustomBuildingConfigurators-Balaji commented Dec 10, 2025

Purpose

Ensure the Extensions tab on the menu bar has a downward arrow icon so all tabs have uniform icons. Use the same icons as Help tab. (DYN-8136)

Declarations

Check these if you believe they are true

  • The code base 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.
image

Release Notes

Extensions menu will appear same as other menus on the top of the home screen.

Reviewers

@reddyashish, @achintyabhat

FYIs

@stevecbc

Copy link
Copy Markdown

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

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

See the ticket for this pull request: https://jira.autodesk.com/browse/DYN-8136

@avidit
Copy link
Copy Markdown
Contributor

avidit commented Dec 10, 2025

@achintyabhat I agree with @hwahlstrom's suggestion of removing the dropdown arrows as well. What do you think?

@avidit
Copy link
Copy Markdown
Contributor

avidit commented Dec 10, 2025

@CustomBuildingConfigurators-Balaji Thank you for the PR. We usually fork the repo and create PR from there.

@CustomBuildingConfigurators-Balaji
Copy link
Copy Markdown
Contributor Author

@CustomBuildingConfigurators-Balaji Thank you for the PR. We usually fork the repo and create PR from there.

Let me know if you want me to delete this branch, fork from master and raise PR again.

@avidit
Copy link
Copy Markdown
Contributor

avidit commented Dec 10, 2025

@CustomBuildingConfigurators-Balaji Thank you for the PR. We usually fork the repo and create PR from there.

Let me know if you want me to delete this branch, fork from master and raise PR again.

No need to delete this one.

@zeusongit zeusongit merged commit c30a090 into master Dec 12, 2025
34 of 36 checks passed
@zeusongit zeusongit deleted the DYN-8136 branch December 12, 2025 19:04
@zeusongit zeusongit modified the milestone: 4.0.1 Dec 15, 2025
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.

3 participants