Skip to content

Conversation

@avently
Copy link

@avently avently commented Feb 13, 2018

This is a fix for #29

@sum01
Copy link
Collaborator

sum01 commented Feb 13, 2018

Shift-Up going to the .. is good, but I'm not sure how I feel about left arrow going to parent.

I guess if it were wrapped in an option that could be disabled I wouldn't mind (so true by default would be fine). Something like filemanager-leftparent or whatever.

@avently
Copy link
Author

avently commented Feb 14, 2018

I added such option. What is a usecase for disabling "go to parent from file" functionality? Why someone may want it?

@sum01
Copy link
Collaborator

sum01 commented Feb 14, 2018

Usecase is since it's making one action do two things based on context. I know I'll personally turn it off since I don't want to go to parent if I accidentally press left twice or something like that. But that's just me.

I'll merge soon.

@sum01 sum01 merged commit 32dfb88 into NicolaiSoeborg:master Feb 15, 2018
sum01 added a commit that referenced this pull request Feb 15, 2018
sum01 added a commit that referenced this pull request Feb 15, 2018
Also some minor optimizations, and a line-break on the README.

We don't need to save the GetOption call.
The compress_target function can also be canceled early to prevent an unnecessary refresh.
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