The default bug view has changed. See this FAQ.

Update Help for |Bug 568283 - Add checkbox for accessibility.browsewithcaret (F7-by-default mode)|

RESOLVED FIXED in seamonkey2.1b3

Status

SeaMonkey
Help Documentation
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: ewong, Assigned: InvisibleSmiley)

Tracking

(Blocks: 1 bug)

Trunk
seamonkey2.1b3
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 2 obsolete attachments)

(Reporter)

Description

6 years ago
Bug #568283 added a checkbox for accessibility.browsewithcaret (F7-by-default mode).

Help must be added for this.
(Reporter)

Updated

6 years ago
Depends on: 568283
(Reporter)

Comment 1

6 years ago
(In reply to comment #0)
> Bug #568283 added a checkbox for accessibility.browsewithcaret (F7-by-default
> mode).
> 
> Help must be added for this.

Addendum: When the bug is fixed.
(Assignee)

Updated

6 years ago
Summary: Need to add help topic for bug #568283. → Update Help for |Bug 568283 - Add checkbox for accessibility.browsewithcaret (F7-by-default mode)|
(Assignee)

Updated

6 years ago
Duplicate of this bug: 635806
(Assignee)

Updated

6 years ago
Blocks: 636993
(Assignee)

Comment 3

6 years ago
Merging bug 623901 (for bug 623889 which moved FAYT preferences to their own panel) with this one, since both can and should be done at the same time.
(Assignee)

Updated

6 years ago
Duplicate of this bug: 623901
(Assignee)

Updated

6 years ago
Depends on: 623889
(Assignee)

Comment 5

6 years ago
Created attachment 515348 [details] [diff] [review]
patch
Assignee: nobody → jh
Status: NEW → ASSIGNED
Attachment #515348 - Flags: review?(stefanh)

Comment 6

6 years ago
Comment on attachment 515348 [details] [diff] [review]
patch

(Bug 568283 hasn't landed yet, so I might not have catched everything)

You need to update the index page as well, search results for find as you type etc still points to the old pane. Please also update the index page with respect to caret browsing

+  <li><strong>Browse With Caret</strong>: Select if and how caret browsing
+    (ability to navigate and select within a page using the cursor keys and a
+    visible caret) should be used:

Please add a link to caret browsing in the glossary instead (and please also update the description in there since we now have more ways of enabling it)

+      checked, the F7 shortcut will toggle caret browsing oni or off.

Oops ;-)
Attachment #515348 - Flags: review?(stefanh) → review-
(Assignee)

Comment 7

6 years ago
Created attachment 515469 [details] [diff] [review]
patch v2
Attachment #515348 - Attachment is obsolete: true
Attachment #515469 - Flags: review?(stefanh)
Hardware: x86 → All
Version: unspecified → Trunk

Comment 8

6 years ago
Comment on attachment 515469 [details] [diff] [review]
patch v2

+  <li><strong>Browse With Caret</strong>: Select if and how <a
+    href="glossary.xhtml#caret_browsing">caret browsing</a> should be used:
+    <li><strong>Use caret browsing</strong>: If checked, caret browsing will
+      be enabled by default when you load a web page.</li>
+    <li><strong>Use the F7 shortcut to toggle caret browsing</strong>: If
+      checked, the F7 shortcut will toggle caret browsing on or off. Otherwise
+      &brandShortName; will not use the shortcut and the below option will have
+      no effect.</li>
+    <li><strong>Warn me before turning on caret browsing</strong>: If checked,
+      &brandShortName; will issue a warning dialog if the caret browsing mode
+      is about to be entered, allowing you to choose whether to proceed or not.
+      (There is no warning when leaving caret browsing mode.)</li>
+  </li>

You forgot to wrap the <li>'s under "Browse With Caret" in an <ul>

   <li><strong>Clear the current search after a few seconds of
     inactivity</strong>: If checked, cancels the search after a few seconds of
     keyboard inactivity.</li>

Mind fixing the second line to read "If checked, the search will be cancelled after a few.."?

+  <li><strong>Show the find toolbar during find as you type</strong>: If
+    checked, using Find As You Type will open the find toolbar if it is not
+    already open and fill in what you are typing. If unchecked, the find toolbar

Hmm, the "fill in what you are typing" sounds a bit strange. How about something like this:

"If checked, the find toolbar will open when Find As You Type is activated and text will be entered in the toolbar search field when you type."

+  text like you do in a text editor. You can turn caret browsing mode on or off
+  using the <kbd>F7</kbd> shortcut or on the Keyboard Navigation preferences

I think "key" sounded better and it should not be "on". Let me suggest this instead:

"You can turn caret browsing mode on or off by pressing the <kbd>F7</kbd> key. Caret browsing mode can also be enabled or disabled in Advanced Preferences - Keyboard Navigation."


Sorry, but this is growing a bit since "Searching Within a Page" and "Using Find-as-you-type" needs an update :-/
(Assignee)

Comment 9

6 years ago
Created attachment 516058 [details] [diff] [review]
patch v2a [Checkin: comment 13]

(In reply to comment #8)
> "If checked, the find toolbar will open when Find As You Type is activated and
> text will be entered in the toolbar search field when you type."

Tweaked it a bit.

> "You can turn caret browsing mode on or off by pressing the <kbd>F7</kbd> key.
> Caret browsing mode can also be enabled or disabled in Advanced Preferences -
> Keyboard Navigation."

And this (didn't want to think about which type of hyphen is correct here).

> Sorry, but this is growing a bit since "Searching Within a Page" and "Using
> Find-as-you-type" needs an update :-/

Sorry, it's late and I'm not sure I understand you; did I forget about something? Do we need to make more changes here?
Attachment #515469 - Attachment is obsolete: true
Attachment #516058 - Flags: review?(stefanh)
Attachment #515469 - Flags: review?(stefanh)

Comment 10

6 years ago
(In reply to comment #9)

> And this (didn't want to think about which type of hyphen is correct here).

We use "-" (I checked).

> 
> > Sorry, but this is growing a bit since "Searching Within a Page" and "Using
> > Find-as-you-type" needs an update :-/
> 
> Sorry, it's late and I'm not sure I understand you; did I forget about
> something? Do we need to make more changes here?

See nav_help.xhtml#578 and nav_help.xhtml#614
(Assignee)

Comment 11

6 years ago
(In reply to comment #10)
> (In reply to comment #9)
> 
> > And this (didn't want to think about which type of hyphen is correct here).
> 
> We use "-" (I checked).

If you want I can change it back, just let me know.

> > > Sorry, but this is growing a bit since "Searching Within a Page" and "Using
> > > Find-as-you-type" needs an update :-/
> > 
> > Sorry, it's late and I'm not sure I understand you; did I forget about
> > something? Do we need to make more changes here?
> 
> See nav_help.xhtml#578 and nav_help.xhtml#614

Those may refer to FAYT but not to Preferences, which this bug is about. I'm not going to fix the world here.

Comment 12

6 years ago
Comment on attachment 516058 [details] [diff] [review]
patch v2a [Checkin: comment 13]

+    the find toolbar will not be opened and the search string will be displayed
+    in the status bar instead.

From the context, I think it's clear that we don't have a toolbar, so you can skip "instead".

(In reply to comment #11)
> > We use "-" (I checked).
> 
> If you want I can change it back, just let me know.

Yes, please do.

> > See nav_help.xhtml#578 and nav_help.xhtml#614
> 
> Those may refer to FAYT but not to Preferences, which this bug is about.

Oh, that's not obvious.

> I'm not going to fix the world here.

OK, but please file a bug on the remaining bits.
Attachment #516058 - Flags: review?(stefanh) → review+
(Assignee)

Comment 13

6 years ago
Comment on attachment 516058 [details] [diff] [review]
patch v2a [Checkin: comment 13]

http://hg.mozilla.org/comm-central/rev/3f9ac1aeea35
with comment 12 addressed
Attachment #516058 - Attachment description: patch v2a → patch v2a [Checkin: comment 13]
(Assignee)

Updated

6 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.1b3
(Assignee)

Updated

6 years ago
Blocks: 638816
You need to log in before you can comment on or make changes to this bug.