Review Request 109340: Use default background colours for the konsolepreviewer

Tianyu Zhu tian.tian098 at gmail.com
Mon Mar 11 13:31:49 UTC 2013


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/109340/
-----------------------------------------------------------

(Updated March 11, 2013, 1:31 p.m.)


Review request for Plasma.


Changes
-------

Use QTextEdit instead of KTextEdit, since its background colour makes text more visible on light and dark themes when read-only is enabled.


Description
-------

- Deleted lines hard-coding the background of the konsolepreviwer to white.


This addresses bug https://bugs.kde.org/show_bug.cgi?id=316274.
    http://bugs.kde.org/show_bug.cgi?id=https://bugs.kde.org/show_bug.cgi?id=316274


Diffs (updated)
-----

  plasmate/konsole/konsolepreviewer.h 543ff77 
  plasmate/konsole/konsolepreviewer.cpp 05d426e 

Diff: http://git.reviewboard.kde.org/r/109340/diff/


Testing
-------

Tested on Kubuntu 12.10 with KDE 4.10.1
Tested with both the default and Krita-Dark colour scheme. In both cases, the text in the konsolepreviewer shows up well against its background.


Thanks,

Tianyu Zhu

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20130311/f3d66716/attachment.html>


More information about the Plasma-devel mailing list