Skip to content

Conversation

skeptrunedev
Copy link
Contributor

Documentation changes

You have to do some extra stuff to prevent wonky scroll behavior with route53. This PR documents that.

Closes


For Reviewers

When reviewing documentation PRs, please consider:

✅ Technical accuracy

  • Code examples work as written
  • Commands and configurations are correct
  • Links resolve to the right destinations
  • Prerequisites and requirements are accurate

✅ Clarity and completeness

  • Instructions are clear and easy to follow
  • Steps are in logical order
  • Nothing important is missing
  • Examples help illustrate the concepts

✅ User experience

  • A new user could follow these docs successfully
  • Common gotchas or edge cases are addressed
  • Error messages or troubleshooting guidance is helpful

Copy link
Collaborator

@ethanpalm ethanpalm left a comment

Choose a reason for hiding this comment

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

A few questions so we can clean this up and merge

Your documentation is now live at `/docs` for your primary domain.

<Propagating />
<Propagating />
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
<Propagating />

Can this be deleted? Not sure what it is and I think it's an artifact from the writer bot.

Comment on lines +161 to +163
### Additional behaviors for layout stability

To prevent layout shifts and ensure smooth navigation, add these additional behaviors for static asset handling:
Copy link
Collaborator

Choose a reason for hiding this comment

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

Are these steps only applicable to some users or should everyone set these behaviors?

If everyone should do it, I'd remove this section and raise all the H4s to H3s so that it doesn't seem like these are extra optional settings.

@cdxker cdxker self-requested a review September 11, 2025 17:31
@cdxker
Copy link
Contributor

cdxker commented Sep 15, 2025

#1110

These changes were merged in this pull request.

@cdxker cdxker closed this Sep 15, 2025
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