[CLOSED] [BezierEditor] Fix fadeout issue

[core-ai-bot]

Issue by MarcelGerber
Wednesday Mar 19, 2014 at 14:03 GMT
Originally opened as adobe/brackets#7248

---

Steps to reproduce the issue, introduced by #7235:

1. Start an Inline Timing Function Editor
2. Make the function invalid, like cubic-bezier(1.000001, 0, 0, 0)
3. Make it valid again
4. Make some other valid changes (like deleting the additional zeros) while the fadeout is still running

Result: The hint is immediately hidden
Expected: The fadeout should continue

This PR makes the code a bit cleaner (using editor.hint object for everything that has to do with the hint) and uses .hide(), suggested by@zaggino.

---

MarcelGerber included the following code: https://github.com/adobe/brackets/pull/7248/commits

[core-ai-bot]

Comment by redmunds
Wednesday Mar 19, 2014 at 15:16 GMT

---

@SAPlayer I could review pull requests faster if the branch names weren't so long :)

[core-ai-bot]

Comment by MarcelGerber
Wednesday Mar 19, 2014 at 15:28 GMT

---

@redmunds I'll try to use a shorter one next time ;)

[core-ai-bot]

Comment by njx
Wednesday Mar 19, 2014 at 18:25 GMT

---

@redmunds Try https://github.com/defunkt/hub - then you can do things like hub checkout https://github.com/adobe/brackets/pull/7248

[core-ai-bot]

Comment by peterflynn
Wednesday Mar 19, 2014 at 20:14 GMT

---

I always just copy-paste the branch name from the top of the PR into the terminal, personally :-) Or you can click the "command line" link at bottom (next to the merge button) and copy the full command line to the clipboard in 1 click...

[core-ai-bot]

Comment by redmunds
Friday Mar 21, 2014 at 19:42 GMT

---

Looks good. Merging.