Firefox/Code Review: Difference between revisions

→‎What does r+ mean?: Add link to localisation best practices
(s/gavin/dcamp/)
(→‎What does r+ mean?: Add link to localisation best practices)
Line 31: Line 31:
** security concerns, especially where content and chrome touch each other;
** security concerns, especially where content and chrome touch each other;
** significant performance issues, especially on key paths like startup and page load;
** significant performance issues, especially on key paths like startup and page load;
** localization and accessibility issues.
** [https://developer.mozilla.org/en-US/docs/Mozilla/Localization/Localization_content_best_practices localization] and accessibility issues.
* If it does cause problems, I will help make it right.
* If it does cause problems, I will help make it right.
* The patch has test coverage appropriate to the change.
* The patch has test coverage appropriate to the change.
canmove, Confirmed users, Bureaucrats and Sysops emeriti
3,627

edits