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

attribute on dynamic mounted tag (`data-is` tag with expression) is not updating #2175

Closed
mcfog opened this Issue Dec 24, 2016 · 3 comments

Comments

Projects
None yet
3 participants
@mcfog

mcfog commented Dec 24, 2016

Help us to manage our issues by answering the following:

  1. Describe your issue:

atrribute with expression and opts.[attribute] is not updating correctly (only first time is good)

seems related to #2102

  1. Can you reproduce the issue?

yes

Post the link using one of our bug report templates:

http://plnkr.co/edit/g8zBg18ekDMidMrPuzQK?p=preview

remove expression in data-is make attribute works => http://plnkr.co/edit/FjqIwnMEU9yeVxJ49BBl?p=preview

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

chrome 55

  1. Which version of Riot does it affect?

3.0.5

  1. How would you tag this issue?
  • Question
  • Bug
  • Discussion
  • Feature request
  • Tip
  • Enhancement
  • Performance
@rsbondi

This comment has been minimized.

Show comment
Hide comment
@rsbondi

rsbondi Dec 24, 2016

Member

It seems to actually be route.query() is an empty object and not an issue with the attribute. Any insight on this @cognitom?

Member

rsbondi commented Dec 24, 2016

It seems to actually be route.query() is an empty object and not an issue with the attribute. Any insight on this @cognitom?

@rsbondi rsbondi added the bug label Dec 24, 2016

@rsbondi

This comment has been minimized.

Show comment
Hide comment
@rsbondi

rsbondi Dec 24, 2016

Member

my bad, it is not empty

Member

rsbondi commented Dec 24, 2016

my bad, it is not empty

@rsbondi rsbondi self-assigned this Dec 24, 2016

@rsbondi rsbondi referenced this issue Dec 27, 2016

Merged

Data is attr #2181

@GianlucaGuarini

This comment has been minimized.

Show comment
Hide comment
@GianlucaGuarini

GianlucaGuarini Jan 10, 2017

Member

fixed in riot@3.0.6

Member

GianlucaGuarini commented Jan 10, 2017

fixed in riot@3.0.6

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