Bug 294398 - Visual glitch when slider is in some positions
Summary: Visual glitch when slider is in some positions
Status: CLOSED FIXED
Alias: None
Product: Oxygen
Classification: Plasma
Component: gtk3-engine (show other bugs)
Version: unspecified
Platform: Unlisted Binaries Linux
: NOR normal
Target Milestone: ---
Assignee: Hugo Pereira Da Costa
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-02-18 23:30 UTC by Ruslan Kabatsayev
Modified: 2012-05-15 04:07 UTC (History)
3 users (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Ruslan Kabatsayev 2012-02-18 23:30:37 UTC
Version:           unspecified
OS:                Linux

Screenshot: http://simplest-image-hosting.net/png-0-screenshot-02192012-032535-am
See some elements not drawn, others have text above them (progressBarEntry wrong rendering is unrelated).

Reproducible: Always

Steps to Reproduce:
1. Start oxygen-gtk3-demo, with any gtk3 version, namely 3.3 and 3.2
2. Go to "Sliders"
3. Move any slider to the top/left border
4. Move it 1px from the border

Actual Results:  
Glitch appears

Expected Results:  
Should render normally

This looks like GTK bug, but still, it might be a bug in our cairo usage or something else.
Comment 1 Hugo Pereira Da Costa 2012-03-01 11:25:41 UTC
Git commit a701938d25e54c0ecfb3cd09a91c77b9c16e04e5 by Hugo Pereira Da Costa.
Committed on 01/03/2012 at 12:23.
Pushed by hpereiradacosta into branch 'gtk3'.

Fix unbalanced cairo_context save() restore(), by moving up exit condition.

M  +3    -3    src/oxygenstyle.cpp

http://commits.kde.org/oxygen-gtk/a701938d25e54c0ecfb3cd09a91c77b9c16e04e5
Comment 2 Hugo Pereira Da Costa 2012-03-01 11:27:02 UTC
That fixes it here. (and makes sense).
I believe the context got corrupted due to imbalanced save() restore() calls.
(basically the next restore() in the loop would not roll-back to the right state, and corruptions would keep going on until the context is discarded).

So closing.
Don't hesitate to re-open if bug is still present.
Comment 3 Ruslan Kabatsayev 2012-03-01 15:11:59 UTC
You're right. I confirm that it's fixed.