Skip to content

Soften heading weight (600) and round video thumbnail corners#500

Merged
LindsayAuch merged 2 commits into
LindsayAuch/visual-refresh-sandboxfrom
LindsayAuch/heading-weight-video-radius
May 12, 2026
Merged

Soften heading weight (600) and round video thumbnail corners#500
LindsayAuch merged 2 commits into
LindsayAuch/visual-refresh-sandboxfrom
LindsayAuch/heading-weight-video-radius

Conversation

@LindsayAuch
Copy link
Copy Markdown
Collaborator

@LindsayAuch LindsayAuch commented May 1, 2026

Changes

  • Heading weight: bold (700) → 600 for h1 and markdown h2/h3/h4. Aligns with the semibold weight used elsewhere in the refresh.
  • Video thumbnails: added border-radius: 12px + overflow: hidden to .reactPlayer and .react-player wrappers, plus the inner iframe/video. Both classes covered (component-supplied .reactPlayer and the package default .react-player).

Test plan

  • Headings render at 600, not 700 (less heavy)
  • YouTube embeds (e.g. on /authoring-recipes/writing-a-java-refactoring-recipe) have rounded corners

- Heading rule font-weight: bold (700) -> 600 (semibold) for h1 and the
  markdown h2/h3/h4 to match the lighter typographic feel of the rest
  of the refresh
- Add border-radius: 12px (with overflow: hidden) to .reactPlayer and
  .react-player wrappers and their inner iframe/video, so embedded
  YouTube videos render with rounded corners

Refs #492

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
@github-project-automation github-project-automation Bot moved this from In Progress to Ready to Review in OpenRewrite May 1, 2026
@LindsayAuch LindsayAuch marked this pull request as ready for review May 12, 2026 15:01
@LindsayAuch LindsayAuch merged commit cae439d into LindsayAuch/visual-refresh-sandbox May 12, 2026
@LindsayAuch LindsayAuch deleted the LindsayAuch/heading-weight-video-radius branch May 12, 2026 15:02
@github-project-automation github-project-automation Bot moved this from Ready to Review to Done in OpenRewrite May 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants