Skip to content

EXLM-5499 fix: double line in header - #2856

Open
ruhisingh1 wants to merge 1 commit into
mainfrom
revert-2855-revert-2851-exlm-5499-fix
Open

EXLM-5499 fix: double line in header#2856
ruhisingh1 wants to merge 1 commit into
mainfrom
revert-2855-revert-2851-exlm-5499-fix

Conversation

@ruhisingh1

Copy link
Copy Markdown
Collaborator

Reverts #2855

@aem-code-sync

aem-code-sync Bot commented Aug 11, 2026

Copy link
Copy Markdown

Hello, I'm the AEM Code Sync Bot and I will run some actions to deploy your branch and validate page speed.
In case there are problems, just click a checkbox below to rerun the respective action.

  • Re-run all PSI checks
  • Re-run failed PSI checks
  • Re-sync branch
Commits

@claude

claude Bot commented Aug 11, 2026

Copy link
Copy Markdown
Contributor

Code Review

Reviewed this revert-of-revert (restores the single-line max-width: none; fix from #2851, EXLM-5499).

I traced a potential concern: the edited selector (.header .nav-wrapper > ul > li .nav-items-secondary > li a, specificity 0,3,4) also matches the per-tab "Browse all" link inside a @tab dropdown's right panel, and outranks .header .nav-item-content-level-1 > ul li > a (0,2,3, max-width: 220px) that applies to the same element there. On closer inspection this isn't a regression: that per-tab secondary link already renders outside the 2-column grid (its own row, with distinct top-border/margin styling at line ~712), so dropping its width cap is consistent with — not in conflict with — the single-line fix this PR restores.

No other issues found in this 1-line diff. LGTM.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant