Skip to content

fix(docs): replace hero PNGs with WebP, add trailingSlash, add perf t…#733

Merged
castrojo merged 4 commits into
projectbluefin:mainfrom
castrojo:feature/docs-perf-phase1
Apr 9, 2026
Merged

fix(docs): replace hero PNGs with WebP, add trailingSlash, add perf t…#733
castrojo merged 4 commits into
projectbluefin:mainfrom
castrojo:feature/docs-perf-phase1

Conversation

@castrojo
Copy link
Copy Markdown
Contributor

@castrojo castrojo commented Apr 9, 2026

…ests

  • bluefin-hero: 3.6 MB PNG -> 188 KB WebP (1920x1080) with fetchpriority=high
  • bluefin-raptors: 1.2 MB PNG -> 64 KB WebP (1120x630) with loading=lazy
  • trailingSlash: true eliminates redirect chain on all doc pages
  • Playwright regression tests for homepage LCP and image optimisation

…ests

- bluefin-hero: 3.6 MB PNG -> 188 KB WebP (1920x1080) with fetchpriority=high
- bluefin-raptors: 1.2 MB PNG -> 64 KB WebP (1120x630) with loading=lazy
- trailingSlash: true eliminates redirect chain on all doc pages
- Playwright regression tests for homepage LCP and image optimisation

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request optimizes image loading on the homepage and artwork pages by replacing markdown image syntax with HTML tags that include explicit dimensions, fetch priority, and loading strategies. It also enables the trailingSlash configuration in Docusaurus and introduces a new E2E test suite to monitor performance, accessibility, and layout stability. The review feedback recommends marking known-failing tests as fixme to avoid blocking the CI pipeline and suggests replacing non-deterministic timeouts with more reliable element-based waits.

Comment thread tests/e2e/performance.spec.ts Outdated
Comment thread tests/e2e/performance.spec.ts Outdated
Comment thread tests/e2e/performance.spec.ts Outdated
castrojo and others added 3 commits April 8, 2026 22:44
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
@castrojo castrojo merged commit 357ca4a into projectbluefin:main Apr 9, 2026
1 check passed
@castrojo castrojo deleted the feature/docs-perf-phase1 branch May 9, 2026 19:31
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.

1 participant