Awn

maximize on multidisplay using twinview does not work as expected

Bug #184358 reported by Mark Hayes
2
Affects Status Importance Assigned to Milestone
Awn
Fix Released
Medium
Michal Hruby

Bug Description

While using AWN with a dual-head configuration (TwinView, nVidia), it is expected that maximized applications would fill the secondary display that AWN is not displayed on.

It appears that this can be traced to a line in awn_x_set_strut function. If I change the xutils_set_strut line to "...icon_offset, x, x + (width - 1));" then all applications maximized on my secondary display cover the entire screen, and applications maximized on the primary display do not cover the bar. Presumably this is because screen coords start at 0, so 1680 in my case would put it on the second desktop space.

Single head configurations work as expected with this change.

--- My apologies for the commenting from myself.

bug #147730 and bug #157714 could benefit from a struts rewrite, similar to gnome-panel's implementation.

Revision history for this message
Mark Hayes (hayesm) wrote :

Edit: tested changes on single head X. Worked as expected, maximized windows did not cover bar. Perhaps there's a flag for XChangeProperty that can be utilized.. width/2 seems like a bit of a hack.

Revision history for this message
Mark Hayes (hayesm) wrote :

Edit: this is something GTK related, and how struts work.

Revision history for this message
Mark Hayes (hayesm) wrote :

Endless additions.....

GNOME-Panel's strut implementation could be adapted, I'll work on it in the next couple of days. Would require some options added to awn-manager. MultiHead checkbox, and orientation drop down (ie, centered on two displays, bottom-left, bottom-right).

New implementation of struts could fix bugs #147730 and #157714

Mark Hayes (hayesm)
description: updated
Mark Hayes (hayesm)
description: updated
Mark Hayes (hayesm)
description: updated
Revision history for this message
Neil J. Patel (njpatel) wrote :

Thanks for the work on this, I know its not ideal at the moment. Any progress you make can also be ported to the awn 0.3 code.

Changed in awn:
importance: Undecided → Medium
status: New → In Progress
Neil J. Patel (njpatel)
Changed in awn:
assignee: nobody → njpatel
milestone: none → 0.2.6
Mark Lee (malept)
tags: added: multi-monitor
Michal Hruby (mhr3)
Changed in awn:
assignee: Neil J. Patel (njpatel) → Michal Hruby (mhr3)
Revision history for this message
Michal Hruby (mhr3) wrote :

Is there any chance you could test this using our testing PPA?
http://wiki.awn-project.org/DistributionGuides#Testing_Package_Archive

Changed in awn:
status: In Progress → Incomplete
Revision history for this message
Michal Hruby (mhr3) wrote :

Since nobody is complaining, marking this as Fix committed. If you find out that this is still an issue, please change the status to New.

Changed in awn:
status: Incomplete → Fix Committed
Mark Lee (malept)
Changed in awn:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

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