Skip to content

ISWA: Net new - Resource Showcase block - #198

Merged
jsandland merged 6 commits into
stagefrom
resource-showcase
Aug 12, 2026
Merged

ISWA: Net new - Resource Showcase block#198
jsandland merged 6 commits into
stagefrom
resource-showcase

Conversation

@jsandland

@jsandland jsandland commented Jul 30, 2026

Copy link
Copy Markdown

Net-new Franklin block for the ISWA redesign page: a featured card
(full-bleed image, title, description, CTA) paired with a stacked list
of secondary items, matching the Figma design with a CSS-only gradient
background.
…ully clickable

Match exact Figma typography tokens (heading, heading-6, title-4, body-md,
label) for colors, sizes, weights, and letter-spacing across the section
heading, featured card, and secondary items. Fix the featured column
collapsing on narrow tablet widths (min-width: 0 on the list grid track),
step down secondary item type size in that range, wrap the whole featured
card in a link to the CTA destination, and add a 20% hover/focus image zoom.
@aem-code-sync

aem-code-sync Bot commented Jul 30, 2026

Copy link
Copy Markdown

Hello, I'm the AEM Code Sync Bot and I will run some actions to deploy your branch and validate page speed.
In case there are problems, just click a checkbox below to rerun the respective action.

  • Re-run all PSI checks
  • Re-run failed PSI checks
  • Re-sync branch
Commits

@aem-code-sync

aem-code-sync Bot commented Jul 30, 2026

Copy link
Copy Markdown
Page Scores Audits Google
📱 /drafts/jsandlan/resource?martech=off PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS PSI
🖥️ /drafts/jsandlan/resource?martech=off PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS PSI

@JasonHowellSlavin JasonHowellSlavin left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Some nitpicky items. I would want to see a test file before I approve though please and thank you!

Comment thread blocks/resource-showcase/resource-showcase.js Outdated
Comment thread blocks/resource-showcase/resource-showcase.js Outdated
Comment thread blocks/resource-showcase/resource-showcase.js
Comment thread blocks/resource-showcase/resource-showcase.css Outdated
Comment thread blocks/resource-showcase/resource-showcase.css Outdated
Remove the authoring-model and CTA-demotion comments, add unit tests,
and improve the featured card's accessibility: the card now gets an
aria-label from its title and hides its (now duplicate) inner content
from assistive tech, instead of reading the full card as a link. Also
drop leading-trim/text-edge (never shipped in any browser under those
names) and the single-line description truncation, letting descriptions
wrap normally instead of risking clipped content.
@aem-code-sync
aem-code-sync Bot temporarily deployed to resource-showcase July 30, 2026 23:36 Inactive
@aem-code-sync
aem-code-sync Bot temporarily deployed to resource-showcase August 3, 2026 17:40 Inactive
@jsandland
jsandland merged commit 0cf409c into stage Aug 12, 2026
7 of 8 checks passed
@jsandland
jsandland deleted the resource-showcase branch August 12, 2026 17:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants