[Bug 30944] New: Regression: Tray icon for furcadia is invisible and non-responsive
http://bugs.winehq.org/show_bug.cgi?id=30944 Bug #: 30944 Summary: Regression: Tray icon for furcadia is invisible and non-responsive Product: Wine Version: 1.3.7 Platform: x86 OS/Version: Linux Status: UNCONFIRMED Severity: minor Priority: P2 Component: -unknown AssignedTo: wine-bugs(a)winehq.org ReportedBy: lethalfalcon(a)endlessrealities.com Classification: Unclassified Regression SHA1: 5d570706c6aca9acbff855c3ae497b35bdde4648 Starting from version 1.3.7, Furcadia's pounce icon disappears from the tray. The space is allocated to it, but the actual icon isn't visible, nor does right-clicking in the space do anything. The icon disappears as well. Running a bisect results in this patch being the culprit: 5d570706c6aca9acbff855c3ae497b35bdde4648 is the first bad commit commit 5d570706c6aca9acbff855c3ae497b35bdde4648 Author: Alexandre Julliard <julliard(a)winehq.org> Date: Mon Nov 8 15:28:09 2010 +0100 winex11: Create Win32 windows corresponding to all the ancestors of embedded windows. This is needed to keep track of the embedded window position by handling ConfigureNotify received by ancestors. :040000 040000 7c3171d11930ae3e1dd7965245f6a90a2b115c7f c30937343ef6532a80879e808f118219e267030c M dlls z If I revert this patch in master, the icon shows up. However, the right-click menu appears at the top of my screen instead of right above the icon as expected. I can assume this is due to one or more of the hunks that failed to patch. I'm not sure what channels I should enable to get a useful log. I tried x11drv, but didn't see anything other than traces, and they looked very similar between versions. Tested using a new profile and an existing profile with identical results. No extra winetricks used on the new profile. This may be related to bug 24719, but the version reported in there is earlier than where I find the issue, so not sure. -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email Do not reply to this email, post in Bugzilla using the above URL to reply. ------- You are receiving this mail because: ------- You are watching all bug changes.
http://bugs.winehq.org/show_bug.cgi?id=30944 Austin English <austinenglish(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Keywords| |regression CC| |julliard(a)winehq.org Summary|Regression: Tray icon for |Tray icon for furcadia is |furcadia is invisible and |invisible and |non-responsive |non-responsive -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email Do not reply to this email, post in Bugzilla using the above URL to reply. ------- You are receiving this mail because: ------- You are watching all bug changes.
http://bugs.winehq.org/show_bug.cgi?id=30944 --- Comment #1 from Alexandre Julliard <julliard(a)winehq.org> 2012-06-19 03:22:40 CDT --- Please test with Wine 1.4, and if still present specify what window manager you are using and how to reproduce the problem. -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email Do not reply to this email, post in Bugzilla using the above URL to reply. ------- You are receiving this mail because: ------- You are watching all bug changes.
http://bugs.winehq.org/show_bug.cgi?id=30944 --- Comment #2 from Joshua Gardynik <lethalfalcon(a)endlessrealities.com> 2012-06-19 09:25:56 CDT --- It's been tested using 1.5.5 and git's master branch, where it's still broken unless I revert this patch. Do you really want me to test every single version from 1.4.0 to 1.5.5 to see which versions break or not from then 'til now? I'm using fluxbox 1.3.2 as a window manager. To reproduce, just start furcadia and select a character (you'll have to create one on first run). Furcadia pounce starts automatically when the character loads. As an alternative, you can try running furc_on.exe (the pounce program) directly instead, but I'm not sure if it will let you with no characters. If it does run, that's all you have to do, since the icon is part of it. -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email Do not reply to this email, post in Bugzilla using the above URL to reply. ------- You are receiving this mail because: ------- You are watching all bug changes.
http://bugs.winehq.org/show_bug.cgi?id=30944 Alexandre Julliard <julliard(a)winehq.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|UNCONFIRMED |RESOLVED Resolution| |UPSTREAM --- Comment #3 from Alexandre Julliard <julliard(a)winehq.org> 2012-06-19 09:58:13 CDT --- (In reply to comment #2)
It's been tested using 1.5.5 and git's master branch, where it's still broken unless I revert this patch. Do you really want me to test every single version from 1.4.0 to 1.5.5 to see which versions break or not from then 'til now?
No, I thought you tested with 1.3.7 since that's what the version field said.
I'm using fluxbox 1.3.2 as a window manager.
To reproduce, just start furcadia and select a character (you'll have to create one on first run). Furcadia pounce starts automatically when the character loads. As an alternative, you can try running furc_on.exe (the pounce program) directly instead, but I'm not sure if it will let you with no characters. If it does run, that's all you have to do, since the icon is part of it.
Thanks. It works fine here with metacity, kwin, xfce, and stalonetray, so it's most likely a bug in fluxbox. -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email Do not reply to this email, post in Bugzilla using the above URL to reply. ------- You are receiving this mail because: ------- You are watching all bug changes.
http://bugs.winehq.org/show_bug.cgi?id=30944 Jerome Leclanche <adys.wh(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |adys.wh(a)gmail.com --- Comment #4 from Jerome Leclanche <adys.wh(a)gmail.com> 2012-06-19 10:40:36 CDT --- If you choose to file an issue with fluxbox, please leave the URL to the bug here or in the See Also field. -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email Do not reply to this email, post in Bugzilla using the above URL to reply. ------- You are receiving this mail because: ------- You are watching all bug changes.
http://bugs.winehq.org/show_bug.cgi?id=30944 --- Comment #5 from Joshua Gardynik <lethalfalcon(a)endlessrealities.com> 2012-06-19 11:49:51 CDT --- I don't want to start a blame game, but I would like to understand what this patch actually does, and why it alone is responsible for icons no longer showing up in fluxbox. Maybe it's a flux issue, but it works fine in flux at wine 1.3.6, and it ALMOST works again if I revert this patch, even with git master, and nothing else seems to be functionally degraded with the reversion either, so what was this patch designed to actually address? The commit comment doesn't seem like it matches with the issue at hand, to me anyway. Fluxbox's development is... honestly, nowhere near as active as wine. I did find a bug open from over a year ago here: https://sourceforge.net/tracker/index.php?func=detail&aid=3311806&group_id=3... It that appears to be the same issue. If I can find time, maybe I can go through the patch piecemeal and try to isolate the issue, but I'm not a c++ developer by any stretch, so I'll be flying rather blind. Luckily, it's only 3 files, so diffing them won't take a horribly long time. It's just having to rebuild after each one that kinda sucks. -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email Do not reply to this email, post in Bugzilla using the above URL to reply. ------- You are receiving this mail because: ------- You are watching all bug changes.
http://bugs.winehq.org/show_bug.cgi?id=30944 --- Comment #6 from Alexandre Julliard <julliard(a)winehq.org> 2012-06-19 12:02:46 CDT --- The patch fixes the tracking of the window location, which as you noticed is broken when you revert it. -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email Do not reply to this email, post in Bugzilla using the above URL to reply. ------- You are receiving this mail because: ------- You are watching all bug changes.
http://bugs.winehq.org/show_bug.cgi?id=30944 --- Comment #7 from Alexandre Julliard <julliard(a)winehq.org> 2012-06-19 12:25:46 CDT --- So the problem is that fluxbox sends a synthetic ConfigureNotify in window coordinates instead of root coordinates as per ICCCM: trace:event:call_event_handler 47 ConfigureNotify for hwnd/window 0x100ea/1000002 trace:event:X11DRV_ConfigureNotify win 0x100ea/1000002 new X rect 0,0,18x18 (event 0,0,18x18) trace:event:X11DRV_ConfigureNotify 0x100ea moving from (0,0) to (-1900,-1180) When mapped from root to window coords it is far off-screen, so the icon doesn't get displayed. Definitely a fluxbox bug. -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email Do not reply to this email, post in Bugzilla using the above URL to reply. ------- You are receiving this mail because: ------- You are watching all bug changes.
http://bugs.winehq.org/show_bug.cgi?id=30944 --- Comment #8 from Joshua Gardynik <lethalfalcon(a)endlessrealities.com> 2012-06-19 13:08:12 CDT --- Cool, thank you very much for looking into it a little more! I'll pass that on, and if necessary, maybe I'll tear apart flux's tray to see if I can figure out how to make that change. Probably won't be any easier, but at least I know now not to blame wine at all. :) -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email Do not reply to this email, post in Bugzilla using the above URL to reply. ------- You are receiving this mail because: ------- You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=30944 Bruno Jesus <00cpxxx(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- See Also| |https://sourceforge.net/tra | |cker/index.php?func=detail& | |aid=3311806&group_id=35398& | |atid=413960 -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=30944 Frédéric Delanoy <frederic.delanoy(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |CLOSED CC| |frederic.delanoy(a)gmail.com --- Comment #9 from Frédéric Delanoy <frederic.delanoy(a)gmail.com> --- Closing -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=30944 --- Comment #10 from Matteo Bruni <matteo.mystral(a)gmail.com> --- (In reply to Frédéric Delanoy from comment #9)
Closing
Looking at the linked bug report, this one might actually be fixed. Has anyone verified that's the case? -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=30944 --- Comment #11 from Frédéric Delanoy <frederic.delanoy(a)gmail.com> --- (In reply to Matteo Bruni from comment #10)
(In reply to Frédéric Delanoy from comment #9)
Closing
Looking at the linked bug report, this one might actually be fixed. Has anyone verified that's the case?
Apparently fixed according to https://github.com/fluxbox/fluxbox/pull/19 -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
participants (1)
-
wine-bugs＠winehq.org