hpereiradacosta added a comment.
Hello, Hi, thanks for the patch ! Few comments: 1/ the code does not compile under kde4 (cmake -DUSE_KDE4). You would need to use the proper #ifdef to prevent this 2/ I think the "blur" option should go. Blur should be controlled centrally by the desktop effect. In other words: BlurBehind should always be set to true, and then left to kwin to handle. Having an extra option here seems like micro-management. Why would you need blur behind plasma widgets (as handled by the desktop effect) and not behind menus ? (or vice versa). Also, right now, selecting "blur" in the style settings, but unselecting the desktop effect results in no blur behind menu, and hence inconsistency with what the style option says. 3/ the slider should follow the same convention and design as for the translucency effect: "transparent <----slider---> opaque". 0% should be transparent, 100% should be opaque. Also, should have the same granularity, so ... percents. (not 10%) 4/ I would really like the VDG opinion on this. IMHO I am not sure if it fits the breeze "guidelines" or design-ideas, in the sense that I do not know why this is needed (what is the use case), aside from "this is pretty". In other words: why would one need to be able to see what is behind the menu you are currently navigating ? Navigating a menu is a very focused thing one wants to do: one is looking for a specific action to perform on the current content of the window. To me at least, seeing-through the menu is more a distraction to this task than anything else. (and adding blur to reduce this distraction would in fact go in the direction of not being translucent at all). If the only reason is "this is pretty", then this has to be balanced with the extra option, code complexity, and configurations to be tested on the maintainer side, that this patch adds. REPOSITORY R31 Breeze REVISION DETAIL https://phabricator.kde.org/D10170 To: anemeth, hpereiradacosta, #plasma, #vdg Cc: zzag, ngraham, plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
