Skip to content

refactor(devtools): make the component metadata menu more prominent #62685

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

hawkgs
Copy link
Member

@hawkgs hawkgs commented Jul 17, 2025

PR Checklist

Please check if your PR fulfills the following requirements:

PR Type

What kind of change does this PR introduce?

  • Feature

What is the new behavior?

The menu content requires additional work but this should at least make it more prominent to the end user.

  • Add an arrow icon to indicate that this is an expansion panel.
  • Reduce the active area, which toggles the menu, to the text only. This will prevent the menu from expanding if you click the header which handles the cases when you misclick the "Show Signal Graph" button.
CollapsedExpanded
Screenshot 2025-07-17 at 15 38 50 Screenshot 2025-07-18 at 10 03 05

@hawkgs hawkgs requested a review from JeanMeche July 17, 2025 12:55
@ngbot ngbot bot added this to the Backlog milestone Jul 17, 2025
@hawkgs hawkgs marked this pull request as draft July 17, 2025 14:09
Add an arrow icon to indicate that this is an expansion panel. Reduce the active area (which toggles the menu) to the text only.
@hawkgs hawkgs force-pushed the devtools/cmp-metadata-menu branch 2 times, most recently from 74b1d14 to 61c89a4 Compare July 18, 2025 07:04
@hawkgs
Copy link
Member Author

hawkgs commented Jul 18, 2025

@JeanMeche, I added the reference icons and decided to do some changes to the menu. Additionally, I've added a new commit which creates the docs-ref-button component that represents this same icon since there are several instances of such doc references in the app.

Used for external Angular docs links.
@hawkgs hawkgs force-pushed the devtools/cmp-metadata-menu branch from 61c89a4 to b4b5bb4 Compare July 18, 2025 07:10
@hawkgs hawkgs marked this pull request as ready for review July 18, 2025 07:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants