-
-
Notifications
You must be signed in to change notification settings - Fork 3
feat: work on styling #20
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
Conversation
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.
Thank you so much for this PR. I really like & agree with the styling. Added my comments, please check and let me know if I should address them :)
apps/site/tsconfig.json
Outdated
@@ -3,7 +3,10 @@ | |||
"references": [], | |||
"compilerOptions": { | |||
"outDir": "../../.moon/cache/types/apps/site", | |||
"paths": {}, | |||
"baseUrl": ".", |
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.
Is there any other reason for this besides that the imports are based on the root instead of relative to the file?
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.
personal preferences, i like using aliases
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.
I can live with the aliases, I don't really like that non aliased imports are based on the root. AFAIK, path aliases work also without baseUrl
in newer typescript version. related PR
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.
I tried to remove baseUrl
but it didn't work, does it require updates to paths
as well?
Co-authored-by: Alexander Niebuhr <[email protected]>
* updates moon * rewrites pages to use layout * adds preview deployment automation * improves preview workflows * adds :sync task * disables :check task (temporarily) * updates deps * adds GitHub deployments * adapts styles for starlight * fixes formatting * fixes pipeline * removes broken dark variant (temporarily) * refactors workflow scripts to esm * fixes lockfile * improves setup * tries to solve conflicts * tries to solve conflicts * cleans package.json * fixes install * adds new design (#20) * improves font loading performance (#22) Co-authored-by: Florian Lefebvre <[email protected]> * improvements (#23) * adds AstroStudio CMS announcement (#21) Co-authored-by: Adam Matthiesen <[email protected]> Co-authored-by: Paul Valladares <[email protected]> Co-authored-by: Jacob Jenkins <[email protected]> * fixes blog frontmatter * adds announcement for `astro-theme-provider` (#19) * Update apps/site/src/content/blog/astro-theme-provider.mdx * Update apps/site/src/content/blog/astro-theme-provider.mdx * Update apps/site/src/content/blog/astro-theme-provider.mdx * Update apps/site/src/content/blog/astro-theme-provider.mdx * Update apps/site/src/content/blog/astro-theme-provider.mdx * Update apps/site/src/content/blog/astro-theme-provider.mdx * Update apps/site/src/content/blog/astro-theme-provider.mdx * Update apps/site/src/content/blog/astrostudio-cms.mdx * Update apps/site/src/content/blog/astrostudio-cms.mdx * support multiple authors * fix build * update projects overview --------- Co-authored-by: Florian Lefebvre <[email protected]> Co-authored-by: Paul Valladares <[email protected]> Co-authored-by: Adam Matthiesen <[email protected]> Co-authored-by: Jacob Jenkins <[email protected]> Co-authored-by: Bryce Russell <[email protected]>
No description provided.