Skip to content

fix: visualization of multiple attachments#13315

Open
madbob wants to merge 1 commit into
nextcloud:mainfrom
madbob:issue_10408
Open

fix: visualization of multiple attachments#13315
madbob wants to merge 1 commit into
nextcloud:mainfrom
madbob:issue_10408

Conversation

@madbob

@madbob madbob commented Jul 18, 2026

Copy link
Copy Markdown
Collaborator

The original issue occurs on mobile, when expanding the "more attachments" area. I've introduced an explicit scrollable area that permits to navigate them.

The most invasive modification here is the change of the background color, which was a linear gradient and now is a static color. A linear gradient is proportional, and the height of the transparent portion is unpredictable as may vary accordingly to the number of attachments. With many attachments, the transparent area growns and interact badly with the overlapped contents of the message.

I've also introduced some minor adjustments to bottom buttons ("View more attachments", "Show all on Files", "Download zip") alignment on mobile.

(Don't care broken attachment previews in my screencasts: I'm breaking stuff around 😛 )

Original state:

test-original.mp4

New state:

test-mine-changes.mp4

Fixes #10408

@ChristophWurst ChristophWurst left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Tested with responsive mode of FF for Linux & works well

The only nitpick is consideration of using multiple of var(--default-grid-baseline) instead of magic numbers

Signed-off-by: Roberto Guido <info@madbob.org>
@madbob

madbob commented Jul 20, 2026

Copy link
Copy Markdown
Collaborator Author

Indeed, you are so right that I've just removed the existing clickable elements and have replaced them with proper NcButton for improved themability and accessibility. So there is no need at all to hard-wire spacing and margins for them.

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.

Mail with lots of attachments not readable, overlays text

2 participants