Skip to content

Conversation

@tammi-23
Copy link
Contributor

Refactor tailwind changes:

Related Issue: #937 (comment)
notification-Area

@tammi-23 tammi-23 self-assigned this Sep 10, 2025
@tammi-23 tammi-23 requested a review from JammingBen September 12, 2025 10:20
@tammi-23 tammi-23 requested a review from JammingBen September 12, 2025 11:45
Comment on lines 78 to 81
class="markdown-container flex min-h-0 [mask-image:none]"
:class="{
'mask-image:linear-gradient(180deg,black,transparent)] collapsed': markdownCollapsed
}"
Copy link
Contributor

Choose a reason for hiding this comment

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

I just saw that this removes the gradient completely, which we want to keep. The intention with the min-height was to not make it as noticeable when the description is short. But it's probably just a bad workaround. Better adjust the gradient to be less prominent in general, e.g. via mask-linear-[180deg,black,80%,transparent].

@tammi-23 tammi-23 requested a review from JammingBen September 12, 2025 15:26
Copy link
Contributor

@JammingBen JammingBen left a comment

Choose a reason for hiding this comment

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

👍 (needs a rebase though)

@JammingBen JammingBen merged commit 0325b1f into main Sep 15, 2025
28 checks passed
@JammingBen JammingBen deleted the refactor/tailwind/diff/fix branch September 15, 2025 12:30
@openclouders openclouders mentioned this pull request Sep 15, 2025
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants