Skip to content

Fix: Add-on highlight message overflows and crushes the add-on title - #1335

Merged
daveearley merged 1 commit into
developfrom
fix/addon-highlight-message-layout
Sep 4, 2026
Merged

Fix: Add-on highlight message overflows and crushes the add-on title#1335
daveearley merged 1 commit into
developfrom
fix/addon-highlight-message-layout

Conversation

@daveearley

Copy link
Copy Markdown
Contributor

Issue this fixes

The add-on highlight message (hi-product-addon-highlight-message, introduced with product add-ons in #1300) was rendered as a white-space: nowrap pill inside the add-on title's flex row. Any message of real length competed with the title for width, so on mobile it either clipped past the card edge or crushed the title onto several lines. It also read as a small tag rather than a highlight, which undercuts the point of the feature. There is no tracked GitHub issue for this; it was found while reviewing the add-on UI.

Change

  • Move the message out of the title into its own block element, rendered as the first child of the add-on card.
  • Style it as a full-bleed solid-accent strip across the top of the highlighted add-on card, using the same uppercase, letter-spaced, centred treatment as the product-level highlight banner. Text wraps freely, so there is no clipping at any width.
  • Give the highlighted add-on card a faint accent border and clip its corners so the strip sits flush inside the rounded card.
  • Drop the flex layout from the add-on title, which only existed to hold the pill.

Reusing the product-level banner language one level down means a highlighted add-on is instantly recognisable, and the strip scales with the narrower inset card. Inactive add-ons (parent product not yet selected) keep the existing dimmed treatment, so the strip fades with the rest of the card. Multiple highlighted add-ons stack as separate cards with their own strips.

Verification

  • npx tsc --noEmit clean for the changed component.
  • Checked on the dev stack against the conference demo event: strip correct in dimmed and active states at desktop width; a long message wraps cleanly at a narrow viewport and sits consistently above a product-level banner; two adjacent highlighted add-ons render as distinct cards.
  • No new user-facing strings; the organiser-side product form preview renders the real widget and picks this up automatically.

The add-on highlight message was a nowrap pill inside the title's flex
row, so any message of real length clipped or forced the title onto
multiple lines, especially on mobile. Render it as a full-width banner
strip across the top of the highlighted add-on card instead, matching
the product-level highlight treatment, so it wraps freely and reads as
a highlight rather than a tag.
@railway-app

railway-app Bot commented Sep 4, 2026

Copy link
Copy Markdown

πŸš… Deployed to the Hi.Events-pr-1335 environment in Hi.Events

Service Status Web Updated
Hi.Events API πŸ•’ Building (View Logs) Web Sep 4, 2026 at 7:35 pm UTC
Hi.Events Frontend πŸ•’ Building (View Logs) Web Sep 4, 2026 at 7:35 pm UTC
3 services not affected by this PR
  • Maildev
  • Hi.Events Scheduler
  • Hi.Events Queue Worker

@railway-app
railway-app Bot temporarily deployed to Hi.Events / Hi.Events-pr-1335 September 4, 2026 19:35 Destroyed
@daveearley
daveearley merged commit dc79472 into develop Sep 4, 2026
3 of 5 checks passed
@github-actions github-actions Bot locked and limited conversation to collaborators Sep 4, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant