Skip to content

docs: show selected menu item visually in context menu example#69453

Open
aparzi wants to merge 1 commit into
angular:mainfrom
aparzi:improvement-element-selected
Open

docs: show selected menu item visually in context menu example#69453
aparzi wants to merge 1 commit into
angular:mainfrom
aparzi:improvement-element-selected

Conversation

@aparzi

@aparzi aparzi commented Jun 19, 2026

Copy link
Copy Markdown
Contributor

Replace console.log with a lastAction signal and render the result in a styled

element with a fade-in animation.

PR Checklist

Please check if your PR fulfills the following requirements:

PR Type

What kind of change does this PR introduce?

  • Bugfix
  • Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • CI related changes
  • Documentation content changes
  • angular.dev application / infrastructure changes
  • Other... Please describe:

What is the current behavior?

The selected item is printed to the console, providing no clear documentation to the user.

What is the new behavior?

Screenshot 2026-06-20 alle 00 41 45

Does this PR introduce a breaking change?

  • Yes
  • No

Replace console.log with a lastAction signal and render the result in a styled <p> element with a fade-in animation.
@pullapprove pullapprove Bot requested a review from kirjs June 19, 2026 22:44
@angular-robot angular-robot Bot added the area: docs Related to the documentation label Jun 19, 2026
@ngbot ngbot Bot added this to the Backlog milestone Jun 19, 2026
@github-actions

Copy link
Copy Markdown

Deployed adev-preview for 05187da to: https://ng-dev-previews-fw--pr-angular-angular-69453-adev-prev-hm1gi2h1.web.app

Note: As new commits are pushed to this pull request, this link is updated after the preview is rebuilt.

@JeanMeche JeanMeche added action: merge The PR is ready for merge by the caretaker target: patch This PR is targeted for the next patch release labels Jun 19, 2026
@JeanMeche JeanMeche removed the request for review from kirjs June 19, 2026 23:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

action: merge The PR is ready for merge by the caretaker adev: preview area: docs Related to the documentation target: patch This PR is targeted for the next patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants