D17730: [WIP] Review KateStatusBar (based on D17729)

[loh tar]

loh.tar added a comment.


  > For the "should we use buttons": I think I tried that in the past and it did look horrible in many styles, but perhaps I am wrong ;)
  
  I see. May it in the meantime not an issue anymore, due to new designs elsewhere? And what's up with the existing buttons, they should look also ugly there.
  
  > I am a bit confused which questions still need answers ;=)
  
  
  
  - Most of them have an answer now ;-) Will add more stuff here
  - Your comment to @zetazeta request ? I have tried to add a QScrollArea but without success :-/ Ideas? Perhaps is a second level of layout/widget needed(?)
  
         updateStatus();
         wordCountChanged(0, 0, 0, 0);
    +
    +    QScrollArea *scrollArea = new QScrollArea; // No status bar
    +//     QScrollArea *scrollArea = new QScrollArea(this); // No status bar
    +//     QScrollArea *scrollArea = new QScrollArea(centralWidget());// No Kate window appears!
    +    scrollArea->setWidget(centralWidget());
     }
  
  
  
  > I think having the dictionary picker per default there takes to much space
  
  Why do you think so? My status bar is almost empty, but I did not show "word count" and such. The extra space is hardly worth mentioning. Add an new pic with my typical window size. The "German version" need some more space but is also hardly worth mentioning.
  F6511683: 1545915793.png <https://phabricator.kde.org/F6511683>
  
  > ..and one should have some "show XXX" in the context menu for it.
  
  Doing it like the other stuff need some effort, like to change the config dialog :-/ *hmpf*

REPOSITORY
  R39 KTextEditor

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

To: loh.tar, #ktexteditor
Cc: zetazeta, cullmann, kwrite-devel, kde-frameworks-devel, #ktexteditor, hase, michaelh, ngraham, bruns, demsking, sars, dhaumann
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kwrite-devel/attachments/20181227/8403ad12/attachment.html>