Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings
/vuePublic

fix: correctly update styles#12946

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Closed
btea wants to merge2 commits intovuejs:mainfrombtea:fix/updateStyle
Closed

Conversation

btea
Copy link

fix#12901

What kind of change does this PR introduce? (check at least one)

  • Bugfix
  • Feature
  • Code style update
  • Refactor
  • Build-related changes
  • Other, please describe:

Does this PR introduce a breaking change? (check one)

  • Yes
  • No

If yes, please describe the impact and migration path for existing applications:

The PR fulfills these requirements:

If adding anew feature, the PR's description includes:

  • A convincing reason for adding this feature (to avoid wasting your time, it's best to open a suggestion issue first and wait for approval before working on it)

Other information:

@yyx990803
Copy link
Member

Thank you for the PR. In Vue 3, the new styles are always set, so I made it to be consistent with Vue 3 behavior inf5ef882.

@bteabtea deleted the fix/updateStyle branchDecember 7, 2023 01:17
@webIcome
Copy link

@yyx990803 the new styles are always set, 这样改有一个问题,比如el-dialog通过指令去改变dom的宽度,然后包含el-dialog组件的响应式变量更改,就会重新渲染,el-dialog 的宽度被重置了

chenyulun reacted with thumbs up emoji

@chenyulun
Copy link

@yyx990803 the new styles are always set, 这样改有一个问题,比如el-dialog通过指令去改变dom的宽度,然后包含el-dialog组件的响应式变量更改,就会重新渲染,el-dialog 的宽度被重置了

我们升级2.7.16也出现了这样的问题,破坏性改动

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@sxzzsxzzsxzz approved these changes

Assignees
No one assigned
Labels
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

bind style merge not as expected
6 participants
@btea@yyx990803@webIcome@chenyulun@sxzz@edison1105

[8]ページ先頭

©2009-2025 Movatter.jp