Closed Bug 981267 Opened 6 years ago Closed 6 years ago

Bad indentation in about:networking

Categories

(Core :: Networking, defect)

30 Branch
x86_64
Linux
defect
Not set

Tracking

()

RESOLVED FIXED
mozilla30

People

(Reporter: alex2august, Assigned: alex2august)

Details

Attachments

(1 file)

User Agent: Mozilla/5.0 (X11; Linux x86_64; rv:30.0) Gecko/20100101 Firefox/30.0 (Beta/Release)
Build ID: 20140305030201
Attached patch bad_indent.patchSplinter Review
Attachment #8388051 - Flags: review?(valentin.gosu)
Comment on attachment 8388051 [details] [diff] [review]
bad_indent.patch

Review of attachment 8388051 [details] [diff] [review]:
-----------------------------------------------------------------

Looks good.

Just note that patches need to be reviewed by a module peer - https://wiki.mozilla.org/Modules/Core
Also, the patch comment should usually look like "Bug 981267: Bad indentation in about:networking"
Attachment #8388051 - Flags: review?(sworkman)
OK - weird. I tried to r+ this several times but bugzilla is giving me a 500 error.

r=me - Go ahead and land the patch with r=sworkman in the description.
Attachment #8388051 - Flags: review?(valentin.gosu) → review+
Attachment #8388051 - Flags: review?(sworkman) → review+
https://hg.mozilla.org/mozilla-central/rev/32a5ae80bfc8
Status: UNCONFIRMED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla30
You need to log in before you can comment on or make changes to this bug.