D16280: Update display after incrementing history index

Maximilian Schiller noreply at phabricator.kde.org
Wed Oct 17 19:25:45 BST 2018


mschiller created this revision.
mschiller added reviewers: cfeck, kde-utils-devel.
mschiller requested review of this revision.

REVISION SUMMARY
  The history index should get incremented before changing the
  displayed value. Without this it requires pressing the history
  navigation shortcuts twice until the first change occurs.

TEST PLAN
  Fill histroy with some values.
  Press Ctrl + Z to move history up.
  
  Without this patch you need to press the hotkey twice until
  something happens.
  
  Same with other direction (Ctrl + Shift + Z).

REPOSITORY
  R353 KCalc

BRANCH
  fix/histnavigation

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

AFFECTED FILES
  kcalcdisplay.cpp

To: mschiller, cfeck, kde-utils-devel
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-utils-devel/attachments/20181017/7a4f596a/attachment.html>


More information about the Kde-utils-devel mailing list