Skip to content

Normalize company heading styles - #15

Merged
JayTeeBat merged 1 commit into
mainfrom
agent/fix-company-heading-style
Jul 20, 2026
Merged

Normalize company heading styles#15
JayTeeBat merged 1 commit into
mainfrom
agent/fix-company-heading-style

Conversation

@JayTeeBat

Copy link
Copy Markdown
Owner

Summary

  • Render every named employer through the same company-heading component.
  • Nest single-role positions beneath the employer exactly like multi-role positions.
  • Keep single-role dates on the company line so the layout does not repeat identical dates.
  • Extend the renderer regression test to compare grouped and single-role employers.

Root cause

Grouped employers used the blue company-name heading, while single-role employers used the dark entry-title pattern as “Position at Company.” Because the markup differed, CSS could not make employer names consistent in heading level, typography, colour, or indentation.

Impact

Forsee Power, McLaren Applied, ARRIVAL, Vantage Power, and Renault now share identical employer heading markup and styling, with roles consistently indented below.

Validation

  • 84 non-browser tests passed
  • 13 Chromium/PDF/site tests passed
  • Codespell passed
  • Two-page PDF budget passed
  • Desktop Chromium render visually inspected

@JayTeeBat
JayTeeBat marked this pull request as ready for review July 20, 2026 00:33
@JayTeeBat
JayTeeBat merged commit fde30c7 into main Jul 20, 2026
4 checks passed
@JayTeeBat
JayTeeBat deleted the agent/fix-company-heading-style branch July 20, 2026 00:34
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