Skip to content

fix(samples): show grid sample in overlay container correctly#3934

Merged
ChronosSF merged 1 commit intovnextfrom
mvenkov/fix-grid-component-dialogs-vnext
Apr 21, 2026
Merged

fix(samples): show grid sample in overlay container correctly#3934
ChronosSF merged 1 commit intovnextfrom
mvenkov/fix-grid-component-dialogs-vnext

Conversation

@ChronosSF
Copy link
Copy Markdown
Member

No description provided.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adjusts the Boston Marathon grid sample overlay configuration so the winner/finish overlays render in the intended container context.

Changes:

  • Updates absolute overlay settings initialization for the grid alerts.
  • Adds styling for a .grid__wrapper__inner container to act as a positioned overlay host.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
projects/app-lob/src/app/grid/grid-boston-marathon/grid.component.ts Changes how OverlaySettings are created for the alert overlays.
projects/app-lob/src/app/grid/grid-boston-marathon/grid.component.scss Introduces .grid__wrapper__inner positioning/sizing to support correct overlay placement.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@ChronosSF ChronosSF merged commit 9b16a57 into vnext Apr 21, 2026
9 checks passed
@ChronosSF ChronosSF deleted the mvenkov/fix-grid-component-dialogs-vnext branch April 21, 2026 11:17
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