Closed Bug 185332 Opened 22 years ago Closed 22 years ago

Rewrite the description for timezone param

Categories

(Bugzilla :: Administration, task, P4)

2.17.1

Tracking

()

RESOLVED FIXED
Bugzilla 2.18

People

(Reporter: jouni, Assigned: jouni)

Details

Attachments

(1 file, 1 obsolete file)

I'm proposing rewriting the description for the timezone parameter. :-)

The current text lacks a space ("thenthe"), contains a "can't" where "won't"
would be more appropriate (you don't always even _want_ the tz to be displayed)
and refers to "sql server" where "database server" would be more appropriate
("sql server" is a specific database server by Microsoft). 

Patch incoming.
Attached patch Patch v1 (obsolete) — Splinter Review
Attachment #109279 - Flags: review?
Comment on attachment 109279 [details] [diff] [review]
Patch v1

nit: would be nice if the patch actually compiled.
Attachment #109279 - Flags: review? → review-
Attached patch Patch v2Splinter Review
Oops. The . operator was cut off when copy-pasting the patch to a text file for
attachment. The actual change did compile, though. :-)
Attachment #109279 - Attachment is obsolete: true
Comment on attachment 109294 [details] [diff] [review]
Patch v2

Take another shot, now that my c/p didn't cut the rightmost column off ;-)
Attachment #109294 - Flags: review?(bugreport)
Comment on attachment 109294 [details] [diff] [review]
Patch v2

r=joel

(optional - puts a comma after the "")
Attachment #109294 - Flags: review?(bugreport) → review+
Status: NEW → ASSIGNED
Flags: approval?
Priority: -- → P4
Target Milestone: --- → Bugzilla 2.18
Flags: approval? → approval+
... and fixed.

Checking in defparams.pl;
/cvsroot/mozilla/webtools/bugzilla/defparams.pl,v  <--  defparams.pl
new revision: 1.106; previous revision: 1.105
done
Status: ASSIGNED → RESOLVED
Closed: 22 years ago
Resolution: --- → FIXED
QA Contact: matty_is_a_geek → default-qa
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: