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

Toms blink new docs #643

Merged
merged 8 commits into from
Dec 12, 2023
Merged

Toms blink new docs #643

merged 8 commits into from
Dec 12, 2023

Conversation

mikelgg93
Copy link
Member

@mikelgg93 mikelgg93 commented Nov 28, 2023

Based on #634 and update to accommodate the latest docs changes. There are still some things to be done:

  • Center images, are we still using v-img? I do not think so, but @nathakits or @marc-tonsen can confirm it?
  • Images with only text look bad on dark mode (new default of docs theme)
  • Sidebar title is perhaps too long (any suggestion @N-M-T or @tpfffr ?)
  • Missing description for landing page

Fix links
Make PR-Check manually available too
fix yt links checking by remapping URI
Fix recordings overview link
Argument is --header not --headers, they have it wrong in the docs
Accept more headers to prevent YT issues
ignore webp and woff2 files
cspell v5, artifact-lyche logic
Fix spacing
Build all the instances and check links later, this allows cross instances link checking
GH Update WIP
Fix links
fix naming
@willpatera willpatera merged commit 6537e79 into master Dec 12, 2023
7 of 8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants