D21803: PanelView: align setting of masks with how it's done for dialogs/tooltips

Friedrich W. H. Kossebau noreply at phabricator.kde.org
Fri Jun 14 16:26:43 BST 2019


kossebau created this revision.
kossebau added a reviewer: Plasma.
Herald added a project: Plasma.
Herald added a subscriber: plasma-devel.
kossebau requested review of this revision.

REVISION SUMMARY
  With Plasma dialogs & tooltips the mask is only set on the widget itself
  if compositing is off. Instead is the mask explicitly passed for the
  KWindowEffects.
  The recent changes to fix the blurbehind & contrast mask with panels
  instead turned to always set the mask on the widget. For some reason
  yet to be understood this can sometimes result in an offset by a pixel
  between the mask used for painting and the mask used for the window effects.
  
  Aligning the mask setting code with the one for dialogs & tooltips makes
  the symptom go away for now, and also delivers consistency.
  
  CCBBUG: 406380

TEST PLAN
  Using different themes, including Adapta & Arc Dark, a gap can no longer
  been seen. Only with compositing turned off, which is old behaviour due to
  the real bug yet to understand completly.
  Resizing panels or changing screen resolution also works without breaking
  rendering or panel contents interaction.

REPOSITORY
  R120 Plasma Workspace

BRANCH
  alignpanelmaskhandlingtodialog

REVISION DETAIL
  https://phabricator.kde.org/D21803

AFFECTED FILES
  shell/panelview.cpp
  shell/panelview.h

To: kossebau, #plasma
Cc: plasma-devel, LeGast00n, jraleigh, GB_2, ragreen, Pitel, ZrenBot, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20190614/d07d4b34/attachment.html>


More information about the Plasma-devel mailing list