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

Enhancement: Docs upgrade- kit-docs and sveltejs/kit #1665

Merged
merged 36 commits into from
May 11, 2023
Merged

Conversation

leightkt
Copy link
Contributor

@leightkt leightkt commented Apr 28, 2023

Description

Upgrades:
@svelteness/kit-docs: 1.1.3
@sveltejs/kit: 1.5.0
Fixes font loading
Routes restructured to work with new kit-docs structure: https://kitdocs.vercel.app/docs/getting-started/quickstart

PLEASE NOTE- Checklist must be complete prior to review.

Checklist

  • [NA] Increment the version field in package.json of the package you have made changes in following semantic versioning and using alpha release tagging
  • Check the box that allows repo maintainers to update this PR
  • Test locally to make sure this feature/fix works
  • Run yarn check-all to confirm there are not any associated errors
  • Confirm this PR passes Circle CI checks
  • Add or update relevant information in the documentation

@leightkt leightkt self-assigned this Apr 28, 2023
@vercel
Copy link

vercel bot commented Apr 28, 2023

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

Name Status Preview Comments Updated (UTC)
web3-onboard-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 11, 2023 5:25pm

@leightkt leightkt changed the title Kitdocs upgrade Enhancement: Docs upgrade- kit-docs and sveltejs/kit Apr 28, 2023
Copy link
Member

@Adamj1232 Adamj1232 left a comment

Choose a reason for hiding this comment

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

Nice!!
Looks like tabs may have been changed to 6 spaces within docs? Could make for a funky layout within IDEs

@leightkt
Copy link
Contributor Author

leightkt commented May 1, 2023

@Adamj1232 realized I forgot to match the .prettierrc to the original version from the docs- updated and reformatted. Hopefully looks less wonky on the IDE now! LMK what you think!

docs/package.json Outdated Show resolved Hide resolved
@@ -1,4 +1,3 @@
export * from './examples'
export * from './gas'
export { default as ThemeCustomizer } from './ThemeCustomizer.svelte'
Copy link
Member

Choose a reason for hiding this comment

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

Just curious why this is removed?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It was moved directly to the +page.svelte file under the theming-tool folder- It would not load/build correctly otherwise.

@@ -1 +0,0 @@
export { Tabs, TabPanel, TabItem } from './tabs'
Copy link
Member

Choose a reason for hiding this comment

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

Do we still need this file or parent folder?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yes to the parent folder (to be able to use Tabs in the .md files that are the docs pages), no to the index.ts file.

@leightkt leightkt merged commit 71e6816 into develop May 11, 2023
@leightkt leightkt deleted the kitdocs-upgrade branch May 11, 2023 19:26
@Adamj1232 Adamj1232 mentioned this pull request May 18, 2023
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