Page MenuHomePhabricator

Newline after "See Also" field in bugzillapreview
Closed, ResolvedPublic

Description

@Nemo_bis says in T848: Empty fields shouldn't be pasted as "none", "unspecified" or "unknown":

why the newline before last "none"?

This is a small problem now present in every task in bugzillapreview, i.e. https://bugzillapreview.wmflabs.org/T7238

Event Timeline

Qgil assigned this task to chasemp.
Qgil raised the priority of this task from to Medium.
Qgil updated the task description. (Show Details)
Qgil added a project: Bugzilla-Preview.
Qgil changed Security from none to None.
Qgil added subscribers: Qgil, Nemo_bis.
Qgil lowered the priority of this task from Medium to Low.Oct 24 2014, 6:28 PM

Still valid in 2nd edition of bugzillapreview, e.g. seen on https://bugzillapreview.wmflabs.org/T69 (though I don't really care whether we fix this or not).

so the old case was

See Also:
none

If none there is nothing present.

Now

See Also:
http://urlofchoise.com

or

See Also:
http//urlsabound.com
http://moreurls.com

in the case of multiple url's not doing a newline is ugly, so I don't know if this issue as first reported is valid or not

So this looks indeed good:

See Also with one URL: https://bugzillapreview.wmflabs.org/T7

See Also with many URLs: https://bugzillapreview.wmflabs.org/T69