xfce4-panel 4.11 does not honor compiz virtual horizontal size correctly

Bug #1292820 reported by Cédric Dufour
20
This bug affects 3 people
Affects Status Importance Assigned to Milestone
Xfce4 Panel
Confirmed
Medium
xfce4-panel (Ubuntu)
Triaged
Undecided
Unassigned

Bug Description

Hello,

When increasing compiz virtual horizontal size, xfce4-panel (4.11; Ubuntu/Trusty) workspace swicther shows the additional "workspaces" but without increasing the swicther width accordingly; the width remains as if a single "workspace" is used, and compiz "workspaces" appear squeezed within the available width (the more so as the virtual horizontal size is increased).

Reverting to xfce4-panel 4.10 (Ubuntu/Saucy) fixes the issue.

Best,

Cédric

Tags: patch compiz
Revision history for this message
Cédric Dufour (cdufour-keyword-ubuntu-086000) wrote :

Hello,
The problem comes from:
1. the pager does not use the 'wnck_workspace_get_{width,height}' to calculate its aspect ratio
2. the pager does not recalculate its aspect ratio when Compiz workspaces/viewports settings are changed
Please consider attached patch to fix this behavior.
Once applied, pager size is correct for both Compiz and XFWM4.
Thanks and cheers,
Cédric

Revision history for this message
Ubuntu Foundations Team Bug Bot (crichton) wrote :

The attachment "xfce4-panel.fix-pager-size-for-compiz-viewports.patch" seems to be a patch. If it isn't, please remove the "patch" flag from the attachment, remove the "patch" tag, and if you are a member of the ~ubuntu-reviewers, unsubscribe the team.

[This is an automated message performed by a Launchpad user owned by ~brian-murray, for any issues please contact him.]

tags: added: patch
Revision history for this message
Thaddaeus Tintenfisch (thad-fisch-deactivatedaccount) wrote :

Thank you for taking the time to report this bug and helping to make Ubuntu better. Please forward your patch to the upstream developers by filing a bug report on the Xfce bug tracker.

https://bugzilla.xfce.org/

Revision history for this message
In , Djnz00 (djnz00) wrote :

Created attachment 6076
patch

This bug is originally submitted here: https://bugs.launchpad.net/ubuntu/+source/xfce4-panel/+bug/1292820 by Cedric Dufour. Patch is also his.

1. the pager does not use the 'wnck_workspace_get_{width,height}' to calculate its aspect ratio
2. the pager does not recalculate its aspect ratio when Compiz workspaces/viewports settings are changed

Please consider attached patch to fix this behavior.
Once applied, pager size is correct for both Compiz and XFWM4.

Changed in xfce4-panel:
importance: Unknown → Medium
status: Unknown → Confirmed
Revision history for this message
In , Djnz00 (djnz00) wrote :

bump!

Revision history for this message
In , Charles Bos (charlesbos1) wrote :

Created attachment 6447
Panel switcher screenshot (wrong aspect ratio)

Revision history for this message
In , Charles Bos (charlesbos1) wrote :

Created attachment 6448
Panel switcher screenshot (correct aspect ratio)

Revision history for this message
In , Charles Bos (charlesbos1) wrote :

I've not got much to say other than that this affects me too - Arch Linux, Compiz 0.9.12, Xfce 4.12. To illustrate the problem, I've attached a couple of screenshots.

Revision history for this message
In , Charles Bos (charlesbos1) wrote :

Attached patch works here on Arch as well.

Revision history for this message
Launchpad Janitor (janitor) wrote :

Status changed to 'Confirmed' because the bug affects multiple users.

Changed in xfce4-panel (Ubuntu):
status: New → Confirmed
Revision history for this message
Theo Linkspfeifer (lastonestanding) wrote :

Is this patch still needed?

Changed in xfce4-panel (Ubuntu):
status: Confirmed → Incomplete
Revision history for this message
In , Michael Kogan (michael-kogan) wrote :

The patch is not compatible with Xfce 4.14 any more, is there somebody knowledgeable enough to look into it and check if it can be modified to work with the new Xfce?

Revision history for this message
In , erikjanp (ej-lfs) wrote :

Created attachment 9004
fix pager size for compiz viewports

Here's an updated patch for xfce4-panel-4.14.
When applying, it can give some 'offset xxx lines' notices but that shouldn't be a problem.

Revision history for this message
In , Michael Kogan (michael-kogan) wrote :

Thanks, works perfectly!

Sean Davis (bluesabre)
Changed in xfce4-panel (Ubuntu):
status: Incomplete → Confirmed
status: Confirmed → Triaged
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.