[Breeze] [Bug 371667] Disabling Widget Animations makes Sidepanel moving impossible

2016-10-25 Thread Hugo Pereira Da Costa via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=371667

--- Comment #9 from Hugo Pereira Da Costa  ---
Thanks ! 
I'll leave this bug open in the meanwhile, waiting for further developments.

-- 
You are receiving this mail because:
You are watching all bug changes.


[Breeze] [Bug 371667] Disabling Widget Animations makes Sidepanel moving impossible

2016-10-25 Thread Hugo Pereira Da Costa via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=371667

--- Comment #7 from Hugo Pereira Da Costa  ---
ok. Not sure what to do about it then ... at least report the bug upstream (Qt)
something like: "setting SH_Widget_Animate to false makes Sidepanel moving
impossible". Will you do that ? Here at the breeze/oxygen level, the only thing
I can do is apply the patch (and force animation of sidepanels all the time
disregarding the actual option), but that would be a regression (triggered by
upstream ...)
Wdyt ?

-- 
You are receiving this mail because:
You are watching all bug changes.


[Breeze] [Bug 371667] Disabling Widget Animations makes Sidepanel moving impossible

2016-10-25 Thread Hugo Pereira Da Costa via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=371667

--- Comment #5 from Hugo Pereira Da Costa  ---
Thanks for testing. So it is an upstream bug indeed (the only thing the patch
does is change the value of a flag which is used internally by Qt and by Qt
only. 
The version of Qt I have is 5.7.0 (and I do not have the bug). what is yours ?

-- 
You are receiving this mail because:
You are watching all bug changes.


[Breeze] [Bug 371667] Disabling Widget Animations makes Sidepanel moving impossible

2016-10-25 Thread Hugo Pereira Da Costa via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=371667

--- Comment #3 from Hugo Pereira Da Costa  ---
... or rather: not crashes, but: if you can again move panels?

-- 
You are receiving this mail because:
You are watching all bug changes.


[Breeze] [Bug 371667] Disabling Widget Animations makes Sidepanel moving impossible

2016-10-25 Thread Hugo Pereira Da Costa via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=371667

Hugo Pereira Da Costa  changed:

   What|Removed |Added

 CC||hugo.pereira.da.costa@gmail
   ||.com

--- Comment #2 from Hugo Pereira Da Costa  ---
Hi,
Thanks for reporting. However I cannot reproduce 
I suspect a bug in Qt, in fact. (which I do not have in my version).
The only thing disabling animations does to panels in breeze and oxygen is
change the returned value of "SH_Widget_Animate"

if you have access to breeze sources, can you apply the following patch and see
if code still crashes ?

diff --git a/kstyle/breezestyle.cpp b/kstyle/breezestyle.cpp
index 8a02d75..fa98a7b 100644
--- a/kstyle/breezestyle.cpp
+++ b/kstyle/breezestyle.cpp
@@ -646,7 +646,8 @@ namespace Breeze
 case SH_Menu_SloppySubMenus: return true;

 #if QT_VERSION >= 0x05
-case SH_Widget_Animate: return
StyleConfigData::animationsEnabled();
+// case SH_Widget_Animate: return
StyleConfigData::animationsEnabled();
+case SH_Widget_Animate: return true;
 case SH_Menu_SupportsSections: return true;
 #endif

-- 
You are receiving this mail because:
You are watching all bug changes.


[Breeze] [Bug 368974] System Settings | App. Style | Window Decorations -> Breeze button size settings are incorrect

2016-10-08 Thread Hugo Pereira Da Costa via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=368974

--- Comment #8 from Hugo Pereira Da Costa  ---
Thanks ! and thanks for reporting in the first place.

-- 
You are receiving this mail because:
You are watching all bug changes.


[Breeze] [Bug 368974] System Settings | App. Style | Window Decorations -> Breeze button size settings are incorrect

2016-10-04 Thread Hugo Pereira Da Costa via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=368974

--- Comment #4 from Hugo Pereira Da Costa  ---
Ok. That fixes the button size.
Not the font, but on my system (which is a mix of kde4 and plasma5, I cannot
get the plasma5 fonts properly taken into account, and thus cannot
confirm/test/fix the issue. Will try once I have a full plasma setup

-- 
You are receiving this mail because:
You are watching all bug changes.


[Breeze] [Bug 368974] System Settings | App. Style | Window Decorations -> Breeze button size settings are incorrect

2016-10-04 Thread Hugo Pereira Da Costa via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=368974

--- Comment #3 from Hugo Pereira Da Costa  ---
Git commit b5fd63ad24bca26d3357847996e2c9ec848efc84 by Hugo Pereira Da Costa.
Committed on 04/10/2016 at 10:24.
Pushed by hpereiradacosta into branch 'Plasma/5.8'.

Call updateButtonsGeometryDelayed on settings reconfigured, so that button
sizes are updated immediately

M  +1-0kdecoration/breezedecoration.cpp

http://commits.kde.org/breeze/b5fd63ad24bca26d3357847996e2c9ec848efc84

-- 
You are receiving this mail because:
You are watching all bug changes.


[Breeze] [Bug 368974] System Settings | App. Style | Window Decorations -> Breeze button size settings are incorrect

2016-10-04 Thread Hugo Pereira Da Costa via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=368974

--- Comment #2 from Hugo Pereira Da Costa  ---
Note that here at least, resizing the window, makes the button size change
visible (probably because of triggering a repaint of the decoration), this
without restarting kwin.
Can you confirm ?

-- 
You are receiving this mail because:
You are watching all bug changes.


[Breeze] [Bug 368974] System Settings | App. Style | Window Decorations -> Breeze button size settings are incorrect

2016-10-04 Thread Hugo Pereira Da Costa via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=368974

Hugo Pereira Da Costa  changed:

   What|Removed |Added

 CC||hugo.pereira.da.costa@gmail
   ||.com

--- Comment #1 from Hugo Pereira Da Costa  ---
Confirmed.
Checking whether this is actually breeze or kdecoration

-- 
You are receiving this mail because:
You are watching all bug changes.


[Breeze] [Bug 369559] Crash when realizing QComboBox with icons

2016-09-30 Thread Hugo Pereira Da Costa via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=369559

Hugo Pereira Da Costa  changed:

   What|Removed |Added

 CC||hugo.pereira.da.costa@gmail
   ||.com

--- Comment #2 from Hugo Pereira Da Costa  ---
Hi,
Thanks for posting.
To see if this is Breeze or Qt, can you try with a different style ? (kmail
-style fusion)

-- 
You are receiving this mail because:
You are watching all bug changes.


[kwin] [Bug 368969] Crashed after changing look'n'feel

2016-09-26 Thread Hugo Pereira Da Costa via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=368969

Hugo Pereira Da Costa  changed:

   What|Removed |Added

  Latest Commit|http://commits.kde.org/bree |http://commits.kde.org/oxyg
   |ze/614b5cbf3711dc43b8f76057 |en/7717a77457d65210577dba55
   |ab7cf6c9c99fd3f7|e468accae3ff33f1

--- Comment #10 from Hugo Pereira Da Costa  ---
Git commit 7717a77457d65210577dba55e468accae3ff33f1 by Hugo Pereira Da Costa.
Committed on 26/09/2016 at 13:10.
Pushed by hpereiradacosta into branch 'Plasma/5.8'.

Check widget->windowHandle() before accessing its parent

M  +1-1kstyle/oxygenshadowhelper.cpp

http://commits.kde.org/oxygen/7717a77457d65210577dba55e468accae3ff33f1

-- 
You are receiving this mail because:
You are watching all bug changes.


[kwin] [Bug 368969] Crashed after changing look'n'feel

2016-09-26 Thread Hugo Pereira Da Costa via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=368969

Hugo Pereira Da Costa  changed:

   What|Removed |Added

 Status|UNCONFIRMED |RESOLVED
  Latest Commit||http://commits.kde.org/bree
   ||ze/614b5cbf3711dc43b8f76057
   ||ab7cf6c9c99fd3f7
 Resolution|--- |FIXED

--- Comment #9 from Hugo Pereira Da Costa  ---
Git commit 614b5cbf3711dc43b8f76057ab7cf6c9c99fd3f7 by Hugo Pereira Da Costa.
Committed on 26/09/2016 at 13:08.
Pushed by hpereiradacosta into branch 'Plasma/5.8'.

Check widget->windowHandle() before accessing its parent

M  +1-1kstyle/breezeshadowhelper.cpp

http://commits.kde.org/breeze/614b5cbf3711dc43b8f76057ab7cf6c9c99fd3f7

-- 
You are receiving this mail because:
You are watching all bug changes.


[kwin] [Bug 368969] Crashed after changing look'n'feel

2016-09-21 Thread Hugo Pereira Da Costa via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=368969

--- Comment #8 from Hugo Pereira Da Costa  ---
(In reply to Tony from comment #7)
> Had another crashed today, on my main Neon install (not a vm), as soon as i
> clicked "Gnome application style". System setting does not have a reporting
> link added
> 
> I attached the backtrace of this one, hope it helps clear things out.

This crash is unrelated (deletion of kstyle). 
It is due to deletion of the style with unloading the plugin.
Should be fixed after committing https://git.reviewboard.kde.org/r/128761/ (or
https://git.reviewboard.kde.org/r/128760)

-- 
You are receiving this mail because:
You are watching all bug changes.


[kwin] [Bug 368969] Crashed after changing look'n'feel

2016-09-20 Thread Hugo Pereira Da Costa via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=368969

Hugo Pereira Da Costa  changed:

   What|Removed |Added

   Target Milestone|--- |5
Product|Breeze  |kwin
   Assignee|hugo.pereira.da.costa@gmail |kwin-bugs-n...@kde.org
   |.com|
  Component|QStyle  |wayland-generic

--- Comment #5 from Hugo Pereira Da Costa  ---
Assigning to kwin. I'm no expert on wayland, don't have a wayland setup to
test, and don't know why this line of code is there. Maybe Martin does.

-- 
You are receiving this mail because:
You are watching all bug changes.


[Breeze] [Bug 368969] Crashed after changing look'n'feel

2016-09-19 Thread Hugo Pereira Da Costa via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=368969

Hugo Pereira Da Costa  changed:

   What|Removed |Added

 CC||hugo.pereira.da.costa@gmail
   ||.com,
   ||kwin-bugs-n...@kde.org

--- Comment #3 from Hugo Pereira Da Costa  ---
Adding kwin people for they added the wayland shadows

-- 
You are receiving this mail because:
You are watching all bug changes.


[Breeze] [Bug 368048] Very strange QLineEdit inside QMenu glitch under extreemly specific conditions

2016-09-08 Thread Hugo Pereira Da Costa via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=368048

--- Comment #6 from Hugo Pereira Da Costa  ---
Note that among the two fixes, the second is prefered: it keeps the menu
corners round.
The first one gets you a square menu (by effectively disabling the style's
rounding feature, by calling setStylesheet after polishing)

-- 
You are receiving this mail because:
You are watching all bug changes.


[Breeze] [Bug 368048] Very strange QLineEdit inside QMenu glitch under extreemly specific conditions

2016-09-08 Thread Hugo Pereira Da Costa via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=368048

--- Comment #5 from Hugo Pereira Da Costa  ---
Hello again,
I can reproduce.
so indeed this is some sort of conflict between setting a stylesheet, and
setting the WA_TranslucentBackground flag. 
So really, something fishy with Qt.

Now, I do not want to remove the flag as set from the style (that would make
all the menus boring square, in breeze and oxygen). However, I found to
possible fixes, on the application side (the qt-bug).

Fix 1: call mMenu->ensurePolished() before setting the stylesheet. For some
reason, setting the TranslucentBackground flag before setting the stylesheet
fixes it (likely because the stylesheet disables again the flag).

Fix 2: rather than using a stylesheed for just changing the background color,
use native Qt API and change the QPalette: 
auto palette = mMenu->palette();
palette.setColor( QPalette::Background, qRgb( 240, 0, 0 ) );
mMenu->setPalette( palette );

Here, both changes fix the issue. Any chance you can push that upstream ? 

Best regards, 

Hugo

-- 
You are receiving this mail because:
You are watching all bug changes.


[Breeze] [Bug 368048] Very strange QLineEdit inside QMenu glitch under extreemly specific conditions

2016-09-08 Thread Hugo Pereira Da Costa via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=368048

Hugo Pereira Da Costa  changed:

   What|Removed |Added

 CC||hugo.pereira.da.costa@gmail
   ||.com

--- Comment #4 from Hugo Pereira Da Costa  ---
Hello,
thanks for reporting. I'm investigating. I suspect this has to do with setting
Qt::WA_TranslucentBackground to QMenu (line 326 of breezestyle.cpp), in order
to have round corners.
Will keep you posted. (in the meanwhile if someone can try disable that setting
and try see if the bug is still present, that would help)

Best,

Hugo

-- 
You are receiving this mail because:
You are watching all bug changes.


[Oxygen] [Bug 367894] On GTK apps Windows drag mode incorrectly drags from non titlebar area

2016-09-08 Thread Hugo Pereira Da Costa via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=367894

--- Comment #11 from Hugo Pereira Da Costa  ---
Hello,
Thanks for reporting.
This is now fixed. I have not modified oxygen-gtk, which is basically 'done' in
the sense that it will not have any new releases because of gtk3 having it
broken upstream. Instead I have re-added the removed "WindowDragEnabled"
option, for backward compatibility with past releases.

Sorry for the trouble.
Best,

Hugo

-- 
You are receiving this mail because:
You are watching all bug changes.


[Oxygen] [Bug 367894] On GTK apps Windows drag mode incorrectly drags from non titlebar area

2016-09-08 Thread Hugo Pereira Da Costa via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=367894

Hugo Pereira Da Costa  changed:

   What|Removed |Added

  Latest Commit||http://commits.kde.org/oxyg
   ||en/06ac09ba9196d9968feefae2
   ||94b6b361fb26cfd6
 Status|CONFIRMED   |RESOLVED
 Resolution|--- |FIXED
  Latest Commit|http://commits.kde.org/oxyg |http://commits.kde.org/oxyg
   |en/06ac09ba9196d9968feefae2 |en/f8e1bb11435881a5e0e8dd0e
   |94b6b361fb26cfd6|d3aa1321a5a742f3

--- Comment #9 from Hugo Pereira Da Costa  ---
Git commit 06ac09ba9196d9968feefae294b6b361fb26cfd6 by Hugo Pereira Da Costa.
Committed on 08/09/2016 at 08:50.
Pushed by hpereiradacosta into branch 'Plasma/5.7'.

Re-added WindowDragEnabled, for backward compatibility with oxygen-gtk

M  +1-0kstyle/config/oxygenstyleconfig.cpp
M  +5-0kstyle/oxygen.kcfg

http://commits.kde.org/oxygen/06ac09ba9196d9968feefae294b6b361fb26cfd6

--- Comment #10 from Hugo Pereira Da Costa  ---
Git commit f8e1bb11435881a5e0e8dd0ed3aa1321a5a742f3 by Hugo Pereira Da Costa.
Committed on 08/09/2016 at 08:48.
Pushed by hpereiradacosta into branch 'master'.

Re-added WindowDragEnabled, for backward compatibility with oxygen-gtk

M  +1-0kstyle/config/oxygenstyleconfig.cpp
M  +5-0kstyle/oxygen.kcfg

http://commits.kde.org/oxygen/f8e1bb11435881a5e0e8dd0ed3aa1321a5a742f3

-- 
You are receiving this mail because:
You are watching all bug changes.


[Oxygen] [Bug 367894] On GTK apps Windows drag mode incorrectly drags from non titlebar area

2016-09-08 Thread Hugo Pereira Da Costa via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=367894

Hugo Pereira Da Costa  changed:

   What|Removed |Added

  Latest Commit||http://commits.kde.org/oxyg
   ||en/06ac09ba9196d9968feefae2
   ||94b6b361fb26cfd6
 Status|CONFIRMED   |RESOLVED
 Resolution|--- |FIXED
  Latest Commit|http://commits.kde.org/oxyg |http://commits.kde.org/oxyg
   |en/06ac09ba9196d9968feefae2 |en/f8e1bb11435881a5e0e8dd0e
   |94b6b361fb26cfd6|d3aa1321a5a742f3

--- Comment #9 from Hugo Pereira Da Costa  ---
Git commit 06ac09ba9196d9968feefae294b6b361fb26cfd6 by Hugo Pereira Da Costa.
Committed on 08/09/2016 at 08:50.
Pushed by hpereiradacosta into branch 'Plasma/5.7'.

Re-added WindowDragEnabled, for backward compatibility with oxygen-gtk

M  +1-0kstyle/config/oxygenstyleconfig.cpp
M  +5-0kstyle/oxygen.kcfg

http://commits.kde.org/oxygen/06ac09ba9196d9968feefae294b6b361fb26cfd6

--- Comment #10 from Hugo Pereira Da Costa  ---
Git commit f8e1bb11435881a5e0e8dd0ed3aa1321a5a742f3 by Hugo Pereira Da Costa.
Committed on 08/09/2016 at 08:48.
Pushed by hpereiradacosta into branch 'master'.

Re-added WindowDragEnabled, for backward compatibility with oxygen-gtk

M  +1-0kstyle/config/oxygenstyleconfig.cpp
M  +5-0kstyle/oxygen.kcfg

http://commits.kde.org/oxygen/f8e1bb11435881a5e0e8dd0ed3aa1321a5a742f3

-- 
You are receiving this mail because:
You are watching all bug changes.


[Oxygen] [Bug 368411] oxygen on qt5 is missing dropdown indicator on toolbuttons

2016-09-08 Thread Hugo Pereira Da Costa via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=368411

Hugo Pereira Da Costa  changed:

   What|Removed |Added

 Resolution|--- |INVALID
 Status|UNCONFIRMED |RESOLVED
 CC||hugo.pereira.da.costa@gmail
   ||.com

--- Comment #2 from Hugo Pereira Da Costa  ---
Hello, 
Thanks for reporting. However, the change is intentional. (the same is
implemented in breeze, as well as in the kde4 "backport" of oxygen at kf5.
The rationale is that
- this arrow was causing a lot of rendering issues
- is not really necessary, for toolbutton with 'immediate' menu popup (that is:
for which the only action is to raise a popup).
The arrow is still there for toolbuttons with a separate menu (the left-most
button in your screenshot), as well as for buttons with "delayed" popup, when
you can either click to perform the action, or press-and-wait to raise the menu
(although the use of such delayed popup menu buttons is discouraged).

Does that make sense ? 
So closing as "invalid" (or rather: design choice)
Best regards,

Hugo

-- 
You are receiving this mail because:
You are watching all bug changes.


[Breeze] [Bug 368337] Consider disabling Qt::WA_OpaquePaintEvent for QSplitterHandle

2016-09-06 Thread Hugo Pereira Da Costa via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=368337

--- Comment #8 from Hugo Pereira Da Costa  ---
what I can try on the style size is to peek from the application what type of
widget would be below the proxy before showing it, and disable, when detecting
GL (or others that would cause issue). 
I guess I'll give it a shot, using your test app. 
what do you think ?

-- 
You are receiving this mail because:
You are watching all bug changes.


[Breeze] [Bug 368337] Consider disabling Qt::WA_OpaquePaintEvent for QSplitterHandle

2016-09-06 Thread Hugo Pereira Da Costa via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=368337

Hugo Pereira Da Costa  changed:

   What|Removed |Added

 CC||hugo.pereira.da.costa@gmail
   ||.com

--- Comment #5 from Hugo Pereira Da Costa  ---
Hello, 
thanks for reporting and posting the test case. This is a known issue,
unfortunately unrelated to Qt::WA_OpaquePaintEvent

This is related to breeze' "extended splitter" feature (also implemented in
oxygen and in some other fancy styles on the web).
The code is in breeze/kstyle/breezesplitterproxy
Basically we increasse the splitter hit region on hover to make it easier to
click on, by means of an internal transparent widget, that capture mouse
events.
The artifact you see is Qt not being able to properly draw this transparent
internal widget on top of a GL widget (it lacks some internal compositing, I
guess).
Note that the size of this widget is reduced to 1x1 when you press the mouse
(which you also see, as a dot, in your test case).

Bottomline: there is not much we can do about it, except disable the feature
... (or ask qt to fix, which I am not sure is easy)

-- 
You are receiving this mail because:
You are watching all bug changes.


[Breeze] [Bug 367976] Add option to toggle titlebar seperator

2016-09-06 Thread Hugo Pereira Da Costa via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=367976

Hugo Pereira Da Costa  changed:

   What|Removed |Added

 Resolution|--- |FIXED
 Status|UNCONFIRMED |RESOLVED

--- Comment #3 from Hugo Pereira Da Costa  ---
Hi, 
I'm not sure I really understand the problem and the fix, but if 'fixed', I am
glad.
Closing.

-- 
You are receiving this mail because:
You are watching all bug changes.


[Breeze] [Bug 367976] Add option to toggle titlebar seperator

2016-09-05 Thread Hugo Pereira Da Costa via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=367976

Hugo Pereira Da Costa  changed:

   What|Removed |Added

 CC||hugo.pereira.da.costa@gmail
   ||.com

--- Comment #1 from Hugo Pereira Da Costa  ---
Hi, 
Thanks for reporting.

What is the use case for having such an option ?

I can understand that one would want to remove it when chosing the same window
decoration background color as the window background (which is done
automatically already), 
but I cannot find a reason for doing it in any other case. Is there ? 

If not, I am afraid this will be a "wontfix" since the separator is seen as a
design choice, and because we try to avoid adding to many "corner case" options
(they tend to clutter the code, generate bugs and make things less
maintainable)

Regards,

Hugo

-- 
You are receiving this mail because:
You are watching all bug changes.


[Breeze] [Bug 368019] Resolution selection box not wide enough with scrollbars

2016-09-05 Thread Hugo Pereira Da Costa via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=368019

Hugo Pereira Da Costa  changed:

   What|Removed |Added

 CC||hugo.pereira.da.costa@gmail
   ||.com

--- Comment #2 from Hugo Pereira Da Costa  ---
mmm. My configuration panel does not show a combobox, but a slider (see:
http://wstaw.org/m/2016/09/05/plasma-desktopHJ3061.png)

This is possibly because I do not have enough resolutions available. Makes it
hard to reproduce the problem. In any case, the easyway to see if this is a
style issue is to run

systemsettings5 -style fusion

look for the same configuration panel, and see if the problem persists. 
@Fabian: can you test/check ? 
In the meanwhile I'll try setup a test case.

@Sebastian, maybe this is related to the way the combobox is filled. I know
combobox have different SizeAdjustPolicy, and they do matter, if for instance
the combobox is filled after it was first shown. 
Again. Can't really test for now.

-- 
You are receiving this mail because:
You are watching all bug changes.


[Breeze] [Bug 347280] Edit Application Specific Settings panel overflows the screen

2016-09-02 Thread Hugo Pereira Da Costa via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=347280

Hugo Pereira Da Costa  changed:

   What|Removed |Added

  Flags||Usability+
 CC||kwin-bugs-n...@kde.org

--- Comment #4 from Hugo Pereira Da Costa  ---
Re-ccing Leon, and adding usability.
Question is whether 
1. Breeze style should be made narrower for all applications and dialogs, in
order for this one to fit in the screen, or
2. This dialog should be redesigned in order for it to fit in the screen,
without changing the style.

Based on the posted screenshot, I would clearly vote for 2

-- 
You are receiving this mail because:
You are watching all bug changes.


[Breeze] [Bug 356940] kalarm crash on start up

2016-08-27 Thread Hugo Pereira Da Costa via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=356940

--- Comment #13 from Hugo Pereira Da Costa  ---
(In reply to Peter Wu from comment #12)
> Created attachment 100797 [details]
> Testcase (ASAN) with normal QApplication::quit() and exit()
> 
> Extended the testcase to do normal QApplication::quit(), the proposed
> patches would still result in a crash for that case. Better rip this delete
> hack completely.

Agreed (as also on said on review board)
Thanks for all the investigation and patch submission etc.

-- 
You are receiving this mail because:
You are watching all bug changes.


[plasmashell] [Bug 352986] Plasma Widget Configuration Dialogue ignores Breeze Windows' Drag mode

2016-08-26 Thread Hugo Pereira Da Costa via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=352986

--- Comment #8 from Hugo Pereira Da Costa  ---
@deshole

Sorry to say, but comments 6 and 7 are quite unrelated to this bug.
Bug refers to plasma QML widgets while your issue is related to Firefox, which
in turn uses gtk as a backend.
Could you actually check which widget style is used for firefox (and other gtk2
applications) ? on your system ?

-- 
You are receiving this mail because:
You are watching all bug changes.


[Breeze] [Bug 367723] Visual glitch in QQC1 ToolButton+Menu

2016-08-24 Thread Hugo Pereira Da Costa via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=367723

Hugo Pereira Da Costa  changed:

   What|Removed |Added

 Resolution|--- |FIXED
 Status|UNCONFIRMED |RESOLVED
  Latest Commit||http://commits.kde.org/bree
   ||ze/325c8f610659fa48a38783f3
   ||1260f970be7600f3

--- Comment #4 from Hugo Pereira Da Costa  ---
Git commit 325c8f610659fa48a38783f31260f970be7600f3 by Hugo Pereira Da Costa.
Committed on 24/08/2016 at 08:14.
Pushed by hpereiradacosta into branch 'Plasma/5.7'.

Homogenize how toolbutton arrow is detected between
- drawToolButtonComplexControl
- toolButtonSizeFromContents
- and toolButtonSubControlRect

M  +2-2kstyle/breezestyle.cpp

http://commits.kde.org/breeze/325c8f610659fa48a38783f31260f970be7600f3

-- 
You are receiving this mail because:
You are watching all bug changes.


[Breeze] [Bug 367723] Visual glitch in QQC1 ToolButton+Menu

2016-08-24 Thread Hugo Pereira Da Costa via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=367723

--- Comment #3 from Hugo Pereira Da Costa  ---
well. Forget about upstream. At least in breeze and oxygen I found an
inconsistency about how detection of toolbutton arrow is detecting, resulting
in the invalid rect passed to rendering the button arrow. This fixed, arrow
dissappears (as it should), and bug is resolved.
I'll push.

-- 
You are receiving this mail because:
You are watching all bug changes.


[Breeze] [Bug 367723] Visual glitch in QQC1 ToolButton+Menu

2016-08-24 Thread Hugo Pereira Da Costa via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=367723

Hugo Pereira Da Costa  changed:

   What|Removed |Added

 CC||hugo.pereira.da.costa@gmail
   ||.com

--- Comment #2 from Hugo Pereira Da Costa  ---
Hi
Thanks for reporting.
I can reproduce the problem with oxygen and breeze.
However, I can also see it bugged with fusion and windows
See: http://wstaw.org/m/2016/08/24/plasma-desktopDF2120.png
The arrow in the center (overlayed with the icon) should definitly not be
there.
Same is true for windows style. Can you reproduce ? 
I am using Qt-5.7.0

(on window its even worse: the icon dissapear on mouse-over, and only the arrow
remains).
In fact the artifact seen with breeze/oxygen, is the style trying to render
this arrow too, but with invalid rectangle passed.
So I would say this is upstream.
I can patch breeze and oxygen in order not to render this arrow at all (since
it is bugged), but that would be just a workaround.
Opinion ?

-- 
You are receiving this mail because:
You are watching all bug changes.


[Breeze] [Bug 366628] Some icons jump when hovered in KDE4 applications

2016-08-22 Thread Hugo Pereira Da Costa via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=366628

Hugo Pereira Da Costa  changed:

   What|Removed |Added

 CC||hugo.pereira.da.costa@gmail
   ||.com

--- Comment #2 from Hugo Pereira Da Costa  ---
Hi
Thanks for reporting.
I cannot reproduce it here.
In order to narrow down the issue, does it also happen
- with another widget style (oxygen ? fusion ?)
- with another icon theme ? 
- other applications/icons ? 

Also, what is your icon size ?

thx

Hugo

-- 
You are receiving this mail because:
You are watching all bug changes.


[Oxygen] [Bug 367483] Close window on Window Menu double click

2016-08-21 Thread Hugo Pereira Da Costa via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=367483

Hugo Pereira Da Costa  changed:

   What|Removed |Added

 CC||hugo.pereira.da.costa@gmail
   ||.com

--- Comment #2 from Hugo Pereira Da Costa  ---
Hello Dotan,
Thanks for reporting.
One question: are we talking about kde4, or Plasma 5 ?
For kde4, the feature is there, available on option (in the "fine tuning"
section of the decoration configuration) and I just checked that it works as
expected.

For plasma5, the feature is there also, it is set once and for all for all
decorations, in the "buttons" tab of the decorations settings. Here also, I
checked that it works as expected, for oxygen as well as breeze.

Can you double check ?

-- 
You are receiving this mail because:
You are watching all bug changes.


[Breeze] [Bug 361811] Breeze causes hangs and crashes in Krita

2016-08-02 Thread Hugo Pereira Da Costa via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=361811

Hugo Pereira Da Costa  changed:

   What|Removed |Added

 CC||hugo.pereira.da.costa@gmail
   ||.com

--- Comment #18 from Hugo Pereira Da Costa  ---
Any 'Steps to Reproduce' ? 
thx

-- 
You are receiving this mail because:
You are watching all bug changes.


[Breeze] [Bug 363995] Cannot scroll at screen corner with QT_AUTO_SCREEN_SCALE_FACTOR=1

2016-07-26 Thread Hugo Pereira Da Costa via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=363995

Hugo Pereira Da Costa  changed:

   What|Removed |Added

 Status|UNCONFIRMED |CONFIRMED
 Ever confirmed|0   |1

--- Comment #2 from Hugo Pereira Da Costa  ---
Confirmed (now that I have brand new Qt-5.7)
more soon

-- 
You are receiving this mail because:
You are watching all bug changes.


[Breeze] [Bug 364426] Right margin too small in KPageDialog

2016-07-20 Thread Hugo Pereira Da Costa via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=364426

Hugo Pereira Da Costa  changed:

   What|Removed |Added

  Latest Commit|http://commits.kde.org/bree |http://commits.kde.org/oxyg
   |ze/4fb5a3bf8b1abcfcdabcf8a9 |en/7d989d35643adfcd987560b9
   |c88276a44b8c2a31|d3266b76edbf8246

--- Comment #5 from Hugo Pereira Da Costa  ---
Git commit 7d989d35643adfcd987560b9d3266b76edbf8246 by Hugo Pereira Da Costa.
Committed on 20/07/2016 at 14:57.
Pushed by hpereiradacosta into branch 'Plasma/5.7'.

Fixed tabwidget size calculation from contents

M  +1-15   kstyle/oxygenstyle.cpp

http://commits.kde.org/oxygen/7d989d35643adfcd987560b9d3266b76edbf8246

-- 
You are receiving this mail because:
You are watching all bug changes.


[Oxygen] [Bug 365866] When using a KDE4 color scheme with contrast=0, KF5 uses a background gradient anyway

2016-07-20 Thread Hugo Pereira Da Costa via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=365866

--- Comment #7 from Hugo Pereira Da Costa  ---
Hello,
did you try, in systemsetting5 (the version for kf5) to set the contrast to
zero ? 
By default I get the same as your screenshot from comment 3, but then if I set
the contrast to zero (from systemsetting5, and not the kde4 version), then I
get something that is similar to that posted in comment 5 (your kde4
screenshot).

-- 
You are receiving this mail because:
You are watching all bug changes.


[Breeze] [Bug 364426] Right margin too small in KPageDialog

2016-07-20 Thread Hugo Pereira Da Costa via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=364426

Hugo Pereira Da Costa  changed:

   What|Removed |Added

 Resolution|--- |FIXED
  Latest Commit||http://commits.kde.org/bree
   ||ze/4fb5a3bf8b1abcfcdabcf8a9
   ||c88276a44b8c2a31
 Status|CONFIRMED   |RESOLVED

--- Comment #4 from Hugo Pereira Da Costa  ---
Git commit 4fb5a3bf8b1abcfcdabcf8a9c88276a44b8c2a31 by Hugo Pereira Da Costa.
Committed on 20/07/2016 at 14:17.
Pushed by hpereiradacosta into branch 'Plasma/5.7'.

Fixed tabwidget size calculation from contents

M  +2-17   kstyle/breezestyle.cpp

http://commits.kde.org/breeze/4fb5a3bf8b1abcfcdabcf8a9c88276a44b8c2a31

-- 
You are receiving this mail because:
You are watching all bug changes.


[Oxygen] [Bug 365866] When using a KDE4 color scheme with contrast=0, KF5 uses a background gradient anyway

2016-07-20 Thread Hugo Pereira Da Costa via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=365866

Hugo Pereira Da Costa  changed:

   What|Removed |Added

 CC||hugo.pereira.da.costa@gmail
   ||.com

--- Comment #2 from Hugo Pereira Da Costa  ---
Hello,
first: thanks for reporting ! 
second: could you post screenshot of your favorite application with and without
the bug ? 
(or: how it looks with kf5 and how with kde4) ? 
That would help me check 
- if I can reproduce
- and how to debug
Thanks ! 

Hugo

-- 
You are receiving this mail because:
You are watching all bug changes.


[Breeze] [Bug 364426] Right margin too small in KPageDialog

2016-07-14 Thread Hugo Pereira Da Costa via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=364426

Hugo Pereira Da Costa  changed:

   What|Removed |Added

 Ever confirmed|0   |1
 Status|UNCONFIRMED |CONFIRMED

-- 
You are receiving this mail because:
You are watching all bug changes.


[Breeze] [Bug 364849] Tooltip Text in several qt apps such as Amarok and Krusader is unreadable (white on white) with the Breeze/Breeze Dark color schemes

2016-07-14 Thread Hugo Pereira Da Costa via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=364849

Hugo Pereira Da Costa  changed:

   What|Removed |Added

 CC||hugo.pereira.da.costa@gmail
   ||.com

--- Comment #2 from Hugo Pereira Da Costa  ---
Can you post a screenshot ? 
Here it looks just fine: 
http://wstaw.org/m/2016/07/14/plasma-desktopmp8175.png

-- 
You are receiving this mail because:
You are watching all bug changes.


[Breeze] [Bug 365318] KRDC is unreadable under breeze dark theme

2016-07-14 Thread Hugo Pereira Da Costa via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=365318

Hugo Pereira Da Costa  changed:

   What|Removed |Added

 CC||hugo.pereira.da.costa@gmail
   ||.com

--- Comment #13 from Hugo Pereira Da Costa  ---
Hi, Chiming in.
>From the screenshots of k3b, ktorrent and okular, it does seem that Qt is set
to use the QT-Gtk theme (which converts a gtk theme into a Qt theme), and then
gtk set to use breeze (breeze-dark ?).
This configuration is not supported. 

I would recommand try to set the Qt theme to use breeze directly (and
breeze-dark as a color theme). this should make (e.g.) okular, look like this: 

http://wstaw.org/m/2016/07/14/plasma-desktopDS8175.png

Which is fine

-- 
You are receiving this mail because:
You are watching all bug changes.


[Breeze] [Bug 365628] SH_Widget_Animate is true despite animations being disabled

2016-07-14 Thread Hugo Pereira Da Costa via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=365628

Hugo Pereira Da Costa  changed:

   What|Removed |Added

 Status|UNCONFIRMED |RESOLVED
  Latest Commit||http://commits.kde.org/oxyg
   ||en/a1573b1756fab439c50df0f0
   ||ab060899bf568dfd
 Resolution|--- |FIXED

--- Comment #4 from Hugo Pereira Da Costa  ---
Git commit a1573b1756fab439c50df0f0ab060899bf568dfd by Hugo Pereira Da Costa.
Committed on 14/07/2016 at 16:22.
Pushed by hpereiradacosta into branch 'Plasma/5.7'.

sync SH_Widget_Animate with internal "enable animations" option

M  +1-0kstyle/oxygenstyle.cpp

http://commits.kde.org/oxygen/a1573b1756fab439c50df0f0ab060899bf568dfd

-- 
You are receiving this mail because:
You are watching all bug changes.


[Breeze] [Bug 365628] SH_Widget_Animate is true despite animations being disabled

2016-07-14 Thread Hugo Pereira Da Costa via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=365628

--- Comment #3 from Hugo Pereira Da Costa  ---
Fixed with rBREEZE2b61ea484d05

-- 
You are receiving this mail because:
You are watching all bug changes.


[Breeze] [Bug 365628] SH_Widget_Animate is true despite animations being disabled

2016-07-14 Thread Hugo Pereira Da Costa via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=365628

--- Comment #2 from Hugo Pereira Da Costa  ---
oh
but then I noticed that the ugly combobox is gone for KF5 ! 
ok. I'll make the switch then.

-- 
You are receiving this mail because:
You are watching all bug changes.


[Breeze] [Bug 365628] SH_Widget_Animate is true despite animations being disabled

2016-07-14 Thread Hugo Pereira Da Costa via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=365628

Hugo Pereira Da Costa  changed:

   What|Removed |Added

 CC||hugo.pereira.da.costa@gmail
   ||.com

--- Comment #1 from Hugo Pereira Da Costa  ---
Hi Christoph
this ... is complicated. 
right now breeze (and oxygen) style return the value from parent kstyle, which
in turn reads: 
KConfigGroup g(KSharedConfig::openConfig(), "KDE-Global GUI Settings");
return g.readEntry("GraphicEffectsLevel", true);

This option is defined in the fine tuning kcm, is a combobox with many choices,
such as "low display resolution and high cpu", "low display resolution and very
high cpu", "high display resolution and low cpu", etc. 

Based on this code, only the first combobox entry (low resolution low cpu)
disable SH_Widget_Animate

So I can either: 
1/ overwrite this in oxygen and breeze and use their own "disable animations"
flag; 
2/ Or remove the disable animations option and rely on this combobox instead.
3/ leave it as it is now.

To be honest, I would vote for 1/ (which you suggest) or 3/ (as it is now)
because I am really not happy about this fine tuning combobox.

Personally I have no clue what high or low cpu means, (especially these days
where we have multiple cores) , nor high and low screen resolutions (especially
since we can plug the same computer  on multiple screens of the different
resolution)

Besides, on the developer side, I have no clue how to turn these multiple
choices into a "should I enable this animation or not".

Your opinion ? 
You still think I should switch to sync  SH_Widget_Animate with breeze/oxygen
internal "enable animations" options ? 
What about this graphical effects level ? Should this guy be revisited, removed
?

-- 
You are receiving this mail because:
You are watching all bug changes.


[Breeze] [Bug 365603] Wrong tab order in the advanced settings for breeze windows decorations

2016-07-14 Thread Hugo Pereira Da Costa via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=365603

Hugo Pereira Da Costa  changed:

   What|Removed |Added

 CC||hugo.pereira.da.costa@gmail
   ||.com

--- Comment #3 from Hugo Pereira Da Costa  ---
Thanks for reporting. This is now fixed (and backported)

-- 
You are receiving this mail because:
You are watching all bug changes.


[Breeze] [Bug 365603] Wrong tab order in the advanced settings for breeze windows decorations

2016-07-14 Thread Hugo Pereira Da Costa via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=365603

Hugo Pereira Da Costa  changed:

   What|Removed |Added

  Latest Commit||http://commits.kde.org/bree
   ||ze/1b90d45bfae12b61ab3f5b4f
   ||3b2ee82fb03f9731
 Resolution|--- |FIXED
 Status|UNCONFIRMED |RESOLVED

--- Comment #2 from Hugo Pereira Da Costa  ---
Git commit 1b90d45bfae12b61ab3f5b4f3b2ee82fb03f9731 by Hugo Pereira Da Costa.
Committed on 14/07/2016 at 15:27.
Pushed by hpereiradacosta into branch 'Plasma/5.7'.

Fixed tab order

M  +4-2kdecoration/config/ui/breezeconfigurationui.ui

http://commits.kde.org/breeze/1b90d45bfae12b61ab3f5b4f3b2ee82fb03f9731

-- 
You are receiving this mail because:
You are watching all bug changes.


[Breeze] [Bug 364546] Breeze style does not apply for KDE4 applications like okular or amarok

2016-06-24 Thread Hugo Pereira Da Costa via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=364546

--- Comment #3 from Hugo Pereira Da Costa  ---
ok thanks !

-- 
You are receiving this mail because:
You are watching all bug changes.


[Breeze] [Bug 363995] Cannot scroll at screen corner with QT_AUTO_SCREEN_SCALE_FACTOR=1

2016-06-24 Thread Hugo Pereira Da Costa via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=363995

Hugo Pereira Da Costa  changed:

   What|Removed |Added

 CC||hugo.pereira.da.costa@gmail
   ||.com

--- Comment #1 from Hugo Pereira Da Costa  ---
Sorry can't test yet because I do not have (yet) Qt-5.6 installed.
Will try as soon as I have it.

-- 
You are receiving this mail because:
You are watching all bug changes.


[Breeze] [Bug 364226] theme should highlight the default action in dialogs

2016-06-24 Thread Hugo Pereira Da Costa via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=364226

Hugo Pereira Da Costa  changed:

   What|Removed |Added

 CC||hugo.pereira.da.costa@gmail
   ||.com

--- Comment #4 from Hugo Pereira Da Costa  ---
"The GTK Breeze theme does not indicate the default buttons in dialogs with
this lighter gray, or at all." this should go to a separate bug report too,
with product being "gtk theme" because the gtk theme is not maintained by the
same people.

For the QStyle at hand, well, when default button is set by the app, it is
indicated (as you noted, by light grey). So it is in oxygen too. 
That this is too subtle is a matter of taste. Especially since there are other
ways to use the button (using the mouse, or with the tab key. Note that the tab
key will always work (also in kate when no default button is set), and that the
button will be highlighted in a much more visible way in this case.

-- 
You are receiving this mail because:
You are watching all bug changes.


[Breeze] [Bug 364546] Breeze style does not apply for KDE4 applications like okular or amarok

2016-06-24 Thread Hugo Pereira Da Costa via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=364546

Hugo Pereira Da Costa  changed:

   What|Removed |Added

  Component|QtQuickControls |QStyle
 CC||hugo.pereira.da.costa@gmail
   ||.com
   Assignee|k...@davidedmundson.co.uk|hugo.pereira.da.costa@gmail
   ||.com

--- Comment #1 from Hugo Pereira Da Costa  ---
If indeed you are using kde4 version of the said applications, they are not
affected by the update to plasma5.7
if you want them to use breeze, you must have the breeze style *for kde4*
installed, which I dont think your "update to plasma 5.7" provides.
I can give you instructions to compile it by hand if you want.
(but you would need some kde4 development packages of course)

-- 
You are receiving this mail because:
You are watching all bug changes.


[Oxygen] [Bug 364693] When upgrading from Qt5.4 to Qt5.6, the color and theme handling ignores local settings

2016-06-24 Thread Hugo Pereira Da Costa via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=364693

--- Comment #5 from Hugo Pereira Da Costa  ---
(In reply to Ruslan Kabatsayev from comment #4)
> @Hugo
> Is kde4-config optional in KF5? It seems to be there with the default
> install.

No clue to be honest. I'm still running kde4 (with bits and pieces of kf5)
I guess you must have it as long as you have qt4/kde4 apps installed ...

-- 
You are receiving this mail because:
You are watching all bug changes.


[Oxygen] [Bug 364693] When upgrading from Qt5.4 to Qt5.6, the color and theme handling ignores local settings

2016-06-24 Thread Hugo Pereira Da Costa via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=364693

Hugo Pereira Da Costa  changed:

   What|Removed |Added

 CC||hugo.pereira.da.costa@gmail
   ||.com

--- Comment #1 from Hugo Pereira Da Costa  ---
Dear Hans-Peter,
First of all, thanks a lot for the kind words !
Concerning the issue at hand I am confused. In principle the way oxygen-gtk
retrieves its configuration from qt should not depend on the qt version (and
even less on qt5, because it is all kde4 stuff).
In more details, it looks for kde4 configuration files in the directory list
returned by "kde4-config --path config", and loads whatever is found in
kdeglobals and in oxygenrc. 
None of this should depend on qt5 version
(in fact, to Ruslan, we should probably update it to also use the output of
"kf5-config --path config")

Can you check the output of kde4-config for the two versions of qt5 you are
testing, see if kdeglobals and oxygenrc are there in both cases, and see
whether its content is consistent (colors, gradient) with what you see on
screen, and what you want to have for your gtk apps ? 

On my side I will try compile qt-5.6 (I'm on 5.5 here), to see if there is
something fishy, but no guarantee.

Best regards,

Hugo

-- 
You are receiving this mail because:
You are watching all bug changes.


[Breeze] [Bug 364426] Right margin too small in KPageDialog

2016-06-18 Thread Hugo Pereira Da Costa via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=364426

Hugo Pereira Da Costa  changed:

   What|Removed |Added

 CC||hugo.pereira.da.costa@gmail
   ||.com

--- Comment #3 from Hugo Pereira Da Costa  ---
Confirmed. will investigate. Must be something inconsistent in some
sizeFromContent calculations ...

-- 
You are receiving this mail because:
You are watching all bug changes.


[Oxygen] [Bug 364194] Now way to tell windows apart and to find which window is active when compositing is off

2016-06-10 Thread Hugo Pereira Da Costa via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=364194

Hugo Pereira Da Costa  changed:

   What|Removed |Added

 CC||hugo.pereira.da.costa@gmail
   ||.com

--- Comment #2 from Hugo Pereira Da Costa  ---
Hi Ruslan,
Agreed, this was not ported.
You can however uncheck the new option "use the same color for the titlebar and
the window content"

This will make the decoration use the "active" and "inactive" colors, and fix
the issue (in the same way as breeze does)

-- 
You are receiving this mail because:
You are watching all bug changes.


[Breeze] [Bug 363811] window titlebar too large

2016-06-03 Thread Hugo Pereira Da Costa via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=363811

Hugo Pereira Da Costa  changed:

   What|Removed |Added

 CC||plasma-de...@kde.org

--- Comment #5 from Hugo Pereira Da Costa  ---
thanks for the screenshot. Not sure if this is related, but indeed, it seems
all your "proportions" have been broken for some reason. I wonder if something
has changed in the  way plasma calculates it's "dpi-independent" scale factors.
(which are then applied everywhere, in QML as well as in the window
decoration).

I've cc'd plasma devs to this report.

-- 
You are receiving this mail because:
You are watching all bug changes.


[Breeze] [Bug 360864] No menu shadows with Qt5 style

2016-06-03 Thread Hugo Pereira Da Costa via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=360864

Hugo Pereira Da Costa  changed:

   What|Removed |Added

 Status|UNCONFIRMED |RESOLVED
 Resolution|--- |UPSTREAM

-- 
You are receiving this mail because:
You are watching all bug changes.


[Breeze] [Bug 358302] Highlighted checkbox or radio button becomes visually inverted

2016-06-02 Thread Hugo Pereira Da Costa via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=358302

Hugo Pereira Da Costa  changed:

   What|Removed |Added

 CC||hugo.pereira.da.costa@gmail
   ||.com

--- Comment #1 from Hugo Pereira Da Costa  ---
This is now fixed hopefully. 
See: http://wstaw.org/m/2016/06/03/plasma-desktopx28797.png
and http://wstaw.org/m/2016/06/03/plasma-desktopR28797.png

Feel free to reopen if you are not convinced though.

-- 
You are receiving this mail because:
You are watching all bug changes.


[Breeze] [Bug 360864] No menu shadows with Qt5 style

2016-06-02 Thread Hugo Pereira Da Costa via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=360864

Hugo Pereira Da Costa  changed:

   What|Removed |Added

 CC||hugo.pereira.da.costa@gmail
   ||.com

--- Comment #1 from Hugo Pereira Da Costa  ---
Thanks for reporting. Sadly I cannot reproduce and have no clue how to debug.
Besides, the code is very similar (well: identical), between breeze for Qt4 and
breeze for Qt5.
Can you first post a screenshot of the issue ? 
(sorry for being slow at answering, i have been busy with other stuff)

-- 
You are receiving this mail because:
You are watching all bug changes.


[Breeze] [Bug 363653] Missing control over settings in breeze theme

2016-06-02 Thread Hugo Pereira Da Costa via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=363653

Hugo Pereira Da Costa  changed:

   What|Removed |Added

   Severity|normal  |wishlist
 CC||hugo.pereira.da.costa@gmail
   ||.com

-- 
You are receiving this mail because:
You are watching all bug changes.


[Breeze] [Bug 363811] window titlebar too large

2016-06-02 Thread Hugo Pereira Da Costa via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=363811

Hugo Pereira Da Costa  changed:

   What|Removed |Added

 CC||hugo.pereira.da.costa@gmail
   ||.com

--- Comment #2 from Hugo Pereira Da Costa  ---
Hi,
Thanks for reporting
Strange though. nothing has changed in breeze window decoration for a long time
now, and I cannot reproduce here. 
Not sure how to debug it. Does something similar happens with other decorations
? (oxygen ? Plastique ?)

-- 
You are receiving this mail because:
You are watching all bug changes.


[Breeze] [Bug 363790] slider dialog is unusable with breeze theme

2016-06-01 Thread Hugo Pereira Da Costa via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=363790

Hugo Pereira Da Costa  changed:

   What|Removed |Added

 CC||hugo.pereira.da.costa@gmail
   ||.com

--- Comment #2 from Hugo Pereira Da Costa  ---
Cannot reproduce. Neither with breeze (@kde4), nor oxygen. Christoph, can you ? 
(and they share the same code with respect to window dragging, so that I would
be surprise that the behave differently)
Otherwise, a complete test case would help

-- 
You are receiving this mail because:
You are watching all bug changes.


[Breeze] [Bug 358925] Icon for mount error is missleading

2016-05-19 Thread Hugo Pereira Da Costa via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=358925

Hugo Pereira Da Costa  changed:

   What|Removed |Added

 CC|hugo.pereira.da.costa@gmail |
   |.com|

-- 
You are receiving this mail because:
You are watching all bug changes.


[kwin] [Bug 362860] kwin missing setting for wheel event and double click, on the frame

2016-05-10 Thread Hugo Pereira Da Costa via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=362860

--- Comment #5 from Hugo Pereira Da Costa  ---
(In reply to Thomas Lübking from comment #3)
> @Hugo, do you intercept wheel events on the edges?

Nope. No event interception whatsoever in breeze decoration.

-- 
You are receiving this mail because:
You are watching all bug changes.

[Oxygen] [Bug 362089] tooltips and menus in Qt5 apps are missing shadow

2016-04-22 Thread Hugo Pereira Da Costa via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=362089

Hugo Pereira Da Costa  changed:

   What|Removed |Added

 CC||hugo.pereira.da.costa@gmail
   ||.com

--- Comment #3 from Hugo Pereira Da Costa  ---
Thanks for reporting the issue.
I cannot reproduce however. (meaning: here I have consistent shadows for both
qt4 and qt5 applications).

Also the issue is surprising, since qt4 and qt5 versions of oxygen (and breeze)
share the same code to pass menu and tooltip shadows to kwin. 
You are running with kwin_x11 (from kf5) as a window manager, right ? 
- can you check that shadows are indeed enabled in oxygen settings (either via
systemsettings5, style, oxygen -> configuration, or using oxygen-settings5)
- do you have the same issue when using breeze as a widget theme ? (you should:
both code path are pretty similar)
- and what is the Qt5 version you are using ?

-- 
You are receiving this mail because:
You are watching all bug changes.


[kwin] [Bug 361154] Running native OpenGL games - with compositing enabled - breaks X11 Plasma 5.6.0 Window Decorations

2016-04-16 Thread Hugo Pereira Da Costa via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=361154

--- Comment #51 from Hugo Pereira Da Costa  ---
(In reply to Thomas Lübking from comment #49)
> @Hugo, I assume shadow pixmaps run on some SHM between breeze style and deco?

Nope. Two independent code path.
One is used "internally" in kdecoration
The other is passed to kwin eiser via X property or Wayland SHM (thanks to
martin code)

In both cases the generation of the shadow is cached, and the rendering of it
is left upstream. so I would look there (upstream) to the unbalanced
save/restore

Not much internet connection on my side atm, but I'll double check whether
there could be some QPainter issue on my side as soon as I get some more.

Hugo

-- 
You are receiving this mail because:
You are watching all bug changes.

[Breeze] [Bug 356940] kalarm crash on start up

2016-03-30 Thread Hugo Pereira Da Costa via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=356940

--- Comment #5 from Hugo Pereira Da Costa  ---
Git commit 32746a734d2825f76ddaaa903f6c80ceaa40a28f by Hugo Pereira Da Costa.
Committed on 30/03/2016 at 13:41.
Pushed by hpereiradacosta into branch 'master'.

Create animation "on demand", that is, when there is at least one animated
progressbar.
Delete it when all animated progressbars have stoped or have been unregistered.
This prevent the underlying QPropertyAnimation to be deleted "too late" after
an application was exited,
which in turn sometimes result in a crash
changes. Lines starting # with '#' will be ignored, and an empty message aborts
the commit.

M  +47   -17   kstyle/animations/oxygenbusyindicatorengine.cpp
M  +3-5kstyle/animations/oxygenbusyindicatorengine.h

http://commits.kde.org/oxygen/32746a734d2825f76ddaaa903f6c80ceaa40a28f

-- 
You are receiving this mail because:
You are watching all bug changes.


[Breeze] [Bug 356940] kalarm crash on start up

2016-03-30 Thread Hugo Pereira Da Costa via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=356940

--- Comment #4 from Hugo Pereira Da Costa  ---
Git commit f56b8b9fbe97f83cf18523dde3a1f83fada9beab by Hugo Pereira Da Costa.
Committed on 30/03/2016 at 13:39.
Pushed by hpereiradacosta into branch 'master'.

Create animation "on demand", that is, when there is at least one animated
progressbar.
Delete it when all animated progressbars have stoped or have been unregistered.
This prevent the underlying QPropertyAnimation to be deleted "too late" after
an application was exited,
which in turn sometimes result in a crash

M  +47   -17   kstyle/animations/breezebusyindicatorengine.cpp
M  +2-3kstyle/animations/breezebusyindicatorengine.h

http://commits.kde.org/breeze/f56b8b9fbe97f83cf18523dde3a1f83fada9beab

-- 
You are receiving this mail because:
You are watching all bug changes.


[Breeze] [Bug 356940] kalarm crash on start up

2016-03-27 Thread Hugo Pereira Da Costa via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=356940

Hugo Pereira Da Costa  changed:

   What|Removed |Added

 CC||hugo.pere...@free.fr

--- Comment #3 from Hugo Pereira Da Costa  ---
Hi, 
thanks for posting the simple test case.
I can reproduce the crash with qt5, (but not with qt4).
Will investigate.

-- 
You are receiving this mail because:
You are watching all bug changes.


[Breeze] [Bug 360061] Colour list squashed vertically.

2016-03-21 Thread Hugo Pereira Da Costa via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=360061

Hugo Pereira Da Costa  changed:

   What|Removed |Added

  Latest Commit|http://commits.kde.org/bree |http://commits.kde.org/bree
   |ze/69fd1e6e519484254eebfcd7 |ze/84d19e95e3e7b6df64ea4aa2
   |76458ea49272e5e9|f44a8a8f6a8c92ab

--- Comment #6 from Hugo Pereira Da Costa  ---
Git commit 84d19e95e3e7b6df64ea4aa2f44a8a8f6a8c92ab by Hugo Pereira Da Costa.
Committed on 21/03/2016 at 14:36.
Pushed by hpereiradacosta into branch 'Plasma/5.6'.

For buttons that have neither icon nor text, assume custom button and use
provided
contentsSize for sizeForContents calculation.

M  +36   -16   kstyle/breezestyle.cpp

http://commits.kde.org/breeze/84d19e95e3e7b6df64ea4aa2f44a8a8f6a8c92ab

-- 
You are receiving this mail because:
You are watching all bug changes.


[Breeze] [Bug 360061] Colour list squashed vertically.

2016-03-21 Thread Hugo Pereira Da Costa via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=360061

Hugo Pereira Da Costa  changed:

   What|Removed |Added

  Latest Commit||http://commits.kde.org/bree
   ||ze/69fd1e6e519484254eebfcd7
   ||76458ea49272e5e9
 Resolution|--- |FIXED
 Status|CONFIRMED   |RESOLVED

--- Comment #5 from Hugo Pereira Da Costa  ---
Git commit 69fd1e6e519484254eebfcd776458ea49272e5e9 by Hugo Pereira Da Costa.
Committed on 21/03/2016 at 14:35.
Pushed by hpereiradacosta into branch 'master'.

For buttons that have neither icon nor text, assume custom button and use
provided
contentsSize for sizeForContents calculation.

M  +36   -16   kstyle/breezestyle.cpp

http://commits.kde.org/breeze/69fd1e6e519484254eebfcd776458ea49272e5e9

-- 
You are receiving this mail because:
You are watching all bug changes.


[Breeze] [Bug 347550] Selection plus-icon is barely visible and has bad hitbox

2016-03-20 Thread Hugo Pereira Da Costa via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=347550

Hugo Pereira Da Costa  changed:

   What|Removed |Added

 CC||hugo.pere...@free.fr
   Assignee|unassigned-b...@kde.org |visual-des...@kde.org

-- 
You are receiving this mail because:
You are watching all bug changes.


[Breeze] [Bug 353369] network-connect and network-disconnect are too similar

2016-03-20 Thread Hugo Pereira Da Costa via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=353369

Hugo Pereira Da Costa  changed:

   What|Removed |Added

   Assignee|uri_herr...@nitrux.in   |visual-des...@kde.org
 CC||hugo.pere...@free.fr

-- 
You are receiving this mail because:
You are watching all bug changes.


[Breeze] [Bug 354820] KMail2 Tray icon doesn't match Breeze icon theme

2016-03-20 Thread Hugo Pereira Da Costa via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=354820

Hugo Pereira Da Costa  changed:

   What|Removed |Added

   Assignee|uri_herr...@nitrux.in   |visual-des...@kde.org
 CC||hugo.pere...@free.fr

-- 
You are receiving this mail because:
You are watching all bug changes.


[Breeze] [Bug 355697] konsole icon lacking contrast in breeze dark

2016-03-20 Thread Hugo Pereira Da Costa via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=355697

Hugo Pereira Da Costa  changed:

   What|Removed |Added

   Assignee|uri_herr...@nitrux.in   |visual-des...@kde.org

-- 
You are receiving this mail because:
You are watching all bug changes.


[Breeze] [Bug 355507] Icons should be pre-rendered to bitmaps at build-time

2016-03-20 Thread Hugo Pereira Da Costa via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=355507

Hugo Pereira Da Costa  changed:

   What|Removed |Added

 CC||hugo.pere...@free.fr
   Assignee|uri_herr...@nitrux.in   |visual-des...@kde.org

-- 
You are receiving this mail because:
You are watching all bug changes.


[Breeze] [Bug 355575] red color of player-volume-muted icon is highly confusing

2016-03-20 Thread Hugo Pereira Da Costa via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=355575

Hugo Pereira Da Costa  changed:

   What|Removed |Added

 CC||hugo.pere...@free.fr
   Assignee|uri_herr...@nitrux.in   |visual-des...@kde.org

-- 
You are receiving this mail because:
You are watching all bug changes.


[Breeze] [Bug 355697] konsole icon lacking contrast in breeze dark

2016-03-20 Thread Hugo Pereira Da Costa via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=355697

Hugo Pereira Da Costa  changed:

   What|Removed |Added

 CC||hugo.pere...@free.fr

-- 
You are receiving this mail because:
You are watching all bug changes.


[Breeze] [Bug 355872] svg icons in konqueror not displayed

2016-03-20 Thread Hugo Pereira Da Costa via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=355872

Hugo Pereira Da Costa  changed:

   What|Removed |Added

 CC||hugo.pere...@free.fr
   Assignee|uri_herr...@nitrux.in   |visual-des...@kde.org

-- 
You are receiving this mail because:
You are watching all bug changes.


[Breeze] [Bug 356544] ktorrent icon

2016-03-20 Thread Hugo Pereira Da Costa via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=356544

Hugo Pereira Da Costa  changed:

   What|Removed |Added

   Assignee|uri_herr...@nitrux.in   |visual-des...@kde.org

-- 
You are receiving this mail because:
You are watching all bug changes.


[Breeze] [Bug 356544] ktorrent icon

2016-03-20 Thread Hugo Pereira Da Costa via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=356544

Hugo Pereira Da Costa  changed:

   What|Removed |Added

 CC||hugo.pere...@free.fr

-- 
You are receiving this mail because:
You are watching all bug changes.


[Breeze] [Bug 356082] Icon for save session is missing

2016-03-20 Thread Hugo Pereira Da Costa via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=356082

Hugo Pereira Da Costa  changed:

   What|Removed |Added

 CC||hugo.pere...@free.fr
   Assignee|uri_herr...@nitrux.in   |visual-des...@kde.org

-- 
You are receiving this mail because:
You are watching all bug changes.


[Breeze] [Bug 357334] Missing icon for status/mail-*

2016-03-20 Thread Hugo Pereira Da Costa via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=357334

Hugo Pereira Da Costa  changed:

   What|Removed |Added

 CC||hugo.pere...@free.fr
   Assignee|uri_herr...@nitrux.in   |visual-des...@kde.org

-- 
You are receiving this mail because:
You are watching all bug changes.


[Breeze] [Bug 357726] volume indicator and wifi quality are too hard to read

2016-03-20 Thread Hugo Pereira Da Costa via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=357726

Hugo Pereira Da Costa  changed:

   What|Removed |Added

   Assignee|uri_herr...@nitrux.in   |visual-des...@kde.org
 CC||hugo.pere...@free.fr

-- 
You are receiving this mail because:
You are watching all bug changes.


[Breeze] [Bug 358925] Icon for mount error is missleading

2016-03-20 Thread Hugo Pereira Da Costa via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=358925

Hugo Pereira Da Costa  changed:

   What|Removed |Added

 CC||hugo.pere...@free.fr
   Assignee|n...@oxygen-icons.org   |visual-des...@kde.org

-- 
You are receiving this mail because:
You are watching all bug changes.


[Breeze] [Bug 359863] Icons "krusader_user" and "krusader_root" look too similar.

2016-03-20 Thread Hugo Pereira Da Costa via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=359863

Hugo Pereira Da Costa  changed:

   What|Removed |Added

 CC||hugo.pere...@free.fr
   Assignee|uri_herr...@nitrux.in   |visual-des...@kde.org

-- 
You are receiving this mail because:
You are watching all bug changes.


[Breeze] [Bug 359913] Firefox extension XPI files have no icon

2016-03-20 Thread Hugo Pereira Da Costa via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=359913

Hugo Pereira Da Costa  changed:

   What|Removed |Added

   Assignee|uri_herr...@nitrux.in   |visual-des...@kde.org

-- 
You are receiving this mail because:
You are watching all bug changes.


[Breeze] [Bug 360593] Dead-lock with --help on Qt 5.6

2016-03-19 Thread Hugo Pereira Da Costa via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=360593

--- Comment #5 from Hugo Pereira Da Costa  ---
ok. So definitly related to the issue reported upstream. 
What should we do ? Close as upstream and wait for the patch to go in ? 
Find a workaround ? (no clue how)
Comment out the code ? (in breeze and oxygen) ? 

The debus calls are necessary so that the widget style is updated "on fly" when
the configuration is changes via system settings (or breeze-settings, or
oxygen-settings)

On one hand Locked application is somewhat a big deal, but on the other hand
this only affects corner cases (--help) 

wdyt ?

-- 
You are receiving this mail because:
You are watching all bug changes.


[Breeze] [Bug 360593] Dead-lock with --help on Qt 5.6

2016-03-16 Thread Hugo Pereira Da Costa via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=360593

Hugo Pereira Da Costa  changed:

   What|Removed |Added

 CC||hugo.pere...@free.fr

--- Comment #2 from Hugo Pereira Da Costa  ---
@Heiko
Interesting. Indeed breeze does connect to some dbus signals in its
constructor.
@Martin,
rather than patching Qt, maybe you can first try the following patch to breeze
? (which just comments the dbus connections) ? 
I cannot test here: do not have recent enough Qt5 (I have 5.5.1)

-- 
You are receiving this mail because:
You are watching all bug changes.


[Breeze] [Bug 360593] Dead-lock with --help on Qt 5.6

2016-03-16 Thread Hugo Pereira Da Costa via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=360593

--- Comment #3 from Hugo Pereira Da Costa  ---
Created attachment 97922
  --> https://bugs.kde.org/attachment.cgi?id=97922&action=edit
test patch (not to be committed)

-- 
You are receiving this mail because:
You are watching all bug changes.


[Breeze] [Bug 353246] OpenLP tool bars are incorrectly styles with Breeze

2016-03-15 Thread Hugo Pereira Da Costa via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=353246

Hugo Pereira Da Costa  changed:

   What|Removed |Added

 Status|RESOLVED|REOPENED
 Ever confirmed|0   |1
 Resolution|WAITINGFORINFO  |---

--- Comment #6 from Hugo Pereira Da Costa  ---
Hi Tim,
Thanks for the update. Indeed I completely missed the issue with the toolbox
text being removed.
That is definitely a bug in breeze.
I'll investigate.

-- 
You are receiving this mail because:
You are watching all bug changes.


[Breeze] [Bug 353246] OpenLP tool bars are incorrectly styles with Breeze

2016-03-14 Thread Hugo Pereira Da Costa via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=353246

Hugo Pereira Da Costa  changed:

   What|Removed |Added

 Resolution|--- |WAITINGFORINFO
 Status|UNCONFIRMED |RESOLVED

-- 
You are receiving this mail because:
You are watching all bug changes.


[Breeze] [Bug 360151] Frequent segfault when a PyQt application exits.

2016-03-14 Thread Hugo Pereira Da Costa via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=360151

Hugo Pereira Da Costa  changed:

   What|Removed |Added

 Status|UNCONFIRMED |RESOLVED
 Resolution|--- |DUPLICATE

--- Comment #6 from Hugo Pereira Da Costa  ---


*** This bug has been marked as a duplicate of bug 356940 ***

-- 
You are receiving this mail because:
You are watching all bug changes.


[Breeze] [Bug 356940] kalarm crash on start up

2016-03-14 Thread Hugo Pereira Da Costa via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=356940

Hugo Pereira Da Costa  changed:

   What|Removed |Added

 CC||anntzer@gmail.com

--- Comment #1 from Hugo Pereira Da Costa  ---
*** Bug 360151 has been marked as a duplicate of this bug. ***

-- 
You are receiving this mail because:
You are watching all bug changes.



[Breeze] [Bug 360061] Colour list squashed vertically.

2016-03-11 Thread Hugo Pereira Da Costa via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=360061

Hugo Pereira Da Costa  changed:

   What|Removed |Added

 CC||hugo.pere...@free.fr
 Status|UNCONFIRMED |CONFIRMED
 Ever confirmed|0   |1

--- Comment #4 from Hugo Pereira Da Costa  ---
... will investigate

-- 
You are receiving this mail because:
You are watching all bug changes.


[Breeze] [Bug 360237] Breeze draws window frame in Window:BackgroundNormal for both active and inactive windows

2016-03-07 Thread Hugo Pereira Da Costa via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=360237

Hugo Pereira Da Costa  changed:

   What|Removed |Added

 Resolution|--- |WONTFIX
 CC||hugo.pere...@free.fr
 Status|UNCONFIRMED |RESOLVED

--- Comment #2 from Hugo Pereira Da Costa  ---
Hi,
Thanks for reporting. 

The fact that the decoration borders blends into the main window and that only
the titlebar has the decoration active/inactive color has been a design choice
for breeze since the very beginning. See for instance how the outlining "blue"
line, at the top, for active windows, would not make any sense if we were
coloring all sides. 
It is not clear to me what is "unintuitive" about that.
For Oxygen, the original design choice was even stronger that the full
decoration should blend into the window, for both color and window gradient, on
the premise that the separation between window content and window decoration is
artificial, and that the WM colors would be ignored all-together. 
Since this created lots of complain, an option was introduced (and recently
re-added for the KF5 version) to oxygen to use the active WM color for the
decoration. For the comming plasma release you just need to uncheck "Use the
same color for titlebar and window content" in the oxygen decoration config
dialog.

Bottomline:
- for breeze the current situation wont change, sorry
- for oxygen, there is an option that allows you to achieve what you want, I
think
- the third window decoration that is shipped with plasma by default
(plastique) does what you want out of the box I think

best, 

Hugo

-- 
You are receiving this mail because:
You are watching all bug changes.


  1   2   3   >