Thanks, Francesco. I like it! I have just one really minor comment.
[]s, Thiago.
https://codereview.appspot.com/6851058/diff/1/app/views/notifications.js File app/views/notifications.js (right):
https://codereview.appspot.com/6851058/diff/1/app/views/notifications.js#newcode50 app/views/notifications.js:50: if (notifierBox && I think will you always have the 'notifier-box' element. It is hard-coded in index.html. So probably you can remove this check.
https://codereview.appspot.com/6851058/
« Back to merge proposal
Thanks, Francesco.
I like it! I have just one really minor comment.
[]s,
Thiago.
https:/ /codereview. appspot. com/6851058/ diff/1/ app/views/ notifications. js notifications. js (right):
File app/views/
https:/ /codereview. appspot. com/6851058/ diff/1/ app/views/ notifications. js#newcode50 notifications. js:50: if (notifierBox &&
app/views/
I think will you always have the 'notifier-box' element. It is
hard-coded in index.html. So probably you can remove this check.
https:/ /codereview. appspot. com/6851058/