Opened 15 years ago

Closed 11 years ago

Last modified 11 years ago

#7958 closed defect (fixed)

gnome panel shows even when "full screen" selected

Reported by: Marc Randolph <mrand@…> Owned by: Isaac Richards
Priority: major Milestone: 0.27.1
Component: MythTV - General Version: 0.22-fixes
Severity: high Keywords:
Cc: Ticket locked: no

Description

Reported by a number of downstream Ubuntu users: if you have a decently powerful desktop that you run your front-end on, the gnome panel shows all the time if you have compiz enabled. I believe they changed how the "go fullscreen" signal is handled a version or two ago.

Attachments (1)

50-fullscreen.diff (576 bytes) - added by Lawrence Rust <lvr@…> 14 years ago.

Download all attachments as: .zip

Change History (10)

comment:1 Changed 15 years ago by stuartm

Priority: minortrivial
Severity: mediumlow
Status: newinfoneeded_new

And when we switched to QT4 we started to use the correct fullscreen flag, so unless someone has changed that since I fixed it, I'm at a loss to explain this as a bug. I cannot replicate with KDE 4.

Does anyone using gnome _not_ have a problem? We need to rule out gnome/ubuntu bugs.

comment:2 Changed 15 years ago by stuartm

Milestone: unknown0.23
Priority: trivialmajor
Severity: lowhigh
Status: infoneeded_newnew

Ok to answer my own question, this was broken by [20534], I'll revert that commit.

comment:3 Changed 15 years ago by stuartm

Resolution: fixed
Status: newclosed

(In [23286]) Revert [20534], reinstate the fullscreen window flag. IsGeometryOverriden? doesn't seem to return true in all cases but that's where the bug fixed by [20534] really lies. Fixes #7958 Reopens #6508

comment:4 Changed 14 years ago by Lawrence Rust <lvr@…>

This bug is still present in branch master running on Ubuntu 10.10 with Compiz enabled. I believe that it's down to a bug in Compiz which needs the window to be visible before the window state is changed to fullscreen. The attached patch fixes this bug.

Changed 14 years ago by Lawrence Rust <lvr@…>

Attachment: 50-fullscreen.diff added

comment:5 Changed 14 years ago by superm1@…

Can someone please reopen this ticket for tracking?

This fix was fixed in 3ad6c81aa83491b7e6d2d54c15daf16894a72299 but then reverted in 003fc8fb14acc9a6c4aeea2001d33d4cdb3c253d until a better solution could be found.

comment:6 Changed 11 years ago by JYA

Resolution: fixed
Status: closednew

comment:7 Changed 11 years ago by Jean-Yves Avenard <jyavenard@…>

Resolution: fixed
Status: newclosed

In 1c440ba970bfa0718bd5bc240f412e0bbc24ccf5/mythtv:

Error: Processor CommitTicketReference failed
GIT backend not available

comment:8 Changed 11 years ago by JYA

Milestone: 0.230.27.1

comment:9 Changed 11 years ago by Jean-Yves Avenard <jyavenard@…>

In 25876a76e4e064ba828d0b92c1bbaea218882c81/mythtv:

Error: Processor CommitTicketReference failed
GIT backend not available
Note: See TracTickets for help on using tickets.