1. 21 Dec, 2019 2 commits
    • Noah Davis's avatar
      Make arrows pixel perfect · bda1b374
      Noah Davis authored
    • Noah Davis's avatar
      Revert "Revert "Reduce the indicator arrow size for press-and-hold menus in QToolButtons"" · 288c3b85
      Noah Davis authored
      This reverts commit c9aa5358.
      Also makes the small down arrow pixel perfect. I will make the other arrows pixel perfect in a separate patch.
      After D19890 (27bcd1be) was reopened, there was some discussion about the reason it was reverted and the problems it introduced. It is possible that an icon could partially cover this arrow, but it seemed uncommon enough that ignoring the problem was deemed acceptable. The type of control that this arrow is used for is also pretty uncommon because it does not feel very good to use with a mouse or touchpad.
      Reviewers: #vdg, #breeze, #plasma, hpereiradacosta, ngraham
      Reviewed By: #vdg, #breeze, ngraham
      Subscribers: plasma-devel
      Tags: #plasma
      Differential Revision: https://phabricator.kde.org/D26139
  2. 19 Dec, 2019 1 commit
    • Noah Davis's avatar
      Add shadow rendering helper functions · 4a78561a
      Noah Davis authored
      This patch makes other helper functions reuse more code by calling these shadow rendering functions instead of defining their own shadows.
      The shadows are perfectly shaped, tested up to 8x scaling.
      Without this patch, the radius of the shadows for checkboxes is slighly too wide and there is a gap between the inside edge of the radio button shadow and the outside edge of the radio button.
      Shadow rendering performance has also been improved a bit.
      Test Plan:
      run `QT_SCALE_FACTOR=8 oxygen-demo5` and examine the shadows. You can try higher scaling levels, but that's overkill.
      8x scaling
      before: {F7824689, size=full} {F7824691, size=full}
      after: {F7824692, size=full} {F7824693, size=full}
      Reviewers: #breeze, hpereiradacosta, #plasma
      Reviewed By: hpereiradacosta
      Subscribers: ngraham, plasma-devel
      Tags: #plasma
      Differential Revision: https://phabricator.kde.org/D26094
  3. 18 Dec, 2019 1 commit
  4. 14 Dec, 2019 2 commits
    • Noah Davis's avatar
    • Noah Davis's avatar
      Fix rubberband selection outline position · 80650e0e
      Noah Davis authored
      Before this patch, the outline of the rubberband selection control was down and to the right half a pixel too far.
      This wasn't visible at 1x scaling because of the lack of antialiasing, but resulted in the top/left being too thin and bottom/right being too thick on high DPI displays.
      To fix the issue, I enabled antialiasing and moved all sides inwards by half a pixel so that the outline would be pixel perfect.
      I also added a function to get a QRectF with the correct size and position for rectangles with pen strokes.
      Test Plan:
      - Open dolphin and make a selection with the mouse
      - Go to a Folder View Plasma widget and make a selection with the mouse
      2x scaling
      before: {F7818498, size=full}
      after: {F7818500, size=full}
      3x scaling
      before: {F7818484}
      after: {F7818483}
      Reviewers: #vdg, #breeze, #plasma, hpereiradacosta, broulik
      Reviewed By: hpereiradacosta
      Subscribers: broulik, plasma-devel
      Tags: #plasma, #breeze, #vdg
      Differential Revision: https://phabricator.kde.org/D26001
  5. 10 Dec, 2019 2 commits
  6. 07 Dec, 2019 1 commit
  7. 06 Dec, 2019 1 commit
  8. 03 Dec, 2019 1 commit
  9. 29 Nov, 2019 1 commit
  10. 12 Nov, 2019 1 commit
  11. 08 Nov, 2019 1 commit
  12. 04 Nov, 2019 1 commit
    • David Edmundson's avatar
      Add KDEClangFormat cmake support · 4a6da365
      David Edmundson authored
      This allows devs to run: "make clang-format" and format all files easily
      using the preset KDE clang format style
      This patch adds support so devs can easily test the intended formatting
      now, it doesn't change the code yet.
      Doing an initial run to commit everything will happen later.
      Test Plan:
      Ran script
      git diff was full of amazingly cleaned code
      See D25134
  13. 31 Oct, 2019 1 commit
  14. 29 Oct, 2019 1 commit
  15. 22 Oct, 2019 1 commit
  16. 19 Oct, 2019 1 commit
  17. 14 Oct, 2019 1 commit
    • Albert Astals Cid's avatar
      Micro optimizations · 993c96a0
      Albert Astals Cid authored
      Add some const & for things that return const & so there's no need to make a copy
      Add a std::move for a thing that we pass by copy and it's just saved in another variable
      Reviewers: zzag
      Reviewed By: zzag
      Subscribers: zzag, plasma-devel
      Tags: #plasma
      Differential Revision: https://phabricator.kde.org/D24614
  18. 10 Oct, 2019 3 commits
  19. 07 Oct, 2019 1 commit
  20. 01 Oct, 2019 1 commit
  21. 19 Sep, 2019 3 commits
  22. 17 Sep, 2019 1 commit
  23. 24 Aug, 2019 1 commit
  24. 17 Aug, 2019 1 commit
  25. 16 Aug, 2019 4 commits
  26. 15 Aug, 2019 1 commit
    • Noah Davis's avatar
      Fix width and separator of ToolButtonComplexControl outline w/ dropdown menu · b43e19e3
      Noah Davis authored
      Summary: Apparently, the code to adjust the rectangle when a menu is present is what was causing the problem. Also added an if statement to move the separator with the button when sunken.
      Test Plan:
      {F7248297, size=full}
      {F7248298, size=full}
      {F7248300, size=full}
      {F7248301, size=full}
      Reviewers: #vdg, #breeze, ngraham
      Reviewed By: #vdg, #breeze, ngraham
      Subscribers: ngraham, plasma-devel
      Tags: #plasma
      Differential Revision: https://phabricator.kde.org/D23169
  27. 12 Aug, 2019 1 commit
  28. 06 Aug, 2019 3 commits