Smattr / clink

a modern re-implementation of Cscope
The Unlicense
43 stars 2 forks source link

shortcuts can access off screen results #127

Closed Smattr closed 1 year ago

Smattr commented 1 year ago

When the number of results returned do not fit on screen, Clink pages them (correct). What is unexpected though, is that shortcuts for options that lie beyond the current page will still work. E.g. if the last result on the page ends at E, it's still possible to use F to open the first result of the next page.

I guess this is kind of handy, but also sort of useless. We should probably make the hot keys do nothing in this situation.

Smattr commented 1 year ago

This was fixed in cc6b1c68290523f998be90b5f6d9925d0f36b5b9.