Created attachment 8600468 [details] [review] Pull Request
Comment on attachment 8600468 [details] [review] Pull Request Just a minor not about the use of `description`. Other than that it looks good!
This bug introduced a couple of strings: "Mark as read/unread". But we still had "Mark as Read/Unread" (not the capitalization). Is the different capitalization actually wanted and/or needed?
I had no explicit intent with the capitalization. For the sake of consistency, I will change the strings from this bug to have the same capitalization as the ones already present (i.e. capitalized Read/Unread). Thanks for noting this.
Francesco, I reported Bug 1161254 for the capitalization and took the liberty of CC'ing you there. A pull request is already available for it.