D5102: Add scroll-barmarks also to the built-in search

2017-04-09 Thread Kåre Särs
sars closed this revision. sars added a comment. https://commits.kde.org/ktexteditor/1bd9d474763ca891ef67c736398289cfd61bf21f REPOSITORY R39 KTextEditor REVISION DETAIL https://phabricator.kde.org/D5102 To: sars, #kate, #ktexteditor, dhaumann Cc: mwolff, dhaumann, kwrite-devel,

D5102: Add scroll-barmarks also to the built-in search

2017-03-23 Thread Milian Wolff
mwolff added a comment. Yes, please reserve it. REPOSITORY R39 KTextEditor REVISION DETAIL https://phabricator.kde.org/D5102 To: sars, #kate, #ktexteditor, dhaumann Cc: mwolff, dhaumann, kwrite-devel, #frameworks

D5102: Add scroll-barmarks also to the built-in search

2017-03-21 Thread Kåre Särs
sars added a comment. I searched through the kdevelop repo and found marks 8, 21-27 in use + the "reserved" for Bookmark,... So for KDevelop it looks OK, but we have other users also. Should I add "SearchMatch = markType32" to the MarkTypes enum to reserve it? REPOSITORY R39

D5102: Add scroll-barmarks also to the built-in search

2017-03-18 Thread Dominik Haumann
dhaumann accepted this revision. dhaumann added a comment. This revision is now accepted and ready to land. If markType32 does not clash with others (e.g. markTypes that are used by KDevelop), I am fine with this. Could you check this before comitting? REPOSITORY R39 KTextEditor REVISION

D5102: Add scroll-barmarks also to the built-in search

2017-03-18 Thread Kåre Särs
sars created this revision. Restricted Application added subscribers: Frameworks, kwrite-devel. Restricted Application added a project: Frameworks. REVISION SUMMARY This patch adds scroll-bar-marks to the matches of "Find all" in the advanced search bar TEST PLAN find all results in