Ticket #4652 (closed defect: fixed)

Opened 9 hours ago

Last modified 7 hours ago

mcedit non-fullscreen ctrl-pgdown jumps too far

Reported by: dextarr Owned by: andrew_b
Priority: major Milestone: 4.8.34
Component: mcedit Version: master
Keywords: Cc:
Blocked By: Blocking:
Branch state: merged Votes for changeset: committed-master

Description (last modified by zaytsev) (diff)

Dear support,

i've bumped into an issue using ctrl-pgdown in _non-fullscreen_ editor mode, it jumps 2 more lines forward than expected. tested on debian-sid-docker (mc amd64 3:4.8.33-1)

reproduce:

  • open a larger file with F4 (large enough to press pageup/down freely without reaching EOF right away)
  • (optional) press alt-n to turn on line numbering
  • (optional) hit ctrl-down couple of times so the first line will be e.g. line #4
  • (optional) verify the normal behavior by pressing ctrl-pgup/down, it always jumps to the first and last visible line in the window as expected
  • click on "*" in the upper right corner
  • now pressing ctrl-pgdown results jumping 2 lines further in the file than the last line in the window (or even 2 more further by pressing it repeatedly).
  • ctrl-pgup works like in fullscreen mode, no anomalies with that

please consider fixing it, thanks!

Change History

comment:1 Changed 8 hours ago by zaytsev

  • Description modified (diff)

comment:2 Changed 8 hours ago by zaytsev

  • Milestone changed from Future Releases to 4.8.34

comment:3 Changed 7 hours ago by andrew_b

  • Status changed from new to accepted
  • Owner set to andrew_b
  • Branch state changed from no branch to on review

Branch: 4652_mcedit_jump_to_bottom_of_screen
changeset:b87bfb85fb68d15c3a7b7d8e7f68b9ff95e08ff8

comment:4 Changed 7 hours ago by zaytsev

  • Votes for changeset set to zaytsev
  • Branch state changed from on review to approved

comment:5 Changed 7 hours ago by andrew_b

  • Status changed from accepted to testing
  • Votes for changeset changed from zaytsev to committed-master
  • Resolution set to fixed
  • Branch state changed from approved to merged

comment:6 Changed 7 hours ago by andrew_b

  • Status changed from testing to closed
Note: See TracTickets for help on using tickets.