Re: [PATCH v2 1/2] perf hists browser: Support horizontal scrolling with '<' and '>' key
From: Namhyung Kim
Date: Sun Aug 09 2015 - 06:38:30 EST
On Sun, Aug 09, 2015 at 11:30:24AM +0200, Jiri Olsa wrote:
> On Sun, Aug 09, 2015 at 05:21:01PM +0900, Namhyung Kim wrote:
> > Currently perf TUI report browser doesn't support horizontal scrolling.
> > So if terminal width is smaller than the actual contents, there's no way
> > to see them. This patch adds support horizontal movement by '<' and '>'
> > keys.
> nice, I wonder we could also have some way to scroll
> by the column width.. it might be more eye friendly?
> would need to try first ;-)
Good suggesion. Please see below..
> I also tried it with SKIP_COLS_STEP=1, and it wasn't bad
> how about having several scroll step options? like:
> , . - SKIP_COLS_STEP=1
> < > - SKIP_COLS_STEP=10
> CTRL-< > - SKIP_COLS_STEP=columns width
I tried to use CTRL but it seems not working.
> we could also bind some of this to regular arrows
> with SHIFT or CTRL, bacause it's probably the most
> convenient binding for this
Yes, I agree with you. But I don't know how to bind the arrow keys
with SHIFT or CTRL to do the thing. So I just changed that < > to
make SKIP_COLS_STEP = column width.