Skip to content

pluma-window: Fix bottom and right panels shrinking at each start - #746

Merged
vkareh merged 1 commit into
mate-desktop:masterfrom
Mintsuki:fix-panels-shrinking
Oct 19, 2026
Merged

vkareh merged 1 commit into
mate-desktop:masterfrom
Mintsuki:fix-panels-shrinking

Conversation

@Mintsuki

Copy link
Copy Markdown
Contributor

The bottom panel gets a pixel smaller every time pluma is started, and so does the right panel. For example, with the Python Console in the bottom panel and bottom-panel-size at 284, the setting reads 283, 282, 281... after each following start.

vpaned_restore_position () puts the divider at the paned's height less the saved size, which leaves the panel that size less the divider's handle, but on exit the panel's own height is saved. hpaned_inner_restore_position () does the same for the right panel. This change saves the distance from the divider to the end of the paned instead, which is what the restore functions expect, so the stored size now includes the handle. The side panel isn't affected: it comes first in its paned, so its width is the divider's position.

Tested on Arch Linux (GTK 3.24.52) with 1.28.1 and master, before and after the change, reading the panels' sizes and the saved settings after each start:

  • Five starts in a row: before, the saved sizes went 284, 283, ..., 279 (right panel: 200, ..., 195); after, they stayed at 284 (right panel: 200).
  • Both panels dragged 40 px bigger: after the change, the new sizes are kept from then on.
  • A theme whose paned handle is 10 px: before, the panels lost 10 px per start; after, none.

vpaned_restore_position () and hpaned_inner_restore_position () put the
divider at the paned's size less the saved panel size, which leaves the
panel that size less the divider's handle. On exit the panel's own size
was saved, so the panel lost the size of the handle (1 px with most
themes) every time pluma was started.

Save the distance from the divider to the end of the paned instead,
which is what the restore functions expect.

Assisted-by: LLM
Signed-off-by: Mintsuki <mintsuki@protonmail.com>

@vkareh vkareh 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.

I hadn't even noticed this bug before but it's there - thanks for fixing it.

@vkareh
vkareh merged commit b5e619e into mate-desktop:master Oct 19, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants