Skip to content

Commit

Permalink
Re-add max-w-xl class to <a> element
Browse files Browse the repository at this point in the history 
Without this, the width of the body is limited, but the grey background of the placard can fil lthe full width of its parent.
  • Loading branch information
@jamesscottbrown
jamesscottbrown authored Oct 13, 2023
1 parent 1c14d41 commit 96fd206
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion packages/ui/src/lib/placardButton/PlacardButton.svelte
View file
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@
{href}
target={newWindow ? '_blank' : '_self'}
rel={newWindow ? 'noopener noreferrer' : ''}
class="bg-core-grey-700 group hover:bg-core-grey-800 transition h-full flex flex-col text-white"
class="bg-core-grey-700 group hover:bg-core-grey-800 transition h-full flex flex-col text-white max-w-xl"
>
<div class="p-4 max-w-lg h-full space-y-2">
{#if $$slots.title}
Expand Down

1 comment on commit 96fd206

@ChrisKnightLDN
Copy link
Contributor

Choose a reason for hiding this comment

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

"Without this, the width of the body is limited, but the grey background of the placard can fill the full width of its parent." - this is as intended! so max-w-xl should NOT be on the a element

Please sign in to comment.