Skip to content

feat(overlay): Add new components and adjust current ones#5516

Closed
Princesseuh wants to merge 3 commits intomainfrom
feat/dev-overlay-main-screen-redesign
Closed

feat(overlay): Add new components and adjust current ones#5516
Princesseuh wants to merge 3 commits intomainfrom
feat/dev-overlay-main-screen-redesign

Conversation

@Princesseuh
Copy link
Copy Markdown
Member

@Princesseuh Princesseuh commented Nov 28, 2023

Description (required)

This change is for next, so don't merge until 4.0!

Docs for withastro/astro#9118

New components were added and some of the current ones were slightly reworked to be more generic.

@vercel
Copy link
Copy Markdown

vercel bot commented Nov 28, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
docs ✅ Ready (Inspect) Visit Preview Dec 2, 2023 6:19pm

@Princesseuh Princesseuh added merge-on-release Don't merge this before the feature is released! (MQ=approved but WAIT for feature release!) 4.0 labels Nov 28, 2023
Copy link
Copy Markdown
Member

@sarah11918 sarah11918 left a comment

Choose a reason for hiding this comment

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

Amazing work, @Princesseuh ! I made some suggestions below for you to consider!

@sarah11918
Copy link
Copy Markdown
Member

Hi @Princesseuh Just wanted to check whether you were OK with these changes! I know you have limited internet right now, so I'm going to go ahead and commit these so you don't have to do it. Then you can comment with any changes needed! 🙌

@sarah11918
Copy link
Copy Markdown
Member

Comparing this to Matthew's newer PR that (re)names everything!

Copy link
Copy Markdown
Member

@sarah11918 sarah11918 left a comment

Choose a reason for hiding this comment

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

Blocking this one, as #5541 combines both these new changes AND the renaming together. If I did this correctly, we do not need this PR anymore. This is just for reference to make sure that #5541 is complete.

@sarah11918
Copy link
Copy Markdown
Member

Closing as these changes were brought in and merged with #5541

@sarah11918 sarah11918 closed this Dec 4, 2023
@sarah11918 sarah11918 deleted the feat/dev-overlay-main-screen-redesign branch August 29, 2024 15:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

4.0 merge-on-release Don't merge this before the feature is released! (MQ=approved but WAIT for feature release!)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants