Disable quadruple-click (quad-click) paragraph selection on the 1.8 branch

RESOLVED FIXED

Status

()

Core
Selection
RESOLVED FIXED
12 years ago
12 years ago

People

(Reporter: Uri Bernstein (Google), Assigned: Uri Bernstein (Google))

Tracking

({fixed1.8, useless-UI})

Trunk
PowerPC
Mac OS X
fixed1.8, useless-UI
Points:
---
Bug Flags:
blocking1.8b5 +

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

12 years ago
Currently, quadruple-click (on Mac, and possibly on Linux) attempts to select a
paragraph.
(http://bonsai.mozilla.org/cvsblame.cgi?file=/mozilla/layout/generic/nsFrame.cpp&rev=3.565#1581)
Sadly, the paragraph-selection code is completely broken, so usually just the
current line is selected, but in some situations (such as in bugzilla comments),
the current and previous lines are selected. This is confusing and unhelpful.

If and when the paragraph-selection code is fixed, it would probably be assigned
to triple-click (see bug 32807). For now, quadruple click should simply be disabled.

This is not an issue on Windows because the OS apparently does not pass on
n-click events (where n>2) to the application.
(Assignee)

Comment 1

12 years ago
Created attachment 187300 [details] [diff] [review]
patch

Disable quad-click. The very broken paragraph selection feature is still
accessable by setting browser.triple_click_selects_paragraph.
Assignee: selection → uriber
Status: NEW → ASSIGNED
Attachment #187300 - Flags: superreview?(roc)
Attachment #187300 - Flags: review?(roc)
Attachment #187300 - Flags: approval1.8b3?
I don't think we should disable this. If we disable it, it's less likely to ever
be fixed. And I'm not aware of any evidence that this is a significant problem.
Are you?

I think the right way to proceed is to fix paragraph selection and then in bug
32807 we may or may not decide to change the UI behavior.
(Assignee)

Comment 3

12 years ago
It's not a *significant* problem, but it's certainly noticable.
I totally agree that in the long run paragraph selection should be fixed (and I
even think it should be the default behaviour for triple-click). I'll even
probably try to fix it myself sometime. However,  I don't see this being fixed
for Firefox 1.1, and having quad-click behave like it does now is certainly
worse than nothing at all. I don't think disabling this will have much of an
effect on the chances for fixing paragraph selection (Windows users, still the
vast majority, can't use quad-click anyway).

Perhaps we should wait for the Forefox 1.1 branch, and then disable quad-click
on the branch only (and work on fixing paragraph selection on the trunk)?
(Assignee)

Comment 4

12 years ago
(In reply to comment #3)
> It's not a *significant* problem, but it's certainly noticable.

What I meant to write was:
It's not a *major* problem, but it's certainly noticable. (I gues that makes it
siginificant).
Comment on attachment 187300 [details] [diff] [review]
patch

Please do not request approval until reviews are complete.
Attachment #187300 - Flags: approval1.8b3?
Comment on attachment 187300 [details] [diff] [review]
patch

this is approved for branch only. Don't land it on the trunk, but you can land
it on the 1.8 branch once it's been created (and you have approval)
Attachment #187300 - Flags: superreview?(roc)
Attachment #187300 - Flags: superreview+
Attachment #187300 - Flags: review?(roc)
Attachment #187300 - Flags: review+
(Assignee)

Updated

12 years ago
Summary: Disable quadruple-click (quad-click) paragraph selection → Disable quadruple-click (quad-click) paragraph selection on the 1.8 branch
(Assignee)

Updated

12 years ago
Keywords: useless-UI
(Assignee)

Comment 7

12 years ago
Comment on attachment 187300 [details] [diff] [review]
patch

Requesting approval1.8b4: This is a branch-specific fix with minimal risk. It
disables a UI (quad-click) to a feature which is broken to the point of being
useless.
Attachment #187300 - Flags: approval1.8b4?
FWIW, clicking the mouse rapidly (within the double-click interval) gives the
following sequences of click counts:
  Windows: 1 2 3 2 3 2 3 2 3 ...
  GTK:     1 2 3 1 2 3 1 2 3 ...
  Mac:     1 2 3 4 5 6 7 8 9 ...
Did this work better in the past?  Or has it always been this broken?  Is it
broken to the point of not being useful?
(Assignee)

Comment 10

12 years ago
(In reply to comment #9)
> Did this work better in the past?  Or has it always been this broken?  Is it
> broken to the point of not being useful?

See bug 32807 for the history of paragraph-selection. As far as I understand,
this never worked properly. I'm currently working on making it work (for 1.9) -
you can see my WIP patches on that bug.

The brokenness is described in comment #0: It usually just selects the line
(i.e., behaves just like triple-click), but in some cases also selects the
previous line (which makes no sense). I havent seen a case where it actually
works (i.e. selects a paragraph).
Comment on attachment 187300 [details] [diff] [review]
patch

Fix the comment typo and a=drivers for 1.8b4.

/be
Attachment #187300 - Flags: approval1.8b4? → approval1.8b4+
Please remember to nominate the bug before or at the same time you nominate a
patch attached to the bug.  This helps us measure how well we're doing at bug
fixing more easily.  Thanks,

/be
Flags: blocking1.8b4+
Fix checked into MOZILLA_1_8_BRANCH only, marking this as fixed since it's a
branch-only sort of bug.

Checking in layout/generic/nsFrame.cpp;
/cvsroot/mozilla/layout/generic/nsFrame.cpp,v  <--  nsFrame.cpp
new revision: 3.574.2.1; previous revision: 3.574
done
Status: ASSIGNED → RESOLVED
Last Resolved: 12 years ago
Keywords: fixed1.8
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.