Skip to content

Fix timeline rendering: year circle overflow, education school name, Kyndryl entry - #1

Merged
donnels merged 2 commits into
masterfrom
copilot/fix-html-rendering-jobs-school
Aug 29, 2026
Merged

donnels merged 2 commits into
masterfrom
copilot/fix-html-rendering-jobs-school

Conversation

Copilot AI commented Aug 29, 2026

Copy link
Copy Markdown

Timeline cards in the Employer History section had a year circle overflow bug and the education entry was displaying an abbreviated/wrong field. Kyndryl is confirmed present as the first role entry.

Changes

  • css/timeline.css — Added min-height: 120px to .main-timeline4 .timeline-content. The year circle is position: absolute; height: 120px but its containing block was only ~80 px tall (short h3 + h5), causing the circle to bleed into the next entry and misalign the connector line.

  • _includes/section_workexperience.html — Education card <h5> and modal body were bound to edu.subject ("Sutton Park", an abbreviation) instead of edu.school ("Sutton Park School Howth Dublin Ireland"). Swapped both to edu.school, consistent with how job cards render role.employer.

…, kyndryl already present

Co-authored-by: donnels <14863623+donnels@users.noreply.github.com>
Copilot AI changed the title Fix HTML rendering of jobs and school timeline section; confirm Kyndryl entry Fix timeline rendering: year circle overflow, education school name, Kyndryl entry Aug 29, 2026
Copilot AI requested a review from donnels August 29, 2026 20:40
Co-authored-by: donnels <14863623+donnels@users.noreply.github.com>
@donnels
donnels marked this pull request as ready for review August 29, 2026 20:47
@donnels
donnels merged commit 42ad1c6 into master Aug 29, 2026
1 check passed
@donnels
donnels deleted the copilot/fix-html-rendering-jobs-school branch August 29, 2026 20:48
Copilot stopped work on behalf of donnels due to an error August 29, 2026 20:52
Copilot stopped work on behalf of donnels due to an error August 29, 2026 20:53
Copilot stopped work on behalf of donnels due to an error August 29, 2026 20:54
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.

2 participants