[CLOSED] Make font settings react to pref changes

[core-ai-bot]

Issue by MarcelGerber
Sunday May 31, 2015 at 22:42 GMT
Originally opened as adobe/brackets#11190

---

This should go with, or before, #11130.

---

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

[core-ai-bot]

Comment by MiguelCastillo
Sunday May 31, 2015 at 23:52 GMT

---

@MarcelGerber changes look good. I am not even sure why I didn't add this with the initial PR, so thanks :)

[core-ai-bot]

Comment by nethip
Monday Jun 01, 2015 at 07:53 GMT

---

@MarcelGerber thanks for this change! I just have a s comment. Can we change the variable name to something like currFontSizeValue. I guess it would improve the readability of the code. Otherwise LGTM.

[core-ai-bot]

Comment by MarcelGerber
Monday Jun 01, 2015 at 13:44 GMT

---

Changes pushed.

[core-ai-bot]

Comment by MiguelCastillo
Monday Jun 01, 2015 at 14:26 GMT

---

looks good, works well. merging in.