Skip to content

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

Merged
jsamr merged 1 commit into
native-html:masterfrom
kioopi:master
Jan 9, 2023
Merged

feat: adds dataAttributes to linkPressTargetToOnDOMLinkPressArgs in plugins-core#45
jsamr merged 1 commit into
native-html:masterfrom
kioopi:master

Conversation

@kioopi

@kioopi kioopi commented Jan 6, 2023

Copy link
Copy Markdown
Contributor

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

codecov Bot commented Jan 9, 2023

Copy link
Copy Markdown

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

jsamr commented Jan 9, 2023

Copy link
Copy Markdown
Collaborator

Thanks for this!

@DanielKuhn

Copy link
Copy Markdown

@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