Skip to content

Add product-management journey section with timeline and styles#7

Open
8Gaston8 wants to merge 1 commit into
mainfrom
codex/find-creative-display-for-work-experience
Open

Add product-management journey section with timeline and styles#7
8Gaston8 wants to merge 1 commit into
mainfrom
codex/find-creative-display-for-work-experience

Conversation

@8Gaston8
Copy link
Copy Markdown
Owner

Motivation

  • Surface product-management experience as a narrative timeline so visitors quickly grasp impact and seniority rather than scanning a plain list of jobs.
  • Keep design consistent with the site’s bold, high-contrast visual language while making PM strengths scannable at a glance.
  • Replace stale content-focused tests with checks that validate the new hero/nav/contact and the PM journey content.

Description

  • Added an EXPERIENCE nav anchor and a new #experience section that presents roles as a story-driven PM timeline (Atly, Lumen, Optimove, earlier startups, NASA) with short outcome-oriented bullets.
  • Introduced PM “chips” for quick scanning of core strengths (DISCOVERY, ROADMAPS, STORYTELLING, SHIPPING, IOS CRAFT, AI WORKFLOWS).
  • Added CSS for the new .pm-journey, .pm-stop, .pm-timeline, and .pm-chips rules in kinetic.css to match the existing typography and color system.
  • Updated tests/test_site.py to assert presence of the new nav anchor, PM journey content, key role lines, chips, and existing contact CTA.

Testing

  • Ran the unit tests with python -m unittest -v tests/test_site.py and all tests passed (6 tests, OK).
  • Rendered the site locally and captured a screenshot of the new experience section as a visual verification artifact.

Codex Task

Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 91cf4d9c33

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread kinetic.css
Comment on lines +436 to +439
.pm-stop p {
margin-top: 10px;
font-size: 16px;
line-height: 1.25;
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Preserve period label typography in timeline cards

The generic .pm-stop p rule overrides .pm-period because it appears later and has higher specificity (.pm-stop p vs .pm-period), so the date labels inside each timeline card render at 16px instead of the intended 11px. This makes the period markers visually compete with body copy and breaks the hierarchy the section is trying to establish; either scope the paragraph rule away from .pm-period or re-assert .pm-period styles after this block.

Useful? React with 👍 / 👎.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant