Skip to content

fix(ui): remove stray sidebar character, fix RSS stylesheet and Feedly URLs, optimize TBT - #83

Merged
kascit merged 1 commit into
mainfrom
fix/ui-sidebar-rss-tbt-optimizations
Aug 23, 2026
Merged

fix(ui): remove stray sidebar character, fix RSS stylesheet and Feedly URLs, optimize TBT#83
kascit merged 1 commit into
mainfrom
fix/ui-sidebar-rss-tbt-optimizations

Conversation

@kascit

@kascit kascit commented Aug 23, 2026

Copy link
Copy Markdown
Owner

Summary of Fixes

  1. Sidebar Stray Character:

    • Removed stray Chinese character from templates/macros/sidebar.html (line 72).
    • Audited all templates, content, and scripts to confirm zero remaining stray characters.
  2. RSS XML Stylesheet:

    • Changed static/rss.xsl XSLT version declaration from version="3.0" to version="1.0". Browser XSLT engines (e.g. libxslt in Chrome, Firefox, Safari) only support XSLT 1.0; declaring 3.0 caused stylesheet parsing errors and fallback to raw unstyled XML.
  3. Feedly Subscription URLs:

    • Fixed double-encoded feed%2F in Feedly subscription links across archive.html, taxonomy_list.html, and taxonomy_single.html to standard feed/.
  4. TBT & Long Task Performance Optimization:

    • Streamlined main.js initialization into three progressive lifecycle stages:
      • Phase 1 (Immediate): Core UI shell sync and navbar dropdowns.
      • Phase 2 (After First Paint): High-priority visual enhancements (tooltips, code blocks, visible page enhancements).
      • Phase 3 (Idle): Deferred non-critical background services and listeners (telemetry, search loader, shortcuts, webmcp, comments, etc.).
    • Eliminates main-thread blocking time during critical paint.

Copilot AI lite review requested due to automatic review settings August 23, 2026 11:01
@kascit
kascit enabled auto-merge (squash) August 23, 2026 11:01

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@kascit
kascit merged commit bff6157 into main Aug 23, 2026
14 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