Closed
Bug 552166
Opened 15 years ago
Closed 15 years ago
Change title of empty tabs from "(Untitled)" to "New Tab"
Categories
(Firefox :: Tabbed Browser, defect)
Firefox
Tabbed Browser
Tracking
()
VERIFIED
FIXED
Firefox 3.7a4
People
(Reporter: markus, Assigned: markus)
References
Details
Attachments
(2 files, 2 obsolete files)
12.32 KB,
image/png
|
Details | |
3.23 KB,
patch
|
asaf
:
review+
|
Details | Diff | Splinter Review |
No other Mac applications use parentheses in untitled titles, and Firefox shouldn't either. The Apple HIG states that untitled windows in document based applications should use "untitled" then "untitled 2", 3, 4 etc. for additional windows:
http://developer.apple.com/Mac/library/documentation/UserExperience/Conceptual/AppleHIGuidelines/XHIGWindows/XHIGWindows.html#//apple_ref/doc/uid/20000961-CHDHBCGE
However, Firefox isn't _exactly_ a document based application but something inbetween, especially since there are both windows and tabs. I therefore think we should do as Safari and use "Untitled" for untitled tabs, as seen in the attached screenshot from Mac Safari 4.0.4.
Also attaching a patch to make this change.
Implementing "Untitled", "Untitled 2" etc. for additional empty windows should probably also be done, but that is a separate bug.
Assignee | ||
Updated•15 years ago
|
Assignee: nobody → markus.magnuson
Assignee | ||
Comment 2•15 years ago
|
||
This patch changes the title of empty tabs from "(Untitled)" to "Untitled" on Mac. If the same change is proper for all platforms, I could produce a much simpler two line-patch.
Attachment #432335 -
Flags: ui-review?(faaborg)
Attachment #432335 -
Flags: review?(mano)
Comment 3•15 years ago
|
||
Comment on attachment 432335 [details] [diff] [review]
Change empty tab title from "(Untitled)" to "Untitled" on Mac
Let's go with "New Tab" on all platforms.
Attachment #432335 -
Flags: ui-review?(faaborg) → ui-review-
Assignee | ||
Comment 4•15 years ago
|
||
This patch changes the empty tab title to "New Tab" on all platforms, as suggested by Alex in comment 3.
Attachment #432335 -
Attachment is obsolete: true
Attachment #432945 -
Flags: ui-review?(faaborg)
Attachment #432945 -
Flags: review?(joshmoz)
Attachment #432335 -
Flags: review?(mano)
Updated•15 years ago
|
OS: Mac OS X → All
Hardware: x86 → All
Summary: Change title of empty tabs from "(Untitled)" to "Untitled" somewhat according to Apple HIG → Change title of empty tabs from "(Untitled)" to "New Tab"
Attachment #432945 -
Flags: review?(joshmoz) → review?(mano)
Comment 5•15 years ago
|
||
Comment on attachment 432945 [details] [diff] [review]
Change empty tab title from "(Untitled)" to "New Tab" on all platforms
Great, to explain my rational a bit more, since the button is called "new tab" it only makes sense that the thing it creates is literally a "new tab". Limi pointed this out awhile ago as well.
Attachment #432945 -
Flags: ui-review?(faaborg) → ui-review+
Comment 6•15 years ago
|
||
Comment on attachment 432945 [details] [diff] [review]
Change empty tab title from "(Untitled)" to "New Tab" on all platforms
>--- a/browser/locales/en-US/chrome/browser/tabbrowser.dtd
>+++ b/browser/locales/en-US/chrome/browser/tabbrowser.dtd
>@@ -1,9 +1,9 @@
>-<!ENTITY untitledTab "(Untitled)">
>+<!ENTITY untitledTab "New Tab">
Bug 347930 removed this entity.
>--- a/browser/locales/en-US/chrome/browser/tabbrowser.properties
>+++ b/browser/locales/en-US/chrome/browser/tabbrowser.properties
>@@ -1,10 +1,10 @@
> tabs.loading=Loadingâ¦
>-tabs.untitled=(Untitled)
>+tabs.untitled=New Tab
You need to change the name for this so that localizers will pick it up.
Assignee | ||
Comment 7•15 years ago
|
||
Same patch as previously, but with the property "tabs.untitled" renamed to "tabs.emptyTabTitle" per comment 6.
Attachment #432945 -
Attachment is obsolete: true
Attachment #433161 -
Flags: review?(mano)
Attachment #432945 -
Flags: review?(mano)
Comment 8•15 years ago
|
||
Comment on attachment 433161 [details] [diff] [review]
Change empty tab title from "(Untitled)" to "New Tab" on all platforms
r=mano
Attachment #433161 -
Flags: review?(mano) → review+
Comment 9•15 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 3.7a4
Comment 10•15 years ago
|
||
Verified fixed using hourly build Mozilla/5.0 (Windows; U; Windows NT 6.1; en-US; rv:1.9.3a4pre) Gecko/20100318 Minefield/3.7a4pre
with changeset http://hg.mozilla.org/mozilla-central/rev/a352d0413476
Status: RESOLVED → VERIFIED
Comment 12•15 years ago
|
||
The following Litmus tests have been updated:
https://litmus.mozilla.org/show_test.cgi?id=10076
https://litmus.mozilla.org/show_test.cgi?id=10085
https://litmus.mozilla.org/show_test.cgi?id=11347
https://litmus.mozilla.org/show_test.cgi?id=11357
Flags: in-litmus+
You need to log in
before you can comment on or make changes to this bug.
Description
•