Document new tab behaviour under Win when starting SM with an instance already running

RESOLVED FIXED

Status

SeaMonkey
Help Documentation
--
trivial
RESOLVED FIXED
12 years ago
12 years ago

People

(Reporter: Giacomo Magnini, Assigned: Giacomo Magnini)

Tracking

({fixed1.8})

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments, 3 obsolete attachments)

(Assignee)

Description

12 years ago
User-Agent:       Mozilla/5.0 (X11; U; Linux i686; it-IT; rv:1.7.12) Gecko/20050922 Fedora/1.0.7-1.1.fc4 Firefox/1.0.7
Build Identifier: 

As a reference, see bug 310803.

Reproducible: Always

Steps to Reproduce:




Patch coming after some discussion on IRC with Neil.
(Assignee)

Comment 1

12 years ago
Created attachment 202222 [details] [diff] [review]
Patch as suggested by Neil on IRC. Not sure if using a class="win" will show also for OS/2, which has the same behaviour.
(Assignee)

Comment 2

12 years ago
Comment on attachment 202222 [details] [diff] [review]
Patch as suggested by Neil on IRC. Not sure if using a class="win" will show also for OS/2, which has the same behaviour.

Ok, class="win" will work on OS/2, too.
Attachment #202222 - Flags: review?(iann_bugzilla)

Comment 3

12 years ago
Comment on attachment 202222 [details] [diff] [review]
Patch as suggested by Neil on IRC. Not sure if using a class="win" will show also for OS/2, which has the same behaviour.

+<p>&brandShortName

You forgot the ";" ;)
(Assignee)

Comment 4

12 years ago
(In reply to comment #3)
> You forgot the ";" ;)

Twice! A 100% error ratio! ;) Fixed locally.

Comment 5

12 years ago
(In reply to comment #4)
> (In reply to comment #3)
> > You forgot the ";" ;)
> 
> Twice! A 100% error ratio! ;) Fixed locally.
> 

I count three! :-P
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true

Updated

12 years ago
Assignee: help.viewer → giacomo.magnini
Status: ASSIGNED → NEW

Comment 6

12 years ago
Comment on attachment 202222 [details] [diff] [review]
Patch as suggested by Neil on IRC. Not sure if using a class="win" will show also for OS/2, which has the same behaviour.

>Index: extensions/help/resources/locale/en-US/cs_nav_prefs_navigator.xhtml
>===================================================================
>+<div class="win">
>+<p>&brandShortName will also use your external link preference to determine
>+  where to load your home page if you try to open &brandShortName while it is
>+  already open.</p>
Missing ; twice. Is it worth pointing out where in the preferences you change your "external link preference"? I do not currently see a preference with this name.

>+
>+<p>To ensure that &brandShortName opens a new window, select the Navigator
>+  option in <a href= "cs_nav_prefs_appearance.xhtml#appearance>Appearance
>+  Preferences - Appearance</a>.</p>
>+</div>
>+
Missing ; again.
Attachment #202222 - Flags: review?(iann_bugzilla) → review-
(Assignee)

Comment 7

12 years ago
(In reply to comment #5)
> I count three! :-P

Ouch! :)
(Assignee)

Comment 8

12 years ago
Created attachment 203126 [details] [diff] [review]
Verified under Win that the preference in question was "Links from other applications"
Attachment #202222 - Attachment is obsolete: true
Attachment #203126 - Flags: review?(iann_bugzilla)

Comment 9

12 years ago
I cannot reproduce the behaviour you are commenting on. I have a browser window open and my "Links from other applications" set to new tab in the current window but when i click on SM it opens a new window not a new tab. I get a similar issue with current tab/window option.

Comment 10

12 years ago
Comment on attachment 203126 [details] [diff] [review]
Verified under Win that the preference in question was "Links from other applications"

>Index: extensions/help/resources/locale/en-US/cs_nav_prefs_navigator.xhtml
>===================================================================
>RCS file: /cvsroot/mozilla/extensions/help/resources/locale/en-US/cs_nav_prefs_navigator.xhtml,v
>retrieving revision 1.30
>diff -u -8 -p -r1.30 cs_nav_prefs_navigator.xhtml
>--- extensions/help/resources/locale/en-US/cs_nav_prefs_navigator.xhtml	6 Nov 2005 22:21:19 -0000	1.30
>+++ extensions/help/resources/locale/en-US/cs_nav_prefs_navigator.xhtml	15 Nov 2005 15:39:45 -0000
>@@ -474,16 +474,26 @@
>       <li><strong>A new tab in the current window</strong>: Open the linked
>         page in a new tab instead of a new window.</li>
>       <li><strong>A new window</strong>: Open the linked page in a new window.
>         (This is the default).</li>
>     </ul>
>   </li>
> </ul>
> 
>+<div class="win">
>+<p>&brandShortName; will also use your Links from other applications preference
>+  to determine where to load your home page if you try to open &brandShortName;
>+  while it is already open.</p>
From what Neil explained to me on IRC, it is not as simple as this:

Neil	when you launch mozilla, it opens windows based on command-line parameters, if there are any
Neil	if none of your command-line parameters open windows, it then looks at your appearance preferences (trunk only - this step is skipped on branch if you have any parameters whether or not they open windows)
Neil	if you have no appearence preferences either, then if you have a home page group then that gets opened in a new window
Neil	otherwise if you have a single home page then it follows your external link preference
Neil	(again, branch always opens the home page in a new window)

I would say the minimum you need to mention is about "Links from other applications" only being effective if you don't have a group of pages as you homepage AND you have no appearance prefs set.
Attachment #203126 - Flags: review?(iann_bugzilla) → review-
(Assignee)

Comment 11

12 years ago
Created attachment 205387 [details] [diff] [review]
Is this any better?
Attachment #203126 - Attachment is obsolete: true
Attachment #205387 - Flags: review?(iann_bugzilla)

Comment 12

12 years ago
Comment on attachment 205387 [details] [diff] [review]
Is this any better?

>Index: extensions/help/resources/locale/en-US/cs_nav_prefs_navigator.xhtml
>===================================================================
>+<div class="win">
>+  <p>To ensure that &brandShortName; opens a new window, select the Navigator
>+    option in <a href= "cs_nav_prefs_appearance.xhtml#appearance>Appearance
>+    Preferences - Appearance</a>.</p>
>+
>+  <p>If you want &brandShortName; to open a new tab instead of a new window when
>+    you launch it and it is already running, ensure the following:
>+
>+  <ul>
>+    <li>If you launch &brandShortName; by command line, do not use command-line
>+      parameters that open windows.</li>
>+    <li>Set the Navigator option only, as described above.</li>
As far as I am aware, from testing, you have to make sure nothing is checked in the appearance preference.

>+    <li>Set Links from other applications preference to A new tab in the current
>+      window.</li>

Do we not put quotes round a preference value or something similar?
Attachment #205387 - Flags: review?(iann_bugzilla) → review-
(Assignee)

Comment 13

12 years ago
Created attachment 205539 [details] [diff] [review]
Another attempt. (Checked in to trunk/branch)

- Used <q></q> around options
- Modified to nothing checked in Appearance
Attachment #205387 - Attachment is obsolete: true
Attachment #205539 - Flags: review?(iann_bugzilla)

Updated

12 years ago
Component: Help Viewer → Help
Product: Documentation → Mozilla Application Suite

Updated

12 years ago
Attachment #205539 - Flags: review?(iann_bugzilla)
Attachment #205539 - Flags: review+
Attachment #205539 - Flags: approval-seamonkey1.0+

Updated

12 years ago
Attachment #205539 - Attachment description: Another attempt. → Another attempt. (Checked in to trunk/branch)

Comment 14

12 years ago
Checked in (trunk/branch) by Standard8.
Status: NEW → RESOLVED
Last Resolved: 12 years ago
Keywords: fixed1.8
Resolution: --- → FIXED

Comment 15

12 years ago
Comment on attachment 205539 [details] [diff] [review]
Another attempt. (Checked in to trunk/branch)

The XML parser doesn't like this new version.

>+<div class="win">
>+  <p>To ensure that &brandShortName; opens a new window, select the Navigator
>+    option in <a href= "cs_nav_prefs_appearance.xhtml#appearance>Appearance
>+    Preferences - Appearance</a>.</p>

You need a " after "cs_nav_prefs_appearance.xhtml#appearance

>+  <p>If you want &brandShortName; to open a new tab instead of a new window when
>+    you launch it and it is already running, ensure the following:

You need to close this paragraph either here or after the <ul>.

>+  <ul>
>+    <li>If you launch &brandShortName; by command line, do not use command-line
>+      parameters that open windows.</li>
>+    <li>Nothing is set in the
>+      <a href= "cs_nav_prefs_appearance.xhtml#appearance>Appearance Preferences
>+      - Appearance</a>.</li>

You need a " after "cs_nav_prefs_appearance.xhtml#appearance
(Assignee)

Comment 16

12 years ago
Created attachment 206228 [details] [diff] [review]
Follow up patch. Will need this also on branch (checked in to trunk/branch).
Attachment #206228 - Flags: review?(iann_bugzilla)
Attachment #206228 - Flags: approval-seamonkey1.0?

Updated

12 years ago
Attachment #206228 - Flags: review?(iann_bugzilla)
Attachment #206228 - Flags: review+
Attachment #206228 - Flags: approval-seamonkey1.0?
Attachment #206228 - Flags: approval-seamonkey1.0+

Comment 17

12 years ago
Comment on attachment 206228 [details] [diff] [review]
Follow up patch. Will need this also on branch (checked in to trunk/branch).

Checked in on trunk and MOZILLA_1_8_BRANCH with a=SeaMonkey Council over IRC.

Updated

12 years ago
Attachment #206228 - Attachment description: Follow up patch. Will need this also on branch. → Follow up patch. Will need this also on branch (checked in to trunk/branch).

Comment 18

12 years ago
Checked in on MOZILLA_1_8_0_BRANCH.
You need to log in before you can comment on or make changes to this bug.