[CLOSED] add current and previous documents to currentDocumentChange event

[core-ai-bot]

Issue by zaggino
Monday Apr 14, 2014 at 23:29 GMT
Originally opened as adobe/brackets#7509

---

Useful nit to avoid calling DocumentManager.getCurrentDocument() everytime when hooked on the event and also the only way to know what was the previous document I think.

---

zaggino included the following code: https://github.com/adobe/brackets/pull/7509/commits

[core-ai-bot]

Comment by lkcampbell
Tuesday Apr 15, 2014 at 13:47 GMT

---

+1 for this. I actually wish every Change event we have provided a newState and an oldState. Would save on maintaining cached states.

[core-ai-bot]

Comment by peterflynn
Monday Apr 28, 2014 at 21:30 GMT

---

@zaggino This looks good -- but please just update the docs at the top of the module to indicate the new 2nd & 3rd args.

[core-ai-bot]

Comment by zaggino
Monday Apr 28, 2014 at 22:02 GMT

---

@peterflynn done

[core-ai-bot]

Comment by peterflynn
Monday Apr 28, 2014 at 22:26 GMT

---

Looks good now -- thanks!