Túlio Ornelas
Túlio Ornelas
This is related to issue #93 and I totally agree that this should be improved. Thanks!
PR fixing this #118 (not merged yet)
It is possible to configure if the links should be opened in a new tab or in place (`openLinksInNewWindow`), maybe this is enough, what do you think?
That's a good idea
Hi @alahosky, I like this feature as well, bear with me :) Right now I'm trying to fix all bugs and corner cases but I keep improving the extension and...
Nice, thanks. I ran your test in development and JSON Viewer is failing here: https://github.com/tulios/json-viewer/blob/master/extension/src/json-viewer/check-if-json.js#L10 I'll try to take a look at this today. Thanks again
I think this is the same case of issue #137
The code can be improved to ignore right clicks, thanks!
Agreed. The original check was much simpler but after a while some people reported some problems and I switched to this https://github.com/tulios/json-viewer/blob/master/extension/src/json-viewer/check-if-json.js#L61 I can try to break down the execution...
@zwacky @matthewoates sure, I think this should be fixed. I'll try to work on this issue ASAP. Thanks for the input!