Skip to content
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

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

Closed
wants to merge 3 commits into from

Conversation

Princesseuh
Copy link
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.

Copy link

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
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
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
Member

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

Copy link
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
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