Skip to content

Bug: Learn Page correct post card and community card display issues#2414

Merged
jlchilders11 merged 8 commits into
developfrom
jc/bug-correct-learn-page
May 20, 2026
Merged

Bug: Learn Page correct post card and community card display issues#2414
jlchilders11 merged 8 commits into
developfrom
jc/bug-correct-learn-page

Conversation

@jlchilders11
Copy link
Copy Markdown
Collaborator

@jlchilders11 jlchilders11 commented May 8, 2026

Issue: #2295

Summary & Context

Implements the new post_card design for the learn page, as well as creating a new community card design.

Changes

  • Changes the left column posts to use the _post_card include
  • Implements _community_card include, with css and html
  • Switches middle column to use _community_card
  • Unhides library carousel on mobile

‼️ Risks & Considerations ‼️

Please list any potential risks or areas that need extra attention during review/testing

Screenshots

Light Mode:

image

Dark Mode:

image

Mobile:

image

Self-review Checklist

  • Tag at least one team member from each team to review this PR

Frontend

  • UI implementation matches Figma design
  • Tested in light and dark mode
  • Responsive / mobile verified
  • Accessibility checked (keyboard navigation, etc.)
  • Ensure design tokens are used for colors, spacing, typography, etc. – No hardcoded values
  • Test without JavaScript (if applicable)
  • No console errors or warnings

@jlchilders11 jlchilders11 linked an issue May 8, 2026 that may be closed by this pull request
Copy link
Copy Markdown
Collaborator

@herzog0 herzog0 left a comment

Choose a reason for hiding this comment

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

Looking good to me, thanks for the fixes!

Comment thread templates/v3/includes/_community_card.html Outdated
@jlchilders11 jlchilders11 requested a review from julhoang May 11, 2026 21:04
Copy link
Copy Markdown
Collaborator

@julioest julioest left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Copy Markdown
Collaborator

@julhoang julhoang left a comment

Choose a reason for hiding this comment

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

@jlchilders11 – I've noticed a few UI issues on this page, it'd would be great if you are able to fix it here while we're at it 🙏

1/ We're missing a bit of space above the link.

Image

2/ The new _post_card and _community_card now have fixed width so mobile and tablet cards are currently looking like this – perhaps we should confirm with Henry if the cards should look like the Post Card (with a max-width) or it should look like Boost Calendar Card (spanning he view-width). The Homepage and Community Page with similar masonry designs are applying the 1st strategy FYI.

Image

3/ We currently have a lot of whitespace at the bottom of the page, is there a way to remove a grid row? 🤔

Image

@ycanales
Copy link
Copy Markdown
Collaborator

Just a minor merge conflict that needs to be taken care of, and I'll have a double check after this merges as #2403 also touches some of the same parts, but shouldn't be a big deal.
Approved 👍

@herzog0 herzog0 mentioned this pull request May 20, 2026
11 tasks
Copy link
Copy Markdown
Collaborator

@kattyode kattyode left a comment

Choose a reason for hiding this comment

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

QA Approved

@jlchilders11 jlchilders11 force-pushed the jc/bug-correct-learn-page branch from 638c7bf to dc6f00f Compare May 20, 2026 20:11
@jlchilders11 jlchilders11 force-pushed the jc/bug-correct-learn-page branch from dc6f00f to 6cf4568 Compare May 20, 2026 21:07
@jlchilders11 jlchilders11 merged commit 3cf9b67 into develop May 20, 2026
4 checks passed
@jlchilders11 jlchilders11 deleted the jc/bug-correct-learn-page branch May 20, 2026 21:10
ycanales added a commit that referenced this pull request May 20, 2026
Adopts the _community_card.html component from PR #2414 for the
'Boost community' section of the Learn page (replacing _join_card.html),
along with its supporting CSS rewrite (learn-page.css cleanup, new
community-card.css) and the registration in components.css. Renames
boost_community_data item 'url' -> 'cta_url' to match the new
component's expected shape.
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.

Task: Learn Page Updates

6 participants