D13390: Fonts KCM: Fix text readability regression

[Marco Martin]

mart added a comment.


  In D13390#275223 <https://phabricator.kde.org/D13390#275223>, @rkflx wrote:
  
  > In D13390#275153 <https://phabricator.kde.org/D13390#275153>, @mart wrote:
  >
  > > what you should do is:
  > >  TextField {
  > >
  > >   readOnly: true
  > >   Kirigami.Theme.inherit: false   //always to be set when you are writing anything to the Theme attached proeprty
  > >   Kirigami.Theme.colorGroup: Kirigami.Theme.Window
  > >
  > > }
  > >
  > > changing the textfield colorgroup to window, will make its background to become gray, so a look somewhat less "clickable"
  >
  >
  > Now it looks as if you can edit the text, and the previews are placeholders for what you can type in:
  
  
  yeah, that's a bug i can reproduce, it's not supposed to look like that, i'll fix it
  
  in the meantime, this form is also correct but it works with current kirigami:
  
    QtControls.TextField {
         readOnly: true
         Kirigami.Theme.inherit: true

REPOSITORY
  R119 Plasma Desktop

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

To: rkflx
Cc: mart, broulik, Zren, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20180607/20453709/attachment.html>