8000 Class duplication for dynamic components · Issue #3051 · riot/riot · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Class duplication for dynamic components #3051

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.

Sign up for GitHub

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

Jump to bottom
Open
7 tasks
kachurun opened this issue Jan 10, 2025 · 4 comments
Open
7 tasks

Class duplication for dynamic components #3051

kachurun opened this issue Jan 10, 2025 · 4 comments
Labels
breaking change fixed

Comments

@kachurun
Copy link
Member

Help us to manage our issues by answering the following

  1. Describe your issue:
    If the component name changes dynamically, the class names are duplicated after rendering.

  2. Can you reproduce the issue?
    https://plnkr.co/edit/G0HTVKRApQSQw35V

  3. On which browser/OS does the issue appear?

  4. Which version of Riot does it affect?

  5. How would you tag this issue?

  • Question
  • Bug
  • Discussion
  • Feature request
  • Tip
  • Enhancement
  • Performance
@kachurun
Copy link
Member Author

I guess its related to #3023

@GianlucaGuarini
Copy link
Member

Yes that's the same problem. It will be fixed with our major release soon. Thank you for reporting it

GianlucaGuarini added a commit that referenced this issue May 16, 2025
@GianlucaGuarini
fix: patch for #3051
45843be
GianlucaGuarini added a commit that referenced this issue May 16, 2025
@GianlucaGuarini
fix: patch for #3051
d75d0d4
GianlucaGuarini added a commit that referenced this issue May 16, 2025
@GianlucaGuarini
updated: Add comment for the #3051 edge case
aa0eb94
@GianlucaGuarini
Copy link
Member

This issue has been fixed in Riot.js@alpha.2
Could you please have a look at it? Thank you
https://plnkr.co/edit/pOaaO4xDlS2jlwf7

@kachurun
Copy link
Member Author

Yep, looks like it works great now!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
breaking change fixed
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@GianlucaGuarini @kachurun
0