Differential D7692

Set text and border color from palette for syndication preview
ClosedPublic
Actions

Authored by thomasp on Sep 4 2017, 7:58 PM.

Details

Reviewers
stikonas
Summary

When using the breeze dark workspace theme the text in the syndication webView is black on dark background.
With this patch the palettes colour is used for text and border.

Test Plan

Visual inspection.

Diff Detail

Lint
Lint Skipped
Unit
Unit Tests Skipped
thomasp created this revision.Sep 4 2017, 7:58 PM
stikonas accepted this revision.Sep 5 2017, 12:54 PM
This revision is now accepted and ready to land.Sep 5 2017, 12:54 PM
stikonas added a comment.Sep 5 2017, 12:56 PM

Should I commit this for you? Or do you have commit access?

thomasp added a comment.Sep 5 2017, 12:57 PM

Please commit.
Thank you!

stikonas added a comment.Sep 5 2017, 1:20 PM
In D7692#143219, @thomasp wrote:

Please commit.
Thank you!

I'll need your email for git commit. Could you send it to me

stikonas closed this revision.EditedSep 5 2017, 2:19 PM

Thanks, committed. Although I messed up REVIEW and DIFFERENTIAL_REVISION keywords, so this didn't get closed automatically...

https://cgit.kde.org/ktorrent.git/commit/?id=bd3aae954b67118e7edd1fc6afbe70e6d24ed70f

Revision Contents
Changeset List

PathPackages
Mplugins/syndication/feedwidget.cpp (11 lines)

Diff 19188

View Options

plugins/syndication/feedwidget.cpp

Loading...