Improve visibility of bold Markdown in game descriptions #927

Closed
opened 2026-08-10 06:46:22 -05:00 by Veilor · 2 comments
Member

Bold Markdown in game descriptions currently has insufficient visual emphasis according to the reported behavior.

The change belongs at the game-description rendering and presentation layer, with regression coverage for the rendered output.

Scope

  • Game description Markdown rendering in the relevant Blade view under resources/views/
  • Typography styles affecting rendered game-description emphasis under resources/css/
  • Focused feature coverage under tests/Feature/ for game-description rendering

Acceptance Criteria

  • Bold Markdown in a game description is visibly distinguishable from surrounding regular text.
  • Regular game-description text retains its existing appearance and readability.
  • Other supported game-description Markdown formatting continues to render correctly.
  • A focused automated test verifies the bold game-description behavior.

Test Coverage Required

  • Add or update a focused feature test for rendering bold Markdown in a game description.
  • Run the existing relevant game-description feature tests and the full test suite according to repository conventions.

Progress Checklist

  • Locate the game-description Blade rendering path and its associated typography styles.
  • Increase the visual emphasis of rendered bold text within game descriptions using the existing styling approach.
  • Add focused regression coverage for bold Markdown rendering.
  • Review adjacent Markdown formatting behavior for regressions.
Bold Markdown in game descriptions currently has insufficient visual emphasis according to the reported behavior. The change belongs at the game-description rendering and presentation layer, with regression coverage for the rendered output. ## Scope - Game description Markdown rendering in the relevant Blade view under resources/views/ - Typography styles affecting rendered game-description emphasis under resources/css/ - Focused feature coverage under tests/Feature/ for game-description rendering ## Acceptance Criteria - Bold Markdown in a game description is visibly distinguishable from surrounding regular text. - Regular game-description text retains its existing appearance and readability. - Other supported game-description Markdown formatting continues to render correctly. - A focused automated test verifies the bold game-description behavior. ## Test Coverage Required - Add or update a focused feature test for rendering bold Markdown in a game description. - Run the existing relevant game-description feature tests and the full test suite according to repository conventions. ## Progress Checklist - [x] Locate the game-description Blade rendering path and its associated typography styles. - [x] Increase the visual emphasis of rendered bold text within game descriptions using the existing styling approach. - [x] Add focused regression coverage for bold Markdown rendering. - [x] Review adjacent Markdown formatting behavior for regressions.
Codex changed title from Making text bold should be more visible to Improve visibility of bold Markdown in game descriptions 2026-08-11 18:10:27 -05:00
Codex self-assigned this 2026-10-28 12:04:45 -05:00
Member

Implemented and merged in PR #1003 (master revision e2bdbd33).

Game descriptions now apply an explicit weight of 700 to Markdown bold text, including links nested inside bold text. The bold-link override matters because the existing link utility otherwise lowers the rendered weight to 500. Regular text, ordinary links, link colors, spacing, italics, and list styling retain their existing behavior; Markdown sanitization is unchanged. Tailwind generates the scoped typography rules from the Blade utilities, so no custom CSS rule is needed.

Regression coverage checks bold and bold, links inside bold, bold inside links, adjacent text/italics/lists, raw-script stripping, and unsafe-link rejection. All 18 game-profile tests passed. The full PHP suite passed with 1,453 passed and 2 skipped using a 512 MB limit; existing duplicate-constant warnings remain unrelated to this change. Pint, PHPStan, the frontend build, and diff checks passed. Chromium verified the rendered Blade wrapper with compiled assets at desktop/mobile widths in light/dark themes: bold weight 700, regular text 300, ordinary links 500, no overflow or relevant console errors. Remote fonts were excluded from those browser checks.

All four progress checklist items are complete. Deployment requires the updated Blade template and rebuilt Vite CSS/manifest; no migrations or dependency changes are needed. Dev and production deployment verification will follow separately.

Implemented and merged in PR #1003 (master revision e2bdbd33). Game descriptions now apply an explicit weight of 700 to Markdown bold text, including links nested inside bold text. The bold-link override matters because the existing link utility otherwise lowers the rendered weight to 500. Regular text, ordinary links, link colors, spacing, italics, and list styling retain their existing behavior; Markdown sanitization is unchanged. Tailwind generates the scoped typography rules from the Blade utilities, so no custom CSS rule is needed. Regression coverage checks **bold** and __bold__, links inside bold, bold inside links, adjacent text/italics/lists, raw-script stripping, and unsafe-link rejection. All 18 game-profile tests passed. The full PHP suite passed with 1,453 passed and 2 skipped using a 512 MB limit; existing duplicate-constant warnings remain unrelated to this change. Pint, PHPStan, the frontend build, and diff checks passed. Chromium verified the rendered Blade wrapper with compiled assets at desktop/mobile widths in light/dark themes: bold weight 700, regular text 300, ordinary links 500, no overflow or relevant console errors. Remote fonts were excluded from those browser checks. All four progress checklist items are complete. Deployment requires the updated Blade template and rebuilt Vite CSS/manifest; no migrations or dependency changes are needed. Dev and production deployment verification will follow separately.
Member

Deployed PR #1003 to dev and production from merged master revision e2bdbd333ff34ee05e6f16b7c20412760b658eda.

The scoped release installed the updated game-description Blade template and rebuilt Vite manifest/assets, refreshed Blade caches, verified installed and publicly served asset checksums, and preserved 450 unrelated runtime/configuration/dependency files on each environment. Both environments passed description rendering checks for both bold syntaxes and nested bold links. Live homepage and game-profile checks returned HTTP 200, and both pages reference the current assets.

No migrations, dependency installations, environment changes, or domain-data writes were performed. Release manifests record the scoped update as ee6b2847+issue992-3d129125+pr1000-c0341df2+pr1001-7d34d5ab+pr1002-f264f6b6+pr1003-e2bdbd33.

Rollback backups contain the original files, checksums, and deployment receipts:

  • Dev: /home/mvgldev/deploy-backups/pr1003-e2bdbd33-xyhpik15
  • Production: /home/mvgl/deploy-backups/pr1003-e2bdbd33-fxwhjyey

All checklist items remain checked; the issue is closed with status/done.

Deployed PR #1003 to dev and production from merged master revision `e2bdbd333ff34ee05e6f16b7c20412760b658eda`. The scoped release installed the updated game-description Blade template and rebuilt Vite manifest/assets, refreshed Blade caches, verified installed and publicly served asset checksums, and preserved 450 unrelated runtime/configuration/dependency files on each environment. Both environments passed description rendering checks for both bold syntaxes and nested bold links. Live homepage and game-profile checks returned HTTP 200, and both pages reference the current assets. No migrations, dependency installations, environment changes, or domain-data writes were performed. Release manifests record the scoped update as `ee6b2847+issue992-3d129125+pr1000-c0341df2+pr1001-7d34d5ab+pr1002-f264f6b6+pr1003-e2bdbd33`. Rollback backups contain the original files, checksums, and deployment receipts: - Dev: `/home/mvgldev/deploy-backups/pr1003-e2bdbd33-xyhpik15` - Production: `/home/mvgl/deploy-backups/pr1003-e2bdbd33-fxwhjyey` All checklist items remain checked; the issue is closed with `status/done`.
Sign in to join this conversation.
No milestone
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
MyVideoGameList/myvideogamelist.com#927
No description provided.