refined-hacker-news icon indicating copy to clipboard operation
refined-hacker-news copied to clipboard

Feature request: configurable keybindings

Open infokiller opened this issue 6 years ago • 6 comments
trafficstars

This can be done similarly to https://github.com/infokiller/web-search-navigator by adding an options page and storing the options in browser.sync.

Happy to send a PR if you are open to merging.

Related HN comment: https://news.ycombinator.com/item?id=20189188

Thanks!

infokiller avatar Jun 15 '19 09:06 infokiller

Hi! This seems like a good idea :)

By adding a separate options page that is different from the popup page, we will have two places where one sets the extension settings, which doesn't look too good. All settings/options regarding the extension should be at one place only - either the popup or the options page.

Could you explain a bit more on how you will implement this in the code? Or better still, if you could create the PR, I can look at the code then and we can discuss about it.

Nevertheless, this is a nice enhancement which I'll be happy to look into.

Cheers!

plibither8 avatar Jun 15 '19 12:06 plibither8

Oh I wasn't aware of the pop up page, yes it definitely makes sense to add the customizations to that page. Great, will send a PR soon.

infokiller avatar Jun 16 '19 07:06 infokiller

Great!

plibither8 avatar Jun 16 '19 07:06 plibither8

Hey @infokiller, any updates regarding this?

plibither8 avatar Jun 29 '19 17:06 plibither8

Sorry, didn't get to this yet... Hope to do it soon

infokiller avatar Jul 02 '19 14:07 infokiller

Alright!

plibither8 avatar Jul 02 '19 14:07 plibither8