! Please note that this is a snapshot of our old Bugzilla server, which is read only since May 29, 2020. Please go to gitlab.xfce.org for our new server !
Previous button on search bar moves the cursor one character backwards if the...
Status:
RESOLVED: FIXED
Product:
Mousepad
Component:
General

Comments

Description Alistair Buxton 2018-12-04 02:01:23 CET
To reproduce:

1. Open mousepad
2. Type "hello"
3. Press ctrl-f
4. Search for 'z'
5. Click 'previous'

Expected result: text cursor does not move since 'z' does not exist in the document.

Actual result: text cursor moves backwards through the document by one character each time you press "previous".
Comment 1 Theo Linkspfeifer editbugs 2019-04-26 15:30:35 CEST
This is a result of the following commit:
https://git.xfce.org/apps/mousepad/commit/?id=0061a2168f820035c278f883bd056a2cee57ea4a
Comment 2 Theo Linkspfeifer editbugs 2019-04-26 22:27:14 CEST
Created attachment 8459 
proposed patch
Comment 3 Git Bot editbugs 2019-04-30 02:48:13 CEST
Theo Linkspfeifer referenced this bugreport in commit 905686ee95cdb110368c7d94328e52fc50b6f613

Do not move text cursor when searching backwards (Bug #14942)

https://git.xfce.org/apps/mousepad/commit?id=905686ee95cdb110368c7d94328e52fc50b6f613
Comment 4 Andre Miranda editbugs 2019-04-30 02:49:32 CEST
Looks good to me, I didn't notice any regression wrt Bug #10999.

Bug #14942

Reported by:
Alistair Buxton
Reported on: 2018-12-04
Last modified on: 2019-04-30

People

Assignee:
Matthew Brush
CC List:
3 users

Version

Target Milestone:
Mousepad 0.4.x

Attachments

proposed patch (1022 bytes, patch)
2019-04-26 22:27 CEST , Theo Linkspfeifer
no flags

Additional information