Some variables in ternary operator are not updating display. #40

mysticatea · 2014-07-22T09:59:35Z

Repro code: https://dl.dropboxusercontent.com/u/5739705/polymer/ternary/test.html

I think that the hidden term at first time has been not observed its value.
The hidden term: It's the third term if the first term is truthy. Otherwise, it's the second term.

arv · 2014-07-22T16:01:50Z

When I extract this into a test it works. I suspect that it is an issue with using select as a property name since that clashes with the builtin select method.

arv · 2014-07-22T16:09:45Z

Scratch that... I can reproduce this. Trying to figure out what is going wrong

For conditionals we failed to setup the observer for both branches as required. This lead to us only observing either the true or false branch but not both. Fixes googlearchive#40

For conditionals we failed to setup the observer for both branches as required. This lead to us only observing either the true or false branch but not both. Now we mark conditional (as well as || and &&) as dynamicDeps so the dependencies gets reevaluated as needed. Fixes googlearchive#40

arv mentioned this issue Jul 22, 2014

Fix issue with conditional expressions #41

Merged

arv closed this as completed in #41 Jul 22, 2014

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Some variables in ternary operator are not updating display. #40

Some variables in ternary operator are not updating display. #40

mysticatea commented Jul 22, 2014

arv commented Jul 22, 2014

arv commented Jul 22, 2014

Some variables in ternary operator are not updating display. #40

Some variables in ternary operator are not updating display. #40

Comments

mysticatea commented Jul 22, 2014

arv commented Jul 22, 2014

arv commented Jul 22, 2014