Skip to content

chore: minor graphic adjustments to quick actions modal#13309

Merged
ChristophWurst merged 1 commit into
nextcloud:mainfrom
madbob:improve_quick_actions
Jul 17, 2026
Merged

chore: minor graphic adjustments to quick actions modal#13309
ChristophWurst merged 1 commit into
nextcloud:mainfrom
madbob:improve_quick_actions

Conversation

@madbob

@madbob madbob commented Jul 16, 2026

Copy link
Copy Markdown
Collaborator

Here a totally unrequested adjustment of the design of Quick Action edit modal.

Changes are purely aesthetic, no functionality has been changed.

Notable changes:

  • margins and spacing
  • the d'n'd handle for unsortable actions (Move Thread and Delete Thread) is grayed out, to indicate to the user that those cannot be moved (I tried to drag one of those actions, it didn't worked, I believed I broke something, only reading at the code I found that those cannot be moved and it is normal...)

Working on this I found the bug #13308 which is NOT here targeted.

Those adjustments are subjective and have not been approved by a real designer! Feel free to reject this!

sample

@ChristophWurst

Copy link
Copy Markdown
Member

have not been approved by a real designer

summoning the real @nimishavijay

@hamza221 hamza221 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good, thank you.

@ChristophWurst

Copy link
Copy Markdown
Member

@madbob small nitpick: please amend and change the commit message to something like fix(ui): improve quick actions modal design

background: I want to backport this, and a chore is for changes that do not change the product. They are not added to the automatic changelog. A fix is always added, and I think this is definitely note-worthy.

@ChristophWurst

Copy link
Copy Markdown
Member

/backport to stable5.10

Signed-off-by: Roberto Guido <info@madbob.org>
@madbob
madbob force-pushed the improve_quick_actions branch from 4fb9b21 to 63d08c1 Compare July 17, 2026 17:43
@madbob

madbob commented Jul 17, 2026

Copy link
Copy Markdown
Collaborator Author

Fixed commit comment.

@ChristophWurst
ChristophWurst enabled auto-merge July 17, 2026 17:47
@ChristophWurst
ChristophWurst merged commit 342baec into nextcloud:main Jul 17, 2026
42 checks passed
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.

4 participants