Banner is used in-flow as a promotional callout — typically between sections on a Home or Dashboard screen. "Within A Container" leaves horizontal padding on either side so the banner sits as a card; "Full Width" bleeds edge-to-edge. The image or icon sits on the opposite side of the text per the position axis.

• Property names use spaces. with link, with button, with preamble, with icon aren't valid identifiers in any native codegen target. Should be camelCase: hasLink, hasAction, hasPreamble, hasLeadingAsset. C2 · Variant & Property Naming
• Property is a meta-name, not a semantic one. Its values (Within A Container / Full Width) describe outer padding and corner radius — a layout concern the parent should own. Rename to padding if kept, or drop the axis entirely and let the consumer control width + padding. C2 · Variant & Property Naming
• with link + with button encode mutually exclusive CTAs as independent booleans. Both are text + chevron link styles — the distinction is cosmetic. This schema admits the impossible state with link=yes + with button=yes (excluded by authoring convention, not by the schema). Collapse into one action enum. C2 · Variant & Property Naming
• Sparse cartesian variant space. 5 boolean-ish axes × 2 container modes would yield 64 combinations; only 20 ship. Author manually pruned invalid combos — that logic should live in the schema (enums, mutually exclusive props), not in variant authoring discipline. C1 · Layer Structure & Naming
• No first-class image slot. The image is an instance of a separate "Banner Asset Placeholder" component — product teams swap via instance-override rather than a declared Figma Slot. Should be a named asset slot accepting an Image, Illustration, or Gradient. C4 · Native Mappability
• with icon=yes renders a drawn grey circle. The icon slot is a flat #C2C6CF circle, not a swappable Icon / Avatar / Illustration instance. Can't carry a token-bound color or glyph. C6 · Asset & Icon Quality
• Chevron is a raster shape_full PNG. The "learn more" arrow ships as an <img> asset. Doesn't scale cleanly and can't take a token tint. C6 · Asset & Icon Quality
• No pressed / focused / disabled states. The whole banner is tappable but only Default is modeled. Native platforms need pressed (tap feedback) and focused (keyboard/D-pad) at minimum. C5 · Interaction State Coverage
• Container padding modeled as a component variant. Within A Container adds 12 px outer padding and wraps the inner card in a rounded container; Full Width drops both. On native this is a layout-level decision (parent gives Banner its width + padding), not a Figma variant. C4 · Native Mappability
• Code Connect mappings not registered. Blocked until properties are renamed, axes are collapsed, and asset/background slots are adopted. C7 · Code Connect Linkability

• Rename space-separated booleans to camelCase. with link → hasLink, with button → hasAction, with preamble → hasPreamble, with icon → hasLeadingAsset. Matches the DS-wide naming fix applied to Carousel Item and the Form family. Rename
• Collapse with link + with button into one action enum. action: .none | .link("Label") | .button("Label"). Mutually exclusive CTAs shouldn't be modeled as independent booleans — the schema should make with link=yes + with button=yes unrepresentable. Property
• Replace with icon boolean with a leading asset slot. Accept an Icon, Avatar, Illustration, or Image instance. Native: leadingAsset: @ViewBuilder (SwiftUI) / leadingAsset: @Composable () -> Unit (Compose). Eliminates the rigid icon-only placeholder. Slot
• Add a background / image slot. Replace the "Banner Asset Placeholder" instance with a first-class Figma Slot that accepts an Image, Illustration, or Gradient. Native: background: AnyView / background: @Composable () -> Unit. Slot
• Rename Property → drop it or make it padding. Within A Container vs Full Width is a padding + radius concern owned by the parent layout on native. Either remove the axis entirely (the banner fills whatever width its parent hands it) or rename to padding: .container | .full. Property
• Rename position → imagePosition. More specific and self-documenting. Keep as .left | .right enum. Rename
• Consolidate with Carousel - Item. Both components have preamble / heading / description / button / image-with-position slots. The only difference is peek/snap behaviour — which belongs on the carousel container (scroll snap + scale/opacity transforms), not a sibling component. Target: one EBBanner used standalone or inside EBCarousel. Family
• Vectorize the chevron. Swap the raster shape_full for a vector Icon instance — crisp at any scale, token-bound tint. Asset
• Add pressed + focused states. Pressed: 0.98 scale or 6–8% overlay on tap. Focused: 2 px focus ring at border/focus. Banners are always tappable and need both feedback signals. State
• Announce as a single actionable element. VoiceOver and TalkBack should read preamble + heading + description + action label as one announcement with a button/link role. A11y