Skip to content

fix: missing page titles; add checks#87

Open
halfcyan wants to merge 4 commits into
mainfrom
cypress/missing-page-titles
Open

fix: missing page titles; add checks#87
halfcyan wants to merge 4 commits into
mainfrom
cypress/missing-page-titles

Conversation

@halfcyan

Copy link
Copy Markdown
Member

No description provided.

@halfcyan

Copy link
Copy Markdown
Member Author

the checks on this make sure that new routes have a page title. you can opt out on a specific +page.svelte page, but it's manual, so you have to think about it.

i also added a new PageTitle component that adds / Stack to the end of tab titles so that you don't have to put it on all of your <svelte:head> elements.

@halfcyan

Copy link
Copy Markdown
Member Author

i forgot to push a commit lol

@halfcyan

Copy link
Copy Markdown
Member Author

resolves #69 btw

@halfcyan
halfcyan enabled auto-merge July 25, 2026 00:08
@addidotlol

addidotlol commented Jul 25, 2026

Copy link
Copy Markdown
Member

a lot of these page titles should be more clear about being sub-pages of colocation or admin, since theres a lot of duplicate or easily confused titles.

im also not sure using a script to force titles on pages is the ideal route here, it feels overkill & theres ways to automate a lot of these titles.

edit: this also should be a lint rule and not a script

@halfcyan

Copy link
Copy Markdown
Member Author

i need to go back through all of this tomorrow

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.

2 participants