Thanks to visit codestin.com
Credit goes to github.com

Skip to content

feat: adds dataAttributes to linkPressTargetToOnDOMLinkPressArgs in plugins-core #45

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

Merged
merged 1 commit into from
Jan 9, 2023

Conversation

kioopi
Copy link
Contributor

@kioopi kioopi commented Jan 6, 2023

formidable-webview/webshell has added dataAttributes to LinkPressTarget in September 2021. [1]

This commit adds it to linkPressTargetToOnDOMLinkPressArgs so that data attributes on anchor tags are available in onPress-handlers.

[1] formidable-webview/webshell@0a2c3e0

…lugins-core

formidable-webview/webshell has added dataAttributes to LinkPressTarget
in September 2021. [1]

This commit adds it to linkPressTargetToOnDOMLinkPressArgs so that data
attributes on achor tags are available in onPress-handlers.

[1] formidable-webview/webshell@0a2c3e0
@codecov
Copy link

codecov bot commented Jan 9, 2023

Codecov Report

Merging #45 (140b6fb) into master (bbbb9b6) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #45   +/-   ##
=======================================
  Coverage   91.44%   91.44%           
=======================================
  Files          14       14           
  Lines         304      304           
  Branches       95       95           
=======================================
  Hits          278      278           
  Misses         26       26           
Flag Coverage Δ
heuristic-table-plugin 95.45% <ø> (ø)
iframe-plugin 87.32% <ø> (ø)
plugins-core 100.00% <ø> (ø)
table-plugin 88.29% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
...ns-core/src/linkPressTargetToOnDOMLinkPressArgs.ts 100.00% <ø> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@jsamr jsamr merged commit 26870f7 into native-html:master Jan 9, 2023
@jsamr
Copy link
Collaborator

jsamr commented Jan 9, 2023

Thanks for this!

@DanielKuhn
Copy link

@jsamr Are you planning to release this?
It's merged in master but the version hasn't been bumped so it doesn't make it into the bundle currently (since @native-html/plugins-core is still at 1.3.0):

"@native-html/[email protected]":
  version "1.3.0"
  resolved "https://registry.yarnpkg.com/@native-html/plugins-core/-/plugins-core-1.3.0.tgz#f1f24622097551930d9dab0214c4929d00f7446e"
  integrity sha512-vce35gqGJKa2oPDZVa2sKjucFFVK+3g8quLayeXiJtj5LzuS8TWGrBFTS5O4ToUtE02AJkCDOLwAguCzK2HWdQ==

"@native-html/[email protected]":
  version "5.3.1"
  resolved "https://registry.yarnpkg.com/@native-html/table-plugin/-/table-plugin-5.3.1.tgz#12d34869c2ff3e5f45de6ff6a8b5aba1471a8a03"
  integrity sha512-zz1ww7tUgIY6fMnKYejKHacaUBV+yjuWDBWDwSp4SP0CFiLAbmLIER0UGw0kI58lAVgbWzdEJUX0dmeBG0qevA==
  dependencies:
    "@formidable-webview/webshell" "2.6.0"
    "@native-html/plugins-core" "1.3.0"
    "@types/prop-types" "^15.7.4"
    prop-types "^15.7.2"

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.

3 participants