New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Capture links in dynamic content #1337
Capture links in dynamic content #1337
Conversation
🚀 Thanks for the pull request! Here are the current build files for testing: Download and unzip the file for your browser. Refer to the readme for detailed install instructions. This build is brought to you by:
Want to sponsor the next build? send some sats to ⚡️builds@getalby.com (don't forget to provide your name) Don't forget: keep stacking sats! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this is mainly the composedPath, correct?
src/extension/inpage-script/index.js
Outdated
(ev) => { | ||
// Use composedPath() for detecting links inside a Shadow DOM | ||
// https://developer.mozilla.org/en-US/docs/Web/API/Event/composedPath | ||
const target = ev.composedPath()[0] || ev.target; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is ev.composedPath()
always returning an array?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I currently can't think of a case where composedPath is null. I'll test and remove the old |
063cdd1
to
a608656
Compare
Describe the changes you have made in this PR
Capture ALL
lightning:
links.Link this PR to an issue
#1307
Type of change (Remove other not matching type)
fix
: Bug fix (non-breaking change which fixes an issue)Checklist