Skip to content

A few minor width fixes for events cards and buttons#17545

Merged
cnunciato merged 1 commit intomasterfrom
cnunciato/buttons-and-things
Feb 12, 2026
Merged

A few minor width fixes for events cards and buttons#17545
cnunciato merged 1 commit intomasterfrom
cnunciato/buttons-and-things

Conversation

@cnunciato
Copy link
Contributor

@cnunciato cnunciato commented Feb 12, 2026

Fixes a few minor layout and button issues:

  • Makes event cards full-width on the index page
  • Uses a centered layout at medium width for the CTA section at the bottom of event landing pages
buttons-and-stuff.mp4

Fixes https://github.com/pulumi/marketing/issues/1602

@claude
Copy link
Contributor

claude bot commented Feb 12, 2026

Documentation Review

I've reviewed the changes in this PR. The modifications primarily focus on CSS/Tailwind class adjustments to improve layout consistency for events cards and template buttons.

✅ Looks Good

  • Whitespace cleanup: Removed trailing spaces throughout layouts/events/list.html (lines 41, 45, 49, 77, 81, 85, 205, 207, 209, etc.)
  • Blank line removal: Removed extraneous blank line at line 98 in layouts/events/list.html
  • Width consistency: Added w-full to event cards to ensure proper full-width display at all breakpoints
  • Button text wrapping: Added whitespace-nowrap spans to prevent button text from wrapping in footer CTAs
  • Layout improvements: Adjusted alignment classes for better centering (md:text-center, md:justify-center)

📝 Changes Summary

layouts/events/list.html (16 deletions, 15 additions)

  • Removed trailing whitespace from multiple lines
  • Removed one extraneous blank line

layouts/partials/events/event-card.html (line 14)

  • Added w-full class to ensure cards take full width of container

layouts/partials/template-partials/template-card.html (line 45)

  • Removed xl:text-left to standardize button alignment as centered

layouts/partials/template-partials/template-footer-cta.html (lines 21-34)

  • Improved title centering with md:text-center and max-width constraints
  • Updated button container with md:justify-center
  • Changed button widths from lg:w-1/2 xl:w-auto to md:w-1/2 w-full lg:w-auto
  • Wrapped button text in whitespace-nowrap spans to prevent text wrapping

layouts/partials/template-partials/template-location.html (line 53)

  • Added block lg:inline-block text-center for better button responsiveness

layouts/partials/template-partials/template-three-column.html (line 50)

  • Removed xl:text-left to standardize button alignment

✅ No Issues Found

All changes are appropriate layout/styling adjustments with no content, accessibility, or functional concerns.


Mention @claude if you'd like me to provide additional reviews or suggest further improvements.

@pulumi-bot
Copy link
Collaborator

@cnunciato cnunciato merged commit 67e2677 into master Feb 12, 2026
23 of 25 checks passed
@cnunciato cnunciato deleted the cnunciato/buttons-and-things branch February 12, 2026 18:37
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.

3 participants