D18164: Review KateGotoBar

loh tar noreply at phabricator.kde.org
Thu Jan 10 18:52:10 GMT 2019


loh.tar created this revision.
loh.tar added a reviewer: KTextEditor.
Herald added projects: Kate, Frameworks.
Herald added subscribers: kde-frameworks-devel, kwrite-devel.
loh.tar requested review of this revision.

REVISION SUMMARY
  - Set minimum value of spin-box to 1
  - Don't force minimum width by buttons, see similar Bug 402904
  - Rename gotoRange -> m_gotoRange
  - Add modified_line_up/down buttons with mouse wheel support
  - Change label text to be less redundant in conjunction with the "goto-button"
  - Change QLabel to button, with an action to go to line number from clipboard. My first intend was to add a clear-button to the spin box (not so easy) because paste from clipboard needs an empty field. But then I had this idea which is much more handy.

TEST PLAN
  F6539214: 1547145791.png <https://phabricator.kde.org/F6539214>

REPOSITORY
  R39 KTextEditor

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

AFFECTED FILES
  src/dialogs/katedialogs.cpp
  src/dialogs/katedialogs.h

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


More information about the KWrite-Devel mailing list