Skip to content

Moving using up/down keys - #94

Closed
Pac72 wants to merge 46 commits into
nickbnf:masterfrom
Pac72:master
Closed

Moving using up/down keys #94
Pac72 wants to merge 46 commits into
nickbnf:masterfrom
Pac72:master

Conversation

@Pac72

@Pac72 Pac72 commented Sep 4, 2015

Copy link
Copy Markdown

These changes:

  • allow moving in AbstractLogView subclasses using key up and key down
  • keep log file in sync with the search pane even when moving with the keyboard

@Pac72

Pac72 commented Dec 13, 2015

Copy link
Copy Markdown
Author

Hi @nickbnf, any comment on this pull request?

@variar

variar commented Dec 3, 2016

Copy link
Copy Markdown
Contributor

As it is now this pull request is easier to apply to my branch. I suggest adapting changes to glogg/master branch, because it seems that a lot of proposed chages are irrelevant to the subject (moving with up/down keys).

@Pac72

Pac72 commented Dec 4, 2016

Copy link
Copy Markdown
Author

Could it be because the pull request is exactly 15 month old? ;-)
I'll have a look at it.

@Pac72

Pac72 commented Dec 4, 2016

Copy link
Copy Markdown
Author

@variar, I'm closing this one in favor of the new pull request #134, which applies cleanly on upstream/master

@Pac72 Pac72 closed this Dec 4, 2016
gin-ahirsch pushed a commit to gin-ahirsch/glogg that referenced this pull request Sep 23, 2019
gin-ahirsch pushed a commit to gin-ahirsch/glogg that referenced this pull request Sep 23, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants