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

fix: ensure tags props are immutable #139

Merged
merged 1 commit into from Oct 14, 2022
Merged

Conversation

harlan-zw
Copy link
Collaborator

Issue

nuxt/nuxt#15159

Description

Tag props are being passed by reference when turned to tags, when we clean up tag props, it's modifying the source props.

@github-actions
Copy link
Contributor

Export Size

@vueuse/head

Name Size Diff
headInputToTags 445 B +5 B (+1.14%) 🔺
resolveHeadEntriesToTags 1.04 KB +4 B (+0.38%) 🔺
renderHeadToString 1.65 KB +1 B (+0.06%) 🔺
createHead 2.32 KB -4 B (-0.17%) 🔽
Unchanged

@vueuse/head

Name Size Diff
createElement 202 B -
Head 535 B -
injectHead 162 B -
IS_BROWSER 87 B -
isEqualNode 183 B -
propsToString 182 B -
resolveHeadEntries 223 B -
resolveUnrefHeadInput 184 B -
setAttrs 219 B -
sortTags 203 B -
tagDedupeKey 262 B -
tagToString 267 B -
updateElements 749 B -
useHead 261 B -
useHeadSafe 573 B -

@harlan-zw harlan-zw merged commit c1d38ae into v1 Oct 14, 2022
harlan-zw added a commit that referenced this pull request Nov 12, 2022
* refactor!: v1 initial, CSR performance, awaitable hooks (#126)

* performance: rewrite reactivity for SSR / CSR

* chore: rolleback artifacts

* chore: progress commit

* fix: use ./src import

* refactor!: simplify utils for smaller pkg (#127)

* performance: rewrite reactivity for SSR / CSR

* chore: rolleback artifacts

* chore: progress commit

* fix: use ./src import

* refactor: simplify util code for smaller payload

* refactor!: continued (#128)

* performance: rewrite reactivity for SSR / CSR

* chore: rolleback artifacts

* chore: progress commit

* fix: use ./src import

* refactor: simplify util code for smaller payload

* wip: part two of refactor

* chore: fix tests

* chore: run github ci on v1

* chore: run github ci on v1

* chore: bump version

* Release v1.0.0-rc.0

* Release v1.0.0-rc.1

* fix: ensure empty attrs reset attrs (#130)

* fix: ensure empty attr is set

* fix: test valid tags

* fix(ci): release on rc

* Release v1.0.0-rc.2

* fix(ci): remove broken --pre

* Release v1.0.0-rc.3

* fix(ci): use --pre

* fix: use simpler API for tags (#131)

* feat: improved hook api (#132)

* chore: prefer runtime prop for id & position (#134)

* chore: prefer runtime prop for id & position

* fix: type errors

* fix: add `headTags` for backwards compatibility (#133)

* Release v1.0.0-rc.4

* fix: add legacy `addHeadObjs` (#135)

* refactor!: useHeadRaw -> useHeadSafe (#136)

* refactor!: useHeadRaw -> useHeadSafe

* feat: tree-shake encoding

* chore: useHeadSafe types

* Release v1.0.0-rc.5

* fix(types): export `MaybeComputedRef`

* chore: add missing `type`

* Release v1.0.0-rc.6

* chore(pkg): bump deps

* fix(titleTemplate): allow titleTemplate to resolve the title (#137)

* fix(titleTemplate): allow titleTemplate to resolve the title

* chore: add happy case

* chore: fix type issues

* chore: maybe fix type issue

* chore: maybe fix type issue

* chore: maybe fix type issue

* chore: maybe fix type issue

* Release v1.0.0-rc.7

* chore: bump deps

* chore: test issue #138

* chore: linting

* chore(pkg): bump deps

* Release v1.0.0-rc.8

* fix: ensure tags props are immutable (#139)

* Release v1.0.0-rc.9

* chore(pkg): bump deps

* Release v1.0.0-rc.10

* fix: ensure single pass of `htmlAttr` sanitization, fixes #142 (#143)

* fix: ensure csr elements are created equally (#145)

* fix: ensure csr elements are created equally

* fix: remove sanitising

* chore: fix doc

* chore(pkg): bump deps

* Release v1.0.0-rc.11

* fix: vue 2.7 globalProperties, fixes #141

* chore(pkg): bump zhead

* Release v1.0.0-rc.12

* add vue 2.7 compatibility to Head component (#147)

* add vue 2.7 compatibility to Head component

* remove vue-demi in favor of checking version from vue itself

* chore(pkg): bump deps

* fix: dedupe `htmlAttrs` and `bodyAttrs` for csr (#149)

* Release v1.0.0-rc.13

* fix: Vue 2.7 compatibility (#150)

* add separate vue2 plugin for 2.7 compatibility

* Vue 2.7 install readme

* fix: handle style / class attrs

* chore: add link back to pinia

* chore: vue2 example

Co-authored-by: Val Trubachev <val.trubachev@zoro.com>

* chore(pkg): ignore eslint in vue2 example

* Release v1.0.0-rc.14

* feat: migrate core to unhead (#157)

* feat: migrate core to unhead

* chore: fix SSR test with timeouts

* chore: readme link fix

* chore(pkg): bump deps

* chore(ci): switch to next channel

* chore: document install

* Release v1.0.0-next.1

* chore: improve doc

* chore: bump unhead, update tests and examples

* Release v1.0.0-next.2

* chore: bump unhead

* chore: expose `VueHeadMixin`

* Release v1.0.0-next.3

Co-authored-by: Valera Trubachev <vetruvet@gmail.com>
Co-authored-by: Val Trubachev <val.trubachev@zoro.com>
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.

None yet

1 participant