Lead single roles with the job title again - #17
Merged
Merged
Conversation
Only the grouped Broadcom run leads with the employer now, since the company is what binds its two titles together. The four standalone roles go back to title first, company beneath. Acquisitions I wasn't around for get an elbow arrow rather than the git merge glyph. Font Awesome's arrow-turn-down-right is Pro only, so this rotates the free arrow-turn-up a quarter turn clockwise; that swaps the glyph's 1em height into its width, hence the 1em icon box, without which the arrow overhangs the left edge of the text above it. The merge icon loses its accent colour, so the icon shape alone carries the during vs after distinction. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Cloudflare previewUpdated for |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Follow-up to #16, reverting one decision from it and tidying the acquisition markers.
Title order
#16 made every experience entry lead with the employer, for consistency with the grouped Broadcom run. That's reverted for the four standalone roles — they lead with the job title again, company beneath:
The grouped run still leads with the employer, since the company is the fact binding its two titles together.
Acquisition markers
arrow-turn-down-rightis Pro only, so this rotates the freearrow-turn-upa quarter turn clockwise —arrow-turn-downrotated the other way gives ↱, not ↳.whois-role-acquisition-duringclass styled nothing after that, so it's removed rather than left as a dead hook.Also picks up an uncommitted copy edit that was sitting in the working tree: the Experience section description is now "Where I've worked, who's paying my bills."
Test plan
pnpm build(includesastro check) — 0 errors, 0 warnings🤖 Generated with Claude Code