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

docs tweaks #70

Merged
merged 5 commits into from
Aug 28, 2024
Merged

docs tweaks #70

merged 5 commits into from
Aug 28, 2024

Conversation

steven-klein
Copy link
Collaborator

@steven-klein steven-klein commented Aug 24, 2024

What this does

This is a WIP ugly update to how docs are written for Trees. A number of pain points have made writing docs difficult, this starts to alleviate some of them. Specifically, adding code examples has been frustrating with how Vue SFC's compile and how prettier parses those files. This starts to introduce markdown files that for now are mostly used for displaying code samples.

  • refactored page declarations, home page layout, and app entry point logic
  • makes numerous file name and file organization changes (WIP - no concrete direction here, but it doesn't fit with app patterns)
  • migrates old code samples into markdown files
  • fixes lack of type checking on /docs directory via tsconfig updates and global component type declarations

@steven-klein steven-klein marked this pull request as ready for review August 24, 2024 01:55
Copy link
Collaborator

@DavidLarsKetch DavidLarsKetch left a comment

Choose a reason for hiding this comment

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

Looks great, only styling tweak I'd look into is the font size for code samples. Given the narrow layout for a doc, I in Tools you end up with a lot of overflow to scroll through. There's some with the original samples, but a lot less has been chopped off such that you can kind of guess what the end of a line is.

Copy link
Contributor

@profsmallpine profsmallpine left a comment

Choose a reason for hiding this comment

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

Defaulting to more horizontal space is the only change request here. Accomplished by removing max-w-3xl and defaulting to the top nav.

This is a great start, look forward to seeing how this develops. Would love to have this be part of my workflow whenever I start a new page and layout for a feature.

window.localStorage.setItem(key, layout)
}

return window.localStorage.getItem(key) || "SidebarLayout"
Copy link
Contributor

Choose a reason for hiding this comment

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

I would use StackedLayout as the default here to maximize horizontal space.

dev/docs/StackingContextDocs.md Show resolved Hide resolved
* these options are used only in the instantiation
* of a new UseBaseAPI composable
*/
export interface UseBaseAPIOptions extends ReqOptions {
Copy link
Contributor

Choose a reason for hiding this comment

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

I wonder if 1. we could write our implementation such that it's importable here rather than redefining and 2. if it's even possible to import code here from elsewhere in the project. Just food for thought, nothing to do now

@@ -1,8 +1,8 @@
<script setup lang="ts">
import { ref } from "vue"
import ColorRow from "../../dev/helpers/ColorRow.vue"
import ColorRow from "../components/ColorRow.vue"
Copy link
Contributor

Choose a reason for hiding this comment

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

It would be nice to remove max-w-3xl from all of our page containers to get more horizontal space.

@steven-klein
Copy link
Collaborator Author

@DavidLarsKetch @profsmallpine I'll add the requested changes on once I merge it all together. The doc's width change will make a mess of the diffs.

@steven-klein steven-klein merged commit d96215f into spk/v0.10.0 Aug 28, 2024
1 check passed
@steven-klein steven-klein deleted the spk/docs-tweaks branch August 30, 2024 19:02
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