Skip to content
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

feat(vue-wrappers)!: forward form values by DOM prop name (VIV-1798) #1789

Merged
merged 4 commits into from
Jun 27, 2024

Conversation

RichardHelm
Copy link
Contributor

  • Determine which props will be passed as attribute and which as DOM prop and consistently pass this way for both Vue 2 and 3.
  • Exceptions added for value attributes to be passed by their DOM prop name.
  • E.g. value prop will now be forwarded to current-value attribute.

@RichardHelm RichardHelm added the Breaking Change Changes that will go into the next major version label Jun 21, 2024
Copy link

codecov bot commented Jun 21, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 100.00%. Comparing base (90092c6) to head (0274e9f).

Current head 0274e9f differs from pull request most recent head cc0154d

Please upload reports for the commit cc0154d to get more accurate results.

Additional details and impacted files
@@               Coverage Diff                @@
##           main-4.0.0-RC1     #1789   +/-   ##
================================================
  Coverage          100.00%   100.00%           
================================================
  Files                 337       337           
  Lines                5925      5923    -2     
  Branches              748       744    -4     
================================================
- Hits                 5925      5923    -2     
Flag Coverage Δ
unittests 100.00% <100.00%> (ø)

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

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@RichardHelm RichardHelm changed the base branch from main to main-4.0.0-RC1 June 26, 2024 08:52
@RichardHelm RichardHelm force-pushed the VIV-1798-wrappers-value-props branch from 3b42576 to 2218cd5 Compare June 26, 2024 08:52
@RichardHelm RichardHelm merged commit fdfcc67 into main-4.0.0-RC1 Jun 27, 2024
11 checks passed
@RichardHelm RichardHelm deleted the VIV-1798-wrappers-value-props branch June 27, 2024 10:56
YonatanKra added a commit that referenced this pull request Jul 1, 2024
* feat(tooltip)!: sets default max width (VIV-1799) (#1798)

* chore: updates docs

---------

Co-authored-by: James Taylor <jstaylor@vonht2fg32k90.home>

* feat(data-grid-cell)!: change default value of block-size and white-space (VIV-1797) (#1793)

* Make PR CI start on any main branch

* feat(audio-player)!: remove css-variable for min-inline-size and max-inline-size on host (VIV-1800) (#1803)

* feat(vue-wrappers)!: forward form values by DOM prop name (VIV-1798) (#1789)

* Improve error messages

* feat(vue-wrappers)!: forward form values by DOM prop name (VIV-1798)

---------

Co-authored-by: James Taylor <146064280+TaylorJ76@users.noreply.github.com>

* chore(docs): v4 migration page (VIV-1801) (#1796)

* fix(tabs)!: change gutters default to small (VIV-1796) (#1804)

* chore(docs)!: refine docs for v-4 (VIV-000) (#1813)

* feat(toggletip)!: add max-inline-size of 30ch & css-variable for custom size (VIV-1846) (#1814)

---------

Co-authored-by: James Taylor <jstaylor@vonht2fg32k90.home>
Co-authored-by: Richard Helm <86777660+RichardHelm@users.noreply.github.com>
Co-authored-by: Yonatan Kra <me@yonatankra.com>
Co-authored-by: Rachel Bratt Tannenbaum <rachelbt@users.noreply.github.com>
Co-authored-by: James Taylor <146064280+TaylorJ76@users.noreply.github.com>
YonatanKra added a commit that referenced this pull request Jul 1, 2024
* feat(tooltip)!: sets default max width (VIV-1799) (#1798)

* chore: updates docs

* feat(data-grid-cell)!: change default value of block-size and white-space (VIV-1797) (#1793)

* Make PR CI start on any main branch

* feat(audio-player)!: remove css-variable for min-inline-size and max-inline-size on host (VIV-1800) (#1803)

* feat(vue-wrappers)!: forward form values by DOM prop name (VIV-1798) (#1789)

* Improve error messages

* feat(vue-wrappers)!: forward form values by DOM prop name (VIV-1798)

* chore(docs): v4 migration page (VIV-1801) (#1796)

* fix(tabs)!: change gutters default to small (VIV-1796) (#1804)

* chore(docs)!: refine docs for v-4 (VIV-000) (#1813)

* feat(toggletip)!: add max-inline-size of 30ch & css-variable for custom size (VIV-1846) (#1814)
YonatanKra added a commit that referenced this pull request Jul 1, 2024
* feat(tooltip): sets default max width (VIV-1799) (#1798)

* chore: updates docs

---------

Co-authored-by: James Taylor <jstaylor@vonht2fg32k90.home>

* feat(data-grid-cell): change default value of block-size and white-space (VIV-1797) (#1793)

* Make PR CI start on any main branch

* feat(audio-player)!: remove css-variable for min-inline-size and max-inline-size on host (VIV-1800) (#1803)

* feat(vue-wrappers)!: forward form values by DOM prop name (VIV-1798) (#1789)

* Improve error messages

* feat(vue-wrappers)!: forward form values by DOM prop name (VIV-1798)

---------

Co-authored-by: James Taylor <146064280+TaylorJ76@users.noreply.github.com>

* chore(docs): v4 migration page (VIV-1801) (#1796)

* fix(tabs)!: change gutters default to small (VIV-1796) (#1804)

* chore(docs)!: refine docs for v-4 (VIV-000) (#1813)

* feat(toggletip)!: add max-inline-size of 30ch & css-variable for custom size (VIV-1846) (#1814)

---------

Co-authored-by: James Taylor <jstaylor@vonht2fg32k90.home>
Co-authored-by: Richard Helm <86777660+RichardHelm@users.noreply.github.com>
Co-authored-by: Yonatan Kra <me@yonatankra.com>
Co-authored-by: Rachel Bratt Tannenbaum <rachelbt@users.noreply.github.com>
Co-authored-by: James Taylor <146064280+TaylorJ76@users.noreply.github.com>
YonatanKra added a commit that referenced this pull request Jul 1, 2024
* feat(tooltip)!: sets default max width (VIV-1799) (#1798)

* chore: updates docs

---------

Co-authored-by: James Taylor <jstaylor@vonht2fg32k90.home>

* feat(data-grid-cell)!: change default value of block-size and white-space (VIV-1797) (#1793)

* Make PR CI start on any main branch

* feat(audio-player)!: remove css-variable for min-inline-size and max-inline-size on host (VIV-1800) (#1803)

* feat(vue-wrappers)!: forward form values by DOM prop name (VIV-1798) (#1789)

* Improve error messages

* feat(vue-wrappers)!: forward form values by DOM prop name (VIV-1798)

---------

Co-authored-by: James Taylor <146064280+TaylorJ76@users.noreply.github.com>

* chore(docs): v4 migration page (VIV-1801) (#1796)

* fix(tabs)!: change gutters default to small (VIV-1796) (#1804)

* chore(docs)!: refine docs for v-4 (VIV-000) (#1813)

* feat(toggletip)!: add max-inline-size of 30ch & css-variable for custom size (VIV-1846) (#1814)

* attempt at RP

---------

Co-authored-by: James Taylor <jstaylor@vonht2fg32k90.home>
Co-authored-by: Richard Helm <86777660+RichardHelm@users.noreply.github.com>
Co-authored-by: Yonatan Kra <me@yonatankra.com>
Co-authored-by: Rachel Bratt Tannenbaum <rachelbt@users.noreply.github.com>
Co-authored-by: James Taylor <146064280+TaylorJ76@users.noreply.github.com>
YonatanKra added a commit that referenced this pull request Jul 1, 2024
* feat(tooltip): sets default max width (VIV-1799) (#1798)

* chore: updates docs

---------

Co-authored-by: James Taylor <jstaylor@vonht2fg32k90.home>

* feat(data-grid-cell): change default value of block-size and white-space (VIV-1797) (#1793)

* Make PR CI start on any main branch

* feat(audio-player)!: remove css-variable for min-inline-size and max-inline-size on host (VIV-1800) (#1803)

* feat(vue-wrappers)!: forward form values by DOM prop name (VIV-1798) (#1789)

* Improve error messages

* feat(vue-wrappers)!: forward form values by DOM prop name (VIV-1798)

---------

Co-authored-by: James Taylor <146064280+TaylorJ76@users.noreply.github.com>

* chore(docs): v4 migration page (VIV-1801) (#1796)

* fix(tabs)!: change gutters default to small (VIV-1796) (#1804)

* chore(docs)!: refine docs for v-4 (VIV-000) (#1813)

* feat(toggletip)!: add max-inline-size of 30ch & css-variable for custom size (VIV-1846) (#1814)

* attempt at RP

---------

Co-authored-by: James Taylor <jstaylor@vonht2fg32k90.home>
Co-authored-by: Richard Helm <86777660+RichardHelm@users.noreply.github.com>
Co-authored-by: Yonatan Kra <me@yonatankra.com>
Co-authored-by: Rachel Bratt Tannenbaum <rachelbt@users.noreply.github.com>
Co-authored-by: James Taylor <146064280+TaylorJ76@users.noreply.github.com>
YonatanKra added a commit that referenced this pull request Jul 1, 2024
* feat(tooltip): sets default max width (VIV-1799) (#1798)

* chore: updates docs

---------

Co-authored-by: James Taylor <jstaylor@vonht2fg32k90.home>

* feat(data-grid-cell)!: change default value of block-size and white-space (VIV-1797) (#1793)

* Make PR CI start on any main branch

* feat(audio-player)!: remove css-variable for min-inline-size and max-inline-size on host (VIV-1800) (#1803)

* feat(vue-wrappers)!: forward form values by DOM prop name (VIV-1798) (#1789)

* Improve error messages

* feat(vue-wrappers)!: forward form values by DOM prop name (VIV-1798)

---------

Co-authored-by: James Taylor <146064280+TaylorJ76@users.noreply.github.com>

* chore(docs): v4 migration page (VIV-1801) (#1796)

* fix(tabs)!: change gutters default to small (VIV-1796) (#1804)

* chore(docs)!: refine docs for v-4 (VIV-000) (#1813)

* feat(toggletip)!: add max-inline-size of 30ch & css-variable for custom size (VIV-1846) (#1814)

* attempt at RP

---------

Co-authored-by: James Taylor <jstaylor@vonht2fg32k90.home>
Co-authored-by: Richard Helm <86777660+RichardHelm@users.noreply.github.com>
Co-authored-by: Yonatan Kra <me@yonatankra.com>
Co-authored-by: Rachel Bratt Tannenbaum <rachelbt@users.noreply.github.com>
Co-authored-by: James Taylor <146064280+TaylorJ76@users.noreply.github.com>
YonatanKra added a commit that referenced this pull request Jul 1, 2024
feat(tooltip)!: sets default max width (VIV-1799) (#1798)

* chore: updates docs

---------

Co-authored-by: James Taylor <jstaylor@vonht2fg32k90.home>

* feat(data-grid-cell)!: change default value of block-size and white-space (VIV-1797) (#1793)

* Make PR CI start on any main branch

* feat(audio-player)!: remove css-variable for min-inline-size and max-inline-size on host (VIV-1800) (#1803)

* feat(vue-wrappers)!: forward form values by DOM prop name (VIV-1798) (#1789)

* Improve error messages

* feat(vue-wrappers)!: forward form values by DOM prop name (VIV-1798)

---------

Co-authored-by: James Taylor <146064280+TaylorJ76@users.noreply.github.com>

* chore(docs): v4 migration page (VIV-1801) (#1796)

* fix(tabs)!: change gutters default to small (VIV-1796) (#1804)

* chore(docs)!: refine docs for v-4 (VIV-000) (#1813)

* feat(toggletip)!: add max-inline-size of 30ch & css-variable for custom size (VIV-1846) (#1814)

* attempt at RP

---------

Co-authored-by: James Taylor <jstaylor@vonht2fg32k90.home>
Co-authored-by: Richard Helm <86777660+RichardHelm@users.noreply.github.com>
Co-authored-by: Yonatan Kra <me@yonatankra.com>
Co-authored-by: Rachel Bratt Tannenbaum <rachelbt@users.noreply.github.com>
Co-authored-by: James Taylor <146064280+TaylorJ76@users.noreply.github.com>
YonatanKra added a commit that referenced this pull request Jul 1, 2024
feat(tooltip): sets default max width (VIV-1799) (#1798)

* chore: updates docs

---------

Co-authored-by: James Taylor <jstaylor@vonht2fg32k90.home>

feat(data-grid-cell): change default value of block-size and white-space (VIV-1797) (#1793)

* Make PR CI start on any main branch

* feat(audio-player)!: remove css-variable for min-inline-size and max-inline-size on host (VIV-1800) (#1803)

* feat(vue-wrappers)!: forward form values by DOM prop name (VIV-1798) (#1789)

* Improve error messages

* feat(vue-wrappers)!: forward form values by DOM prop name (VIV-1798)

---------

Co-authored-by: James Taylor <146064280+TaylorJ76@users.noreply.github.com>

* chore(docs): v4 migration page (VIV-1801) (#1796)

* fix(tabs)!: change gutters default to small (VIV-1796) (#1804)

* chore(docs)!: refine docs for v-4 (VIV-000) (#1813)

* feat(toggletip)!: add max-inline-size of 30ch & css-variable for custom size (VIV-1846) (#1814)

* attempt at RP

---------

Co-authored-by: James Taylor <jstaylor@vonht2fg32k90.home>
Co-authored-by: Richard Helm <86777660+RichardHelm@users.noreply.github.com>
Co-authored-by: Yonatan Kra <me@yonatankra.com>
Co-authored-by: Rachel Bratt Tannenbaum <rachelbt@users.noreply.github.com>
Co-authored-by: James Taylor <146064280+TaylorJ76@users.noreply.github.com>
YonatanKra added a commit that referenced this pull request Jul 1, 2024
feat(tooltip)!: sets default max width (VIV-1799) (#1798)

feat(data-grid-cell)!: change default value of block-size and white-space (VIV-1797) (#1793)

feat(audio-player)!: remove css-variable for min-inline-size and max-inline-size on host (VIV-1800) (#1803)

feat(vue-wrappers)!: forward form values by DOM prop name (VIV-1798) (#1789)

feat(vue-wrappers)!: forward form values by DOM prop name (VIV-1798)

chore(docs): v4 migration page (VIV-1801) (#1796)

fix(tabs)!: change gutters default to small (VIV-1796) (#1804)

chore(docs)!: refine docs for v-4 (VIV-000) (#1813)

feat(toggletip)!: add max-inline-size of 30ch & css-variable for custom size (VIV-1846) (#1814)
@github-actions github-actions bot mentioned this pull request Jul 1, 2024
YonatanKra added a commit that referenced this pull request Jul 4, 2024
* feat(tooltip)!: sets default max width (VIV-1799) (#1798)

* chore: updates docs

* feat(data-grid-cell)!: change default value of block-size and white-space (VIV-1797) (#1793)

* Make PR CI start on any main branch

* feat(audio-player)!: remove css-variable for min-inline-size and max-inline-size on host (VIV-1800) (#1803)

* feat(vue-wrappers)!: forward form values by DOM prop name (VIV-1798) (#1789)

* Improve error messages

* feat(vue-wrappers)!: forward form values by DOM prop name (VIV-1798)

* chore(docs): v4 migration page (VIV-1801) (#1796)

* fix(tabs)!: change gutters default to small (VIV-1796) (#1804)

* chore(docs)!: refine docs for v-4 (VIV-000) (#1813)

* feat(toggletip)!: add max-inline-size of 30ch & css-variable for custom size (VIV-1846) (#1814)

* attempt at RP

---------

Co-authored-by: James Taylor <jstaylor@vonht2fg32k90.home>
Co-authored-by: Richard Helm <86777660+RichardHelm@users.noreply.github.com>
Co-authored-by: Yonatan Kra <me@yonatankra.com>
Co-authored-by: Rachel Bratt Tannenbaum <rachelbt@users.noreply.github.com>
Co-authored-by: James Taylor <146064280+TaylorJ76@users.noreply.github.com>
YonatanKra added a commit that referenced this pull request Jul 8, 2024
* feat(tooltip)!: sets default max width (VIV-1799) (#1798)

* chore: updates docs

---------

Co-authored-by: James Taylor <jstaylor@vonht2fg32k90.home>

* feat(data-grid-cell)!: change default value of block-size and white-space (VIV-1797) (#1793)

* Make PR CI start on any main branch

* feat(audio-player)!: remove css-variable for min-inline-size and max-inline-size on host (VIV-1800) (#1803)

* feat(vue-wrappers)!: forward form values by DOM prop name (VIV-1798) (#1789)

* Improve error messages

* feat(vue-wrappers)!: forward form values by DOM prop name (VIV-1798)

---------

Co-authored-by: James Taylor <146064280+TaylorJ76@users.noreply.github.com>

* chore(docs): v4 migration page (VIV-1801) (#1796)

* fix(tabs)!: change gutters default to small (VIV-1796) (#1804)

* chore(docs)!: refine docs for v-4 (VIV-000) (#1813)

* feat(toggletip)!: add max-inline-size of 30ch & css-variable for custom size (VIV-1846) (#1814)

* attempt at RP

---------

Co-authored-by: James Taylor <jstaylor@vonht2fg32k90.home>
Co-authored-by: Richard Helm <86777660+RichardHelm@users.noreply.github.com>
Co-authored-by: Yonatan Kra <me@yonatankra.com>
Co-authored-by: Rachel Bratt Tannenbaum <rachelbt@users.noreply.github.com>
Co-authored-by: James Taylor <146064280+TaylorJ76@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Breaking Change Changes that will go into the next major version
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants