Banner:Multiline With Buttons

Build 2357 on banner-implementation
Reviewing disabled
View latest build
New storyBuild 2357 on banner-implementation
View Storybook
Baseline History
This story has no baseline yet. Accept this story to set the baseline.
Commenting is disabled because there's a newer build on this branch. Go to Build 2360 now
Pinned
Reony Tonneyck
Were we doing any queries to change the buttons to small  size when it wraps to multi-line?
Resolved
Reony Tonneyck
Reony Tonneyck
3y ago on build 2357

Actually, thinking about it again I don't think we can since we can't detect when text wraps, right? I think it'll be up to the designer to use the small size buttons for a Banner if they choose, and since the ones in Figma use small for multi-line, that'll be the norm coming from them at least.

Nisha Yerunkar
Nisha Yerunkar
3y ago on build 2357

Talked about this offline - we decided to change all buttons to size small.

Reony Tonneyck
Reony Tonneyck
3y ago on build 2359

still seeing the font-size as 16px when I inspect this via the Storybook build. 🤔

Nisha Yerunkar
Nisha Yerunkar
3y ago on build 2360

Yup, I was gonna change it in a new PR.