about:cache title capitalization inconsistent

VERIFIED FIXED

Status

()

Core
Networking: Cache
--
trivial
VERIFIED FIXED
15 years ago
14 years ago

People

(Reporter: Jeremy M. Dolan, Assigned: Darin Fisher)

Tracking

Trunk
x86
Linux
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

(Reporter)

Description

15 years ago
Number of entries: 	457
Maximum storage size: 	8192 KiB
Storage in use: 	5402 KiB
Inactive Storage: 	4328 KiB

Here, "Inactive Storage" has its second word capitalized. None of the others do.

And for individual entries:

           Key: http://graphics7.nytimes.com/images/2003/12/26/...
     Data size: 151800 Bytes
   Fetch count: 3
 Last Modified: Fri Dec 26 15:45:25 2003
       Expires: Fri Dec 26 17:57:34 2003

Last Mod'd is double-cap'd, and nothing else.

Also, "Bytes" being capitalized looks silly, being not a title and mid-"sentence".

I'll attach a patch for this minor issue.
(Reporter)

Comment 1

15 years ago
Created attachment 138005 [details] [diff] [review]
Make capitalization consistent - Table headers as titles (All major words cap'd)

Someone can double check with an English prof that the preposition 'of' should
remain lowercase but the "In" should be capitalized because it is functioning
as an adverb in this case. I've never been good with the whole sentence
labeling thing. Either way this is a definite improvement. Reads and looks much
better.
(Reporter)

Comment 2

15 years ago
Darin, bz, since you two just reviewed another minor "about:cache" change
elsewhere, I though you could take a quick look at this one. -thanks
Assignee: general → darin
Component: Browser-General → Networking: Cache
QA Contact: general → cacheqa
Comment on attachment 138005 [details] [diff] [review]
Make capitalization consistent - Table headers as titles (All major words cap'd)

Please set review flags if you want reviews....

r=bzbarsky
Attachment #138005 - Flags: superreview?(darin)
Attachment #138005 - Flags: review+

Comment 4

15 years ago
Comment on attachment 138005 [details] [diff] [review]
Make capitalization consistent - Table headers as titles (All major words cap'd)

I disagree with all of your choices except "bytes"

consider
http://landfill.mozilla.org/mxr-test/seamonkey/search?string=+in+&find=dtd&filt
er=
ignore calendar since it's almost always wrong.

15:22 < stephend> NEVER
15:22 < stephend> capitalize In
15:22 < stephend> prepositions, never.


I'd rather
"Sentence style capitalization."
Attachment #138005 - Flags: superreview?(darin) → superreview-
(Reporter)

Comment 5

15 years ago
Created attachment 138184 [details] [diff] [review]
lowercase extra words

I think this looks significantly worse, especially for the entry listings which
are right-aligned, but whatever, I'll settle for any form of consistency.

Besides, this way I can blame timeless.
(Reporter)

Updated

15 years ago
Attachment #138184 - Flags: superreview?(timeless)
Attachment #138184 - Flags: review?(bz-vacation)
timeless can't sr, so it's no use asking him for sr...

This needs moa from darin in any case, like I requested; I won't even bother
looking at the changes till darin OKs them -- it's just a waste of my time as
the first patch showed.

Updated

15 years ago
Attachment #138184 - Flags: superreview?(timeless) → superreview?(darin)
(Reporter)

Comment 7

15 years ago
Comment on attachment 138005 [details] [diff] [review]
Make capitalization consistent - Table headers as titles (All major words cap'd)

If he can't sr +, he shouldn't sr -.
Attachment #138005 - Attachment description: Capitalization cleanup → Make capitalization consistent - Table headers as titles (All major words cap'd)
Attachment #138005 - Flags: superreview- → superreview?(darin)
(Assignee)

Comment 8

15 years ago
Comment on attachment 138184 [details] [diff] [review]
lowercase extra words

i vote for "Sentence style capitalization" .. i think it's plenty clear.
Attachment #138184 - Flags: superreview?(darin) → superreview+
(Assignee)

Updated

15 years ago
Attachment #138005 - Flags: superreview?(darin) → superreview-
Comment on attachment 138184 [details] [diff] [review]
lowercase extra words

r=bzbarsky
Attachment #138184 - Flags: review?(bz-vacation) → review+
(Reporter)

Comment 10

15 years ago
checked in. thanks.
Status: NEW → RESOLVED
Last Resolved: 15 years ago
Resolution: --- → FIXED

Comment 11

14 years ago
Comment on attachment 138184 [details] [diff] [review]
lowercase extra words

-    mBuffer.Append("\n<b>     Data size: </b>");
+    mBuffer.Append("\n<b>     Data size:</b>");

Umm, you removed formatting in preformatted text. Those spaces were there for a
reason.

Though perhaps a space after </b> instead of before it would of been more
sensible.
(Reporter)

Comment 12

14 years ago
Right you are, Alexey. The fields at the top are in table cells, and they also
had the extra space after the colon. I only meant to trim it off of those. Filed
bug 239149, and I'll make a patch tomorrow. This bug (capitalization) is fixed.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.