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

This app is totally broken #10

Closed
adardesign opened this issue Jun 28, 2012 · 8 comments
Closed

This app is totally broken #10

adardesign opened this issue Jun 28, 2012 · 8 comments

Comments

@adardesign
Copy link

After choosing something and changing a later or previous selection it doesn't update as it should

@garann
Copy link
Owner

garann commented Jun 28, 2012

Totally broken? Oh no. Do you think there's any point in trying to fix it, or should I just voluntarily exile myself to a new career as a carrot farmer in Uruguay? If you think there's a chance some crumb of usefulness can be chipped from the facade of frustrating impotence, maybe you could tell me a little more about what isn't updating and what your environment looks like?

@adardesign
Copy link
Author

first, sorry for calling it totaly broken.

Sure, ill try to explain you what happened, (wish i can screencast it)

Using chrome canary (Version 20.0.1132.43) // maybe thats why i had this expireince.

When i choose the first option (client or server) from the series of questions it behaves as expected, it narrowed down the list of templates according to my selction, great!

All seems to be working fine.

But when i then go to the 3rd question, it hides all of the templates, but now even if i go back and change the first and or the second question, it still has "all templlates" hidden.

Thanx anyway

@garann
Copy link
Owner

garann commented Jun 29, 2012

You can call it totally broken or anything else you like, as long as you explain what needs to be fixed. :)

What you're describing sounds like a problem I'm aware of, but haven't yet fixed. The tool works as a process of elimination, but there's no way to un-set a given option, taking it out of the equation. I'll comment again when I add that in, and hopefully it will work for you at that point. Thanks for explaining!

@adardesign
Copy link
Author

Thanks much!

On Fri, Jun 29, 2012 at 11:04 AM, Garann Means <
reply@reply.github.com

wrote:

You can call it totally broken or anything else you like, as long as you
explain what needs to be fixed. :)

What you're describing sounds like a problem I'm aware of, but haven't yet
fixed. The tool works as a process of elimination, but there's no way to
un-set a given option, taking it out of the equation. I'll comment again
when I add that in, and hopefully it will work for you at that point.
Thanks for explaining!


Reply to this email directly or view it on GitHub:
#10 (comment)

@ravi
Copy link

ravi commented Feb 19, 2013

Garann, the tool is actually enormously useful -- I did not even know about Transparency until I used the tool! But the issue is a bit larger than unsetting a choice. Even flipping a choice in a particular way causes somewhat erratic behaviour (that can be cleared by reloading the page). E.g: leave everything else as-is, and open the DOM structure question and select DOM. Now click on DOM again. Then click on String. The choices go blank. Clicking some number of times on String brings the relevant choices back into view, and/or clicking back on DOM followed by String in a sequence of clicks returns the correct values. I haven't had the time to figure out the algorithm to induce the right behaviour once the strangeness starts, nor have I had a chance to look at your code to see what could be causing this. I would like to do the latter though, in case I can submit a patch/pull. In the meantime, documenting my finding here. Thank you for the tool itself. Very helpful.

@xixixao
Copy link

xixixao commented Feb 23, 2013

I confirm ravi's findings, on FF18. The tool looks great though, thanks .)

@garann
Copy link
Owner

garann commented Feb 24, 2013

@ravi - thanks for the steps to reproduce! Should be fixed now.

@ravi
Copy link

ravi commented Feb 27, 2013

@garann looks good, I cannot reproduce the problem any longer. Thank you for the fix.

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

No branches or pull requests

4 participants