-
-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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(examples): New basics template #12472
Conversation
|
@@ -4,5 +4,3 @@ | |||
export { default as Code } from './Code.astro'; | |||
// @ts-ignore | |||
export { default as Debug } from './Debug.astro'; | |||
// @ts-ignore | |||
export { default as Welcome } from './Welcome.astro'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It moved it back into the template for multiple reasons:
- You can't import from
astro:components
a component that has styles defined the "Astro way", you cause styles to leak everywhere - Inside Astro, you could check out the code, but not modify it, which feels like pedagogically it was a downgrade
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
FWIW (and perhaps you already know) for no1, I think is:inline
can be used to avoid that, unless I miss something that requires processing the styles.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ugh. Forgot to hit send on a couple of comments. May well be out of date now…
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Spotted a few issues, but it’s looking great @Princesseuh!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested on Safari and looks great!
Beautiful! :) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks really pretty! Tested by opening this branch with StackBlitz in various browsers and testing it out.
Looked as expected in Safari and Firefox on macOS and in Firefox Android. Even on FF Android I didn’t see the issues I usually associate with heavy blurs (probably helped by the fact stuff doesn’t really have to scroll in this template).
On smaller screens (like my phone), the “What’s new in Astro 5.0” card can cover the CTA buttons, but I don’t know how much that matters (screenshot below). The buttons themselves look great on mobile now! 🙌
Changes
This updates the design of the basics template for Astro 5. It aims to provide a mix between the old basics and minimal templates, so that it's easy for someone to take the basics, remove a few things easily and get minimal.
Looks like this
![image](https://private-user-images.githubusercontent.com/3019731/388123149-b3757c59-4adc-40da-89d0-dba8077d28a5.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3Mzk3OTgyMDcsIm5iZiI6MTczOTc5NzkwNywicGF0aCI6Ii8zMDE5NzMxLzM4ODEyMzE0OS1iMzc1N2M1OS00YWRjLTQwZGEtODlkMC1kYmE4MDc3ZDI4YTUucG5nP1gtQW16LUFsZ29yaXRobT1BV1M0LUhNQUMtU0hBMjU2JlgtQW16LUNyZWRlbnRpYWw9QUtJQVZDT0RZTFNBNTNQUUs0WkElMkYyMDI1MDIxNyUyRnVzLWVhc3QtMSUyRnMzJTJGYXdzNF9yZXF1ZXN0JlgtQW16LURhdGU9MjAyNTAyMTdUMTMxMTQ3WiZYLUFtei1FeHBpcmVzPTMwMCZYLUFtei1TaWduYXR1cmU9YTRiMDY5NGExNTQ4ZTIyMjM4ODg1MGEzMzNhNWNkMmY5YjA2MDc0MTMyNDRhNTJkNjQ2YzJiYTc1YjlkYmVhZiZYLUFtei1TaWduZWRIZWFkZXJzPWhvc3QifQ.qQ6Q_gC3vDz8yRghLC4C4qQOCHx4meWOn9cz2Sa1jRk)
Testing
Tested manually
Docs
N/A. Docs doesn't seem to describe this template as far as I can tell.