Bug 316995 - oxygen-gtk causes MonoDevelop source editor to be blank
Summary: oxygen-gtk causes MonoDevelop source editor to be blank
Status: RESOLVED FIXED
Alias: None
Product: Oxygen
Classification: Plasma
Component: gtk2-engine (show other bugs)
Version: 4.10
Platform: openSUSE Linux
: NOR grave
Target Milestone: ---
Assignee: Hugo Pereira Da Costa
URL: http://paste.opensuse.org/view/raw/7a...
Keywords:
Depends on:
Blocks:
 
Reported: 2013-03-18 20:23 UTC by Cygni X
Modified: 2014-02-18 16:44 UTC (History)
10 users (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments
Eclipse dialog after commit 86c83b4 (30.46 KB, image/png)
2014-02-18 14:38 UTC, lutz.wrage
Details
Eclipse dialog before commit 86c83b4 (94.01 KB, image/png)
2014-02-18 14:40 UTC, lutz.wrage
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Cygni X 2013-03-18 20:23:34 UTC
When using oxygen-gtk as the GTK2 Theme, MonoDevelop's source editor appears blank or rather gray, rendering the application useless. I have attached a screen shot showing exactly what happens. This was tested on MonoDevelop 3.06 and 4.0.

Reproducible: Always

Steps to Reproduce:
1. Open KDE System Settings
2. Select Application Appearance
3. Select Gtk Configuration
4. Under GTK Themes, choose oxygen-gtk for GTK2 Theme
5. Load MonoDevelopment and try editing your source
Actual Results:  
The source editor is a blank gray screen.

Expected Results:  
A working MonoDevelop source editor.

A workaround is to use a different GTK2 Theme; Adwaita, Oxygen-Molecule, QtCurve, etc.
Comment 1 Ruslan Kabatsayev 2013-03-18 20:25:33 UTC
Does using OXYGEN_DISABLE_INNER_SHADOWS_HACK=1 environment variable when starting the app make this look OK?
Comment 2 Cygni X 2013-03-18 20:34:48 UTC
(In reply to comment #1)
> Does using OXYGEN_DISABLE_INNER_SHADOWS_HACK=1 environment variable when
> starting the app make this look OK?

Yes, I MonoDevelopment is started with "OXYGEN_DISABLE_INNER_SHADOWS_HACK=1 monodevelop" the source editor works fine. By the way, thanks for the quick response.
Comment 3 Cygni X 2013-03-18 20:47:02 UTC
(In reply to comment #1)
> Does using OXYGEN_DISABLE_INNER_SHADOWS_HACK=1 environment variable when
> starting the app make this look OK?

Yes, if MonoDevelopment is started with 'OXYGEN_DISABLE_INNER_SHADOWS_HACK=1 monodevelop', the source editor works fine. By the way, thanks for the quick response.

Correction*
Comment 4 Ruslan Kabatsayev 2013-03-22 20:12:30 UTC
I can't reproduce using MonoDevelop 2.8.6.3 and oxygen-gtk v1.3.2.1-82-g939d3d3 (Ubuntu Precise). And on their website, I get "Packages for MonoDevelop 4.0 not yet available. The latest available version is MonoDevelop 2.6.0.1". Where did you get your versions?
Comment 5 Cygni X 2013-03-25 23:32:25 UTC
(In reply to comment #4)
> I can't reproduce using MonoDevelop 2.8.6.3 and oxygen-gtk
> v1.3.2.1-82-g939d3d3 (Ubuntu Precise). And on their website, I get "Packages
> for MonoDevelop 4.0 not yet available. The latest available version is
> MonoDevelop 2.6.0.1". Where did you get your versions?

The issue exists with MonoDevelop versions 3 and 4 and oxygen-gtk.  Version 4.0 stable was released (Source) February 20.  I have compiled the source and used the one available from (openSUSE), and the issue can be reproduced every time. 

Source: http://download.mono-project.com/sources/monodevelop/monodevelop-4.0.tar.bz2
openSUSE: http://download.opensuse.org/repositories/Mono:/Factory/openSUSE_12.3/
Comment 6 Cygni X 2013-03-25 23:51:17 UTC
(In reply to comment #5)
> (In reply to comment #4)
> > I can't reproduce using MonoDevelop 2.8.6.3 and oxygen-gtk
> > v1.3.2.1-82-g939d3d3 (Ubuntu Precise). And on their website, I get "Packages
> > for MonoDevelop 4.0 not yet available. The latest available version is
> > MonoDevelop 2.6.0.1". Where did you get your versions?
> 
> The issue exists with MonoDevelop versions 3 and 4 and oxygen-gtk.  Version
> 4.0 stable was released (Source) February 20.  I have compiled the source
> and used the one available from (openSUSE), and the issue can be reproduced
> every time. 
> 
> Source:
> http://download.mono-project.com/sources/monodevelop/monodevelop-4.0.tar.bz2
> openSUSE:
> http://download.opensuse.org/repositories/Mono:/Factory/openSUSE_12.3/

I should add that this is issue only exist in MonoDevelop 4.0.
Comment 7 leo_rockway 2013-05-23 02:16:31 UTC
Can replicate using MonoDevelop 4.0 and oxygen-gtk (in Arch, if that matters).
OXYGEN_DISABLE_INNER_SHADOWS_HACK=1 monodevelop works for me.
Comment 8 Cygni X 2013-08-30 19:39:24 UTC
This bug remains unsolved.
Comment 9 Yannick 2013-09-22 09:48:11 UTC
Hi, I've the same problem with Mageia 4 but when I change the GTK theme it doesn't work and I've not an usable editor.
Comment 10 Hugo Pereira Da Costa 2013-09-22 09:52:21 UTC
@Yannick
see comment #3. That should be a quick fix.
Pretty sure its an upstream bug, though. Will have a closer look.
Comment 11 Yannick 2013-09-22 10:58:53 UTC
@Hugo Pereira Da Costa It's working - thanks. But without this hack we have still this problem
Comment 12 Hugo Pereira Da Costa 2013-09-25 14:57:14 UTC
ok. Can reproduce.
Now I can't get any debug output (from stdout or stderr), when running the application, for a reason that I ignore ... this will make it very hard to ... debug ...
Comment 13 Emmanuel Lepage Vallée 2014-02-02 20:39:58 UTC
Same problem with virt-manager
Comment 14 Hugo Pereira Da Costa 2014-02-04 09:36:41 UTC
Git commit 7a9a75bc86923912760699b1b37991cc12fcfd9f by Hugo Pereira Da Costa.
Committed on 06/12/2013 at 12:08.
Pushed by hpereiradacosta into branch 'master'.

only mark treeview children as composited when frame is effectively sunken.
This is similar to commit c7832547dcdfc4f846c7ebd6aa5116cf0c0ae1e7 on the gtk3 branch

M  +4    -1    src/animations/oxygeninnershadowdata.cpp
M  +17   -7    src/oxygenstylewrapper.cpp

http://commits.kde.org/oxygen-gtk/7a9a75bc86923912760699b1b37991cc12fcfd9f
Comment 15 Hugo Pereira Da Costa 2014-02-04 09:38:44 UTC
Commit above fixes it in master
(basically, we don't enable compositing on viewports that are not part of sunken scrolledwindow any more, as was the case before), and monodevelop is not sunken.
Will backport that to 1.4 branch before next release (in a couple of weeks) after making more or less sure that there are no regressions.

Ruslan ? Can you give it a shot too ? 

Hugo
Comment 16 Hugo Pereira Da Costa 2014-02-12 09:18:05 UTC
Git commit 86c83b45e2c3adccaaaed8fc82348b6dea34f367 by Hugo Pereira Da Costa.
Committed on 06/12/2013 at 12:08.
Pushed by hpereiradacosta into branch '1.4'.

only mark treeview children as composited when frame is effectively sunken.
This is similar to commit c7832547dcdfc4f846c7ebd6aa5116cf0c0ae1e7 on the gtk3 branch

M  +4    -1    src/animations/oxygeninnershadowdata.cpp
M  +17   -7    src/oxygenstylewrapper.cpp

http://commits.kde.org/oxygen-gtk/86c83b45e2c3adccaaaed8fc82348b6dea34f367
Comment 17 lutz.wrage 2014-02-18 14:35:05 UTC
Unfortunately this change (86c83b4) breaks functionality in eclipse (kepler 4.3.1 on kubuntu 13.10 with all updates as of today). Tree and table views are either not filled at all when they are shown for the first time, or are not updated in response to mouse clicks. Only after resizing the view the content is updated and they react normally. Overall this makes eclipse pretty much unusable. Without this commit, this problem does not occur. I've switched back and forth a couple of times to confirn that it's really causes by this one commit. I'll attach two screen shots that show the issue.
Comment 18 lutz.wrage 2014-02-18 14:38:18 UTC
Created attachment 85218 [details]
Eclipse dialog after commit 86c83b4
Comment 19 lutz.wrage 2014-02-18 14:40:40 UTC
Created attachment 85219 [details]
Eclipse dialog before commit 86c83b4
Comment 20 Hugo Pereira Da Costa 2014-02-18 15:31:05 UTC
That's probably fixable.
What dialog is that on the screenshots ? How do I open it  ?
Comment 21 lutz.wrage 2014-02-18 15:40:25 UTC
(In reply to comment #20)
> That's probably fixable.
> What dialog is that on the screenshots ? How do I open it  ?
Help -> About Eclipse, then click on "Installation Details"
Comment 22 Hugo Pereira Da Costa 2014-02-18 16:40:34 UTC
Git commit 640fe6d6eb7140b0907e5db6a4c30eaa6b9acc5d by Hugo Pereira Da Costa.
Committed on 18/02/2014 at 16:39.
Pushed by hpereiradacosta into branch '1.4'.

Force shadow on known scrolled windows allready in innerShadowHook

M  +4    -0    src/animations/oxygenanimations.cpp

http://commits.kde.org/oxygen-gtk/640fe6d6eb7140b0907e5db6a4c30eaa6b9acc5d
Comment 23 Hugo Pereira Da Costa 2014-02-18 16:40:57 UTC
That fixes it.
Thanks for reporting !
Comment 24 Hugo Pereira Da Costa 2014-02-18 16:44:22 UTC
Git commit 9d875565e492edce8dbe597d6da7415a9a3487c7 by Hugo Pereira Da Costa.
Committed on 18/02/2014 at 16:39.
Pushed by hpereiradacosta into branch 'gtk3'.

Force shadow on known scrolled windows allready in innerShadowHook

M  +4    -0    src/animations/oxygenanimations.cpp

http://commits.kde.org/oxygen-gtk/9d875565e492edce8dbe597d6da7415a9a3487c7