[Konsole-devel] Review Request 112983: A new prototype feature for Konsole: Scroll Marks

[Sven Brauch]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/112983/#review40966
-----------------------------------------------------------


Just two small comments, otherwise this seems like a cool feature to me. I'm not a konsole dev though, so they'll decide whether it gets in or not.


src/SessionController.cpp
<http://git.reviewboard.kde.org/r/112983/#comment30086>

    This doesn't seem like a good icon, how about "bookmark-new"?



src/SessionController.cpp
<http://git.reviewboard.kde.org/r/112983/#comment30087>

    You have _scrollMarks be a QSet -- so either make it a vector or remove this check.


- Sven Brauch


On Sept. 28, 2013, 9 p.m., Phillip Taylor wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/112983/
> -----------------------------------------------------------
> 
> (Updated Sept. 28, 2013, 9 p.m.)
> 
> 
> Review request for Konsole.
> 
> 
> Description
> -------
> 
> I'm not a KDE developer or QT person myself, but I had an "itch to scratch" as we say. A desire to add a new feature to Konsole. The feature is called 'Scroll Marks' and in a nutshell it allows you to mark line numbers in Konsole's output and then using next and previous menu options, quickly scroll to those points.
> 
> It basically makes scrolling up and down easier because you can jump through your output history even more quickly.
> 
> I went step further and made these scroll marks automatically get created when the control character 6 is seen. This means that you can put 'echo -en \\06' in your PS1 prompt and Ctrl+Up and Ctrl+Down immediately scroll you up and down between all the commands you've executed. So when that cat or grep statement suprises you with a massive amount of output, it's just a key combination away to go up past it again. And no fear of manually scrolling too far in each direction.
> 
> Since it works using a simple control character approach, it works recursively across ssh, screen and older systems really well. If you're a dev you can put it in your warn statements so when tailing logs it's easier to navigate to where you want to. If you work with grep/cat a lot, a simple sed command can make finding stuff much quicker and simpler.
> 
> The feature works quite well and I think a lot of people would like it. I hope some of you will take the time to try it out!
> 
> I'm casually requesting its integration into Konsole depending on how others feel, but if you don't like it, I enjoyed writing it and will still probably continue to use it myself.
> 
> I would also appreciate a code review of the work so that even if you don't want this feature, I don't introduce bugs into my own forks.
> 
> 
> Diffs
> -----
> 
>   desktop/konsoleui.rc 1dd4f69 
>   desktop/sessionui.rc 67d89ae 
>   src/Emulation.h 92a0ea7 
>   src/Emulation.cpp 02ed4be 
>   src/Session.h 9e982df 
>   src/Session.cpp e27bf78 
>   src/SessionController.h 2ff7910 
>   src/SessionController.cpp 62c1d0b 
>   src/Vt102Emulation.cpp 0b6d2ed 
> 
> Diff: http://git.reviewboard.kde.org/r/112983/diff/
> 
> 
> Testing
> -------
> 
> I'm not really sure how to do this. Some pointers would be very much appreciated. The feature does work though. :-)
> 
> 
> Thanks,
> 
> Phillip Taylor
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/konsole-devel/attachments/20130928/aadacdfd/attachment.html>