Skip to content

fix(gui): remove gap after status messages - #10245

Merged
Rello merged 1 commit into
masterfrom
bugfix/noid/messages-sizehint
Jun 24, 2026
Merged

fix(gui): remove gap after status messages#10245
Rello merged 1 commit into
masterfrom
bugfix/noid/messages-sizehint

Conversation

@nilsding

@nilsding nilsding commented Jun 24, 2026

Copy link
Copy Markdown
Member

option.rect seems to be invalid on some platforms within sizeHint; e.g. on Linux it had a width of -1. This caused the text height to be calculated differently to paint() and a visual gap when a message is displayed.

Resolves

#

Summary

TODO

  • ...

Checklist

AI (if applicable)

`option.rect` seems to be invalid on some platforms within `sizeHint`;
e.g. on Linux it had a width of `-1`.  This caused the text height to be
calculated differently to `paint()` and a visual gap when a message is
displayed.

Signed-off-by: Jyrki Gadinger <[email protected]>
@nilsding

Copy link
Copy Markdown
Member Author

/backport to stable-34.0 please

@nilsding

Copy link
Copy Markdown
Member Author

/backport to stable-33.0 please

@nilsding nilsding added this to the 34.0.0 milestone Jun 24, 2026
@nilsding nilsding added feature: ☁️ GUI System tray icon and menu. 3. to review labels Jun 24, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Artifact containing the AppImage: nextcloud-appimage-pr-10245.zip

Digest: sha256:10203d2a2f2e09c2f928fb43373a5d54ae913fdf26f9e9f793e245293be74708

To test this change/fix you can download the above artifact file, unzip it, and run it.

Please make sure to quit your existing Nextcloud app and backup your data.

@Rello
Rello self-requested a review June 24, 2026 13:48
@sonarqubecloud

Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
E Reliability Rating on New Code (required ≥ A)
12 New Code Smells (required ≤ 0)
E Maintainability Rating on New Code (required ≥ A)
1 New Bugs (required ≤ 0)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

@Rello
Rello merged commit 820afef into master Jun 24, 2026
22 of 23 checks passed
@Rello
Rello deleted the bugfix/noid/messages-sizehint branch June 24, 2026 16:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review feature: ☁️ GUI System tray icon and menu.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants