links not working correctly about lax

[srirangan]

Thank you @brandly

[srirangan]

Yep, I see this as well. I don't mind the behavior, as long as there's a back button.

But I guess the expected behavior is to open in default browser.

[brandly]

But I guess the expected behavior is to open in default browser.

Yes! I swear it used to behave like this. I don’t think I’ve touched anything over there in a while, so maybe updating Electron itself broke it. It’s probably not a difficult fix.

[srirangan]

I see you're using shell.openExternal which seems correct. Perhaps the event is bubbling up to react-linkify nevertheless.

[srirangan]

Curious if we need handleLink() at all. Does does passing target: '_blank' not behave within the Electron shell?

[brandly]

@srirangan thanks for chiming in, helped motivate me to actually do something lol

the latest version of react-linkfy doesn't seem to take a properties prop at all. instead of figuring out if there's a new approach, i just rolled back.

after rolling back, i tried the target: '_blank' approach, since i thought that'd be nice. however, that opened a second electron window with the site in it. with openExternal, it takes you to your real browser.

[brandly]

Lax v0.6.5 should be out soon. try it out and lmk if it works well for you. thanks again! ✨