Closed Bug 1629712 Opened 5 years ago Closed 4 years ago

Update Code Reviews Checklist

Categories

(DevTools :: Documentation, enhancement, P3)

enhancement

Tracking

(firefox81 fixed)

RESOLVED FIXED
81 Branch
Tracking Status
firefox81 --- fixed

People

(Reporter: victoria, Assigned: Honza)

Details

Attachments

(1 file)

Looks like due to the conversion to Sphinx, the Code Reviews Checklist bulleted lists became flattened by a level (compare to the old version).

Also, I'd like to update this sentence:

If a new piece of UI or new user interaction is added/modified, then UX is ui-review? on the bug.

To this:

If any user-facing interfaces are added/modified, double-check the changes with the UX mockups or specs, if available. If there's any confusion, need-info the UX designer.

I can see the issue, thanks for the report Victoria.

Honza

Severity: -- → S3
Priority: -- → P3
Assignee: nobody → odvarko
Status: NEW → ASSIGNED
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → 81 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: