Closed Bug 1097251 Opened 11 years ago Closed 11 years ago

Diff viewer "Lines of Context" setting should be 8 instead of 5 to match our patch guidelines

Categories

(MozReview Graveyard :: General, defect)

Product:
MozReview Graveyard
Component:
General
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: MattN, Assigned: smacleod)

Details

Even though I can click to see more context, I'm used to 8 lines so the current setting of 5 seems like too little.
Seems like a reasonable request. I've updated the setting[1] to 8 lines which I think will be fine unless someone wants to push back. Note that these diff fragments are cached and not invalidated by this setting, so you may only see 5 lines on currently published diffs which are still in the cache. [1] https://reviewboard.mozilla.org/admin/settings/diffs/
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Assignee: nobody → smacleod
Product: Developer Services → MozReview
You need to log in before you can comment on or make changes to this bug.