Skip to content

improvements#9

Merged
AlemTuzlak merged 41 commits intomainfrom
improvements
Sep 19, 2025
Merged

improvements#9
AlemTuzlak merged 41 commits intomainfrom
improvements

Conversation

@abrulic
Copy link
Collaborator

@abrulic abrulic commented Sep 7, 2025

  • added Github Icon Link in header
  • improved lighthouse
  • fixed problem in browser console due useCurrentVersion and useParams usage
  • small refactoring with mobile sidebar
  • added meta for routes
  • typesafe loading pages and sections
  • allPages and allSections are loaded once on app starting
  • fly deploy workflow

@github-actions
Copy link

github-actions bot commented Sep 7, 2025

Coverage Report

Status Category Percentage Covered / Total
🔵 Lines 88.73% 323 / 364
🔵 Statements 88.73% 323 / 364
🔵 Functions 83.33% 30 / 36
🔵 Branches 97.53% 79 / 81
File Coverage
File Stmts Branches Functions Lines Uncovered Lines
Changed Files
app/components/sidebar/build-breadcrumbs.ts 100% 100% 100% 100%
app/utils/path-builders.ts 37.14% 100% 20% 37.14% 5-15, 28-30, 33-35, 38-45
app/utils/split-slug.ts 71.42% 75% 100% 71.42% 13-16
Generated in workflow #27 for commit d2c3788 by the Vitest Coverage Report Action

@abrulic abrulic changed the title small improvements improvements Sep 7, 2025
@abrulic abrulic marked this pull request as ready for review September 10, 2025 15:47
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR improves the documentation site with GitHub integration, better mobile UX, SEO enhancements, and structural refactoring. The changes focus on making the site more accessible and developer-friendly.

  • Added GitHub icon link to header navigation and improved mobile responsiveness
  • Implemented comprehensive SEO meta generation for all documentation routes
  • Fixed React hooks usage issues and improved type safety throughout the codebase

Reviewed Changes

Copilot reviewed 46 out of 71 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
app/routes/documentation-page.tsx Removed - replaced with split sectioned/standalone route handlers
app/routes/documentation-page-sectioned.tsx New route handler for sectioned documentation pages with meta generation
app/routes/documentation-page-standalone.tsx New route handler for standalone documentation pages with meta generation
app/utils/version-resolvers.ts Fixed hooks usage by replacing useParams with useRouteLoaderData
app/components/sidebar/ Refactored sidebar to use new SidebarTree structure and improved mobile breadcrumbs
app/utils/seo.ts New SEO utility for generating consistent meta tags
content/ Added new documentation content files

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@abrulic abrulic marked this pull request as draft September 10, 2025 16:10
@abrulic abrulic marked this pull request as ready for review September 11, 2025 09:23
@thomasfr
Copy link

would it be possible to get a deployed version of this somewhere to check it out?

@abrulic abrulic marked this pull request as draft September 16, 2025 07:33
@abrulic abrulic marked this pull request as ready for review September 19, 2025 10:55
@abrulic abrulic requested a review from thomasfr September 19, 2025 10:58
@AlemTuzlak AlemTuzlak merged commit ba88dba into main Sep 19, 2025
5 checks passed
@abrulic abrulic deleted the improvements branch November 18, 2025 11:13
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.

4 participants