https://bugs.kde.org/show_bug.cgi?id=415526

Arjen Hiemstra <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
      Latest Commit|                            |https://commits.kde.org/kir
                   |                            |igami/e770b0ddd2af7fc861be4
                   |                            |5044476b4fa062ccc1d
         Resolution|---                         |FIXED
             Status|REPORTED                    |RESOLVED

--- Comment #2 from Arjen Hiemstra <[email protected]> ---
Git commit e770b0ddd2af7fc861be45044476b4fa062ccc1d by Arjen Hiemstra.
Committed on 21/04/2020 at 09:56.
Pushed by ahiemstra into branch 'master'.

Use ShadowedRectangle for Card backgrounds

Summary:
This switches cards to use ShadowedRectangle for their background. It also
refreshes their
design a bit, based on feedback from the VDG.

Still WIP because of potential design issues and I'm not entirely happy with
the code for
the actual cards. Ideally I'd also move a bunch of this stuff to C++ but that'd
involve a
reimplementing a bunch of Control behaviour.

Test Plan:
New cards, normal colour scheme:

{F8221005}

Breeze Dark:

{F8221010}

Reviewers: #kirigami, #vdg, cblack, mart

Reviewed By: #kirigami, #vdg, cblack, mart

Subscribers: mart, ngraham, nicolasfella, cblack, plasma-devel

Tags: #kirigami

Differential Revision: https://phabricator.kde.org/D28625

M  +20   -1    src/colorutils.cpp
M  +18   -0    src/colorutils.h
M  +18   -15   src/controls/AbstractCard.qml
M  +5    -1    src/controls/Card.qml
M  +10   -4    src/controls/ShadowedImage.qml
M  +15   -21   src/controls/private/BannerImage.qml
M  +11   -81   src/controls/private/DefaultCardBackground.qml
M  +1    -1    src/scenegraph/shaders/sdf.glsl
M  +1    -1    src/scenegraph/shadowedtexturenode.cpp
M  +12   -33   src/shadowedtexture.cpp

https://commits.kde.org/kirigami/e770b0ddd2af7fc861be45044476b4fa062ccc1d

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

Reply via email to