Closed
Bug 217472
Opened 21 years ago
Closed 20 years ago
Side bar inconsistency: single-click bookmark vs double-click history item
Categories
(Firefox :: Bookmarks & History, defect, P2)
Tracking
()
VERIFIED
FIXED
Firefox1.0beta
People
(Reporter: pufiad, Assigned: mconnor)
References
Details
(Keywords: fixed-aviary1.0)
Attachments
(1 file, 1 obsolete file)
(deleted),
patch
|
p_ch
:
review+
asa
:
approval-aviary+
|
Details | Diff | Splinter Review |
User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.5a) Gecko/20030728 Mozilla Firebird/0.6.1
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.5a) Gecko/20030728 Mozilla Firebird/0.6.1
This is a inconsistency:
A bookmark (in the side bar) can be single-clicked to load, a history item (in
the side bar) must be double-clicked to load. I believe that a single click is
just fine for a history item as well.
Reproducible: Always
Steps to Reproduce:
Comment 1•21 years ago
|
||
Confirming with 20030826 build on WinXP
Status: UNCONFIRMED → NEW
Ever confirmed: true
Assignee | ||
Comment 2•21 years ago
|
||
if we're asking for History to change, then it should go to History
Assignee: pierre_tmp → blake
Component: Bookmarks → History
QA Contact: mpconnor → mozilla
Comment 3•21 years ago
|
||
Seeing in Linux as well
--> All
Definitely an annoyance
OS: Windows 2000 → All
Updated•21 years ago
|
Priority: -- → P2
Target Milestone: Firefox1.0 → Firefox1.0beta
Comment 4•21 years ago
|
||
Oh.
I would prefer single click to select, double click (or drag and drop)
to take action (Open).
I would change the bookmark behaviour, which would also match the
Bookmark Manager, but if I am in a minority of one ...
Ben.
Updated•21 years ago
|
Flags: blocking1.0?
Comment 6•21 years ago
|
||
(In reply to comment #5)
> indeed.
Indeed what? Are we standardising on single-click or double-click?
Updated•20 years ago
|
Severity: minor → major
Assignee | ||
Comment 8•20 years ago
|
||
we've moved to double click in other places. Single click makes it difficult
for some users to use context menus, so we should match bookmarks manager and
make it double click.
Assignee: firefox → p_ch
Component: History → Bookmarks
QA Contact: mozilla → mconnor
Changing my vote to double-click actually. It is better to be able to select
several items at once and to delete it.
Updated•20 years ago
|
Assignee: p_ch → bugs
Comment 10•20 years ago
|
||
standardize on single click across UI - matches IE, click action (load of URL)
should not be invoked when right clicking on links.
Assignee: bugs → varga
Comment 11•20 years ago
|
||
Updated•20 years ago
|
Attachment #152933 -
Flags: review?(bugs)
Comment 12•20 years ago
|
||
Shouldn't single or double click be determined by the user's OS settings?
I *HATE* double-clicking (it's a complete waste of time) so to be forced to do
it in certain parts of the interface is annoying.
Winzip has an option that checks whether double or single click has been set as
default for the OS, and then it just matches that setting, so everyone gets what
they want.
Personally I would like to see single click all the way. You don't double click
menus, or buttons, or web links, so why double click anywhere else? (unless it
is a secondary action). Of course there should be a double click method as an
option (for people who want it) but it should not be made compulsory.
Comment 13•20 years ago
|
||
P.S. Internet Explorer is far more consistent in this regard.
It's History and Favorites side bars work in exactly the same way as the rest of
the interface. There is no need to keep switching back and forth between
different methods based on where you are in the program.
Comment 14•20 years ago
|
||
fixed on the trunk
Comment 15•20 years ago
|
||
sorry, wrong bug
Comment 16•20 years ago
|
||
The bookmarks sidebar should use a single click mouse action. Using a dubble
click mouse action would imply that the "bookmarks Toolbar" should work in the
same way. Which is not right.
The Bookmarks manager doesn't have to be consistent with the rest because it is
designed to manage bookmarks. The bookmarks sidebar, the bookmarks toolbar and
the drop down menu (Bookmarks) are all designed to access bookmarks. These three
have to be consistent (single click).
The history sidebar should have the same behavior as the bookmarks sidebar
(single click).
Comment 17•20 years ago
|
||
Isn't this a history sidebar bug and not a bookmarks bug?
Component: Bookmarks → History
Comment 18•20 years ago
|
||
(In reply to comment #17)
> Isn't this a history sidebar bug and not a bookmarks bug?
Yes, it is a history sidebar bug
Assignee | ||
Comment 19•20 years ago
|
||
Comment on attachment 152933 [details] [diff] [review]
fix
this leaves out changes that should happen, as well as opens up some oddball
side effects. Better patch coming.
Attachment #152933 -
Flags: review?(bugs) → review-
Assignee | ||
Comment 20•20 years ago
|
||
fixed Bug 255027 by not showing a context menu if no items are selected.
supports all Ctrl/Shift/Ctrl+Shift key modifiers in the same way as bookmarks
sidebar does.
Prevents clicking on a twisty from triggering a load event (one of the problems
from the previous patch).
Supports middle-clicking to open in a new tab etc, including the shift modifier
to flip the foreground/background action.
removes the select all menuitem, and a fair bit of unneeded code.
Assignee | ||
Updated•20 years ago
|
Attachment #157619 -
Flags: review?(bugs)
Comment 21•20 years ago
|
||
I don't think that disabling the context menu if no item is selected is a good
idea. That's not what is done in the bookmarks panel. It may be used to delete
or bookmark single items and delete all the content of a folder.
And the same for select all. I am not sure if the result of deleting all the
selection work correctly in every view, but it is also really useful. I know
people that never open the preference dialog...
Assignee | ||
Comment 22•20 years ago
|
||
we do odd things if there's no selection, but there's a patch to suppress the
context menu if there's no selection.
select all doesn't coexist with a single-selection tree. After this patch,
users can still delete entire days/sites. If they're relying on the history
panel to protect their privacy, they're ignoring cookies, saved form info, and
saved passwords, so its a false economy by not pushing them into the privacy panel.
Comment 23•20 years ago
|
||
Comment on attachment 157619 [details] [diff] [review]
better patch
since IE doesn't provide a select All option, this patch should be fine.
A bit of refactoring with the functions handleHistoryClick, OpenURLLinkIn and
OpenURL may be nice to do, but shouldn't block this patch.
r=me if you change OpenURL to openURL and bonus if you change also
OpenURLLinkIn to openURLLinkIn.
Attachment #157619 -
Flags: review?(bugs) → review+
Assignee | ||
Comment 24•20 years ago
|
||
Comment on attachment 157619 [details] [diff] [review]
better patch
I'll make the changes to the casing as noted by Pierre on checkin, but this
should really go in before PR.
Attachment #157619 -
Flags: approval-aviary?
Comment 25•20 years ago
|
||
Comment on attachment 157619 [details] [diff] [review]
better patch
a=asa for aviary checkin
Attachment #157619 -
Flags: approval-aviary? → approval-aviary+
Assignee | ||
Updated•20 years ago
|
Keywords: fixed-aviary1.0
Comment 26•20 years ago
|
||
confirmed fixed on Windows Firefox Branch 2004-09-10-08-0.9 and Mac Firefox
Branch 2004-09-10-07-0.9
Comment 27•20 years ago
|
||
Just to inquire, it's intentional that you can't select more than two entries in
the sidebars now (via click & Shift+Down or Ctrl-click & Ctrl-click; like you
can do for select multiple fields), right?
Assignee | ||
Comment 28•20 years ago
|
||
that's correct. Multiple selection only works right with double-click default
actions, not single click.
Comment 29•20 years ago
|
||
Is the lack of multiple selection due to a limitation of XUL apps (and how they
handle click detection)? or was it a design decision?
Comment 30•20 years ago
|
||
I fear that with these changings the history sidebar loses the functionality to
select and delete multiple items. Or is this functionality kept?
(This is why I have submitted bug 257411.)
Assignee | ||
Comment 31•20 years ago
|
||
(In reply to comment #29)
> Is the lack of multiple selection due to a limitation of XUL apps (and how they
> handle click detection)? or was it a design decision?
Design decision, absolutely. We can do lots with clicks. Click, then
shift-click, would result in loading the page, then deleting it from history
(since that's the only useful thing that multiple selection did here). I guess
you could ctrl-click, then shift-click, but that's really unintuitive and not
how trees should work with single-click action.
Plus, if we're going for consistency, respecting the same click modifiers in the
same way as the bookmarks sidebar and bookmarks themselves.
Comment 32•20 years ago
|
||
Please add in the ability to revert to the old style history sidebar behaviour.
I like to be able to delete multiple items from my history and hitting delete 20
times is absurd.
Also it's very annoying for it to respond to single click as I tend to left
click before right click(and many others do too!) and it is a pain having to hit
back everytime.
Perhaps assign ctrl+middle click and shift+middle click to what was the old
behaviour with the left click?
Or perhaps even use the arror keys to change which item is 'in focus' so to
speak and use space to keep it selected?
Eg. I use the arror keys to navigate along the tree and hit space on the ones I
want deleted, when I'm finished I can then hit delete, or just use shift+space
to select the first and last of a range to be selected?
Multiple selections are also good for when someone wants to open multiple items
from the history in multiple tabs.
Basically just *some* method of getting a multiple selection.
Comment 33•20 years ago
|
||
(In reply to comment #32)
> Please add in the ability to revert to the old style history sidebar behaviour.
> I like to be able to delete multiple items from my history and hitting delete 20
> times is absurd.
> Also it's very annoying for it to respond to single click as I tend to left
> click before right click(and many others do too!) and it is a pain having to hit
> back everytime.
> Perhaps assign ctrl+middle click and shift+middle click to what was the old
> behaviour with the left click?
> Or perhaps even use the arror keys to change which item is 'in focus' so to
> speak and use space to keep it selected?
>
> Eg. I use the arror keys to navigate along the tree and hit space on the ones I
> want deleted, when I'm finished I can then hit delete, or just use shift+space
> to select the first and last of a range to be selected?
>
> Multiple selections are also good for when someone wants to open multiple items
> from the history in multiple tabs.
>
> Basically just *some* method of getting a multiple selection.
I second this request. I really miss being able to select specific entries and
delete them all together.
Comment 34•20 years ago
|
||
Agree with Leo and Jeff.
The best solution is to make it act like on KDE Linux. If someone holds ctrl on
keyboard, then he should be able to select more than one link.
If it is not possible to get this until 1.0 then users should be at least able
to revert to old behaviour, or probably old behaviour should be default. Maybe
the *best solution* is to switch to old behaviour and resolve user cofusion with
visual feedback - pointer over bookmarks won't be like pointer over history
(part of bug 258510).
Unfortunatly, there is no history manager, and you can't play with this.
Assignee | ||
Comment 35•20 years ago
|
||
that WAS our behaviour. We had a choice between two incompatible (and not
easily interchangeable) designs, and we collectively decided that this behaviour
was more consistent and appropriate within the greater aspect of the app.
We're not going to add a bunch of conditional code to make it an either/or
situation. If anything, I would suggest that it would be relatively trivial to
hook up an extension that would provide the old behaviour, simply by getting the
older versions of history.js and history-panel.xul and renaming/packaging them.
Comment 36•20 years ago
|
||
I have to agree with Mike's comment #35, the devs really should not be spending
their time putting inconsistency into the interface (an especially not when they
have just gone through the process of removing it).
I am all for spending time fixing interface issues that break the conventions of
the Operating System on which it is run (or which deviates from the design
standards set for the program itself) but not if such changes break consistency
or adds unnecessary complexity.
Of course, it is perfectly easy to have multiple file selection *AND* single
click activation at the same time (this is how I have my entire Windows OS set
up) and I filed a bug about this very issue 2 months ago (Bug 251910). However,
I don't know if XUL has such capabilities.
That is why I asked if the current method of Sidebar single-click was by design
or due to XUL limitations (comment #29) and I was told that the current
behaviour was very much by design. So, given that this is entirely intended
behaviour, I did not bother to mention my single-click multiple selection bug.
My preferred solution (which would solve ALL such problems like this in one go)
is described by Ivo Marques Martins in his Bug 226256. But I don't know if XUL
is capable of doing this.
Even if the current Sidebar implementation is to be kept exactly as it is now,
then I would suggest that at least the bookmarks should change visually upon
mouseover (as this is standard behaviour for anything that is activated by a
single click). You only need to look through the rest of Firefox to see this
principal in action.
Comment 37•20 years ago
|
||
I'm finding the single-click to launch bookmark from side bar incredibly
frustrating. Frequently I find myself clicking on a bookmark with expection
that I will highlight it (select it) for a subsequent operation such as
dragging, getting properties. Also related, is a bug a just filed, in which
dragging a bookmark in the sidebar has the side-effect of launching it after the
drag is complete. I believe this is all due to launch by single-click. Can't
this be a preference.
Thanks.
Comment 38•20 years ago
|
||
I'm finding the single-click to launch bookmark from side bar incredibly
frustrating. Frequently I find myself clicking on a bookmark with expection
that I will highlight it (select it) for a subsequent operation such as
dragging, getting properties. Also related, is a bug a just filed, in which
dragging a bookmark in the sidebar has the side-effect of launching it after the
drag is complete. I believe this is all due to launch by single-click. Can't
this be a preference.
Thanks.
Comment 39•20 years ago
|
||
in addition comment #30
> I fear that with these changings the history sidebar loses the functionality to
> select and delete multiple items. Or is this functionality kept?
> (This is why I have submitted bug 257411.)
Please have a look at Bug 261622. I think this would be a better solution for
such problems than mutilating Firefox.
Comment 40•20 years ago
|
||
(In reply to comment #39)
(comment #10 says that this is only be done to behave like IE.)
Comment 41•20 years ago
|
||
Wow! 40 comments ...
See my comment 4 and the vote in comment 9. See also comment 35 - have
we started something that is hard to stop? See comment 36: it should go
without saying that objects with single-click behaviour should have
have hover or mouse-down feedback.
We were only asking for standardising or harmonising. The request for
single click to select and double click to open was a perfectly
reasonable one.
We now have single click to open, that is the objects in the sidebar
act as buttons rather than icons.
The stimulus for this (comment 10) was to match IE, though other reasons
exist.
This use of button-behaviour - single click for action denies:
multiple selection (comment 28) - we do not expect to select
multiple buttons
deletion (comment 30) - buttons have only one action, in this case open
select before action (comment 32) - we don't select buttons we click them
drag and drop (comment 37) - we don't expect to be able to edit buttons
in browse mode
I am sorry I come across like a sore loser, so I repeat, if I am in a
minority of one, fair dos; but as others have requested please also handle
bug 226256 and bug 251910 .
Comment 42•20 years ago
|
||
The Seamonkey equivalent of this bug report was bug 20455, which includes
(gasp!) actual usability test results. Those who do not learn from history
are condemned to repeat it ...
Comment 43•20 years ago
|
||
Sorry to spam you all with add an additional comment when I have already stated
my case earlier (in Comment #36), but I do have an important point that I would
like to make on this topic.
There is constant talk of "single click" this and "double click" that, rather
than using the correct concepts of "selection" and "activation". By wrongly
using terms like "double click" (when in fact they actually mean "activate") it
causes untold confusion because it implies that the method of 2 clicks is going
to be forced upon ALL users (even those that have their OS set to single-click
activation).
Invariably it is "double-clickers" who are the ones that are so lax and
inaccurate with their terminology (see Bug 229217). A single-click user would
never use the phase "double-click" to mean 'launch' or 'activate', because they
never actually double-click anything (except when forced to do so in something
like the Windows System Tray).
I have just looked at the SeaMonkey bug 20455 (quoted in Comment #42) and it's
full of squabbling over exactly how many clicks should or should not be used in
each individual panel. This argument would have been greatly simplified if
inveterate double-clickers did not keep assuming that everyone uses the same
click activation system that they do. By sticking to the concepts of selection
and activation, they could have concentrated on how the panels should actually
function (and leave aside the issue of the actual number of clicks to be dealt
with by the OS).
As I said in my Bug 251910, it is perfectly possible to have single-click
activation and still have multiple file selection that works in a way that both
single-click and double-click users can use (without either having to change the
selection behaviour that they are familiar with).
Comments (in bug 20455) such as "there may be cases like Bookmarks where
double-click is the correct answer" are, in my view, totally wrong. Double click
is *NEVER* the correct answer on a system where the OS has been set to
single-click activation. It's just as wrong as forcing English dialogs on an OS
that is set to a different language.
It would be like using the terms "left click" and "right click" literally
(instead of 'primary mouse button' and 'secondary mouse button'). If you take
these terms literally and then made a program so that you HAD to "right click"
in certain places, then it will make that program very user hostile for
left-handed people (who have their mouse buttons swapped so that their right
button invokes the primary action). So the issue is not left button / right
button, but primary / secondary. Likewise the issue is not 'single click' or
'double click', but 'select' or 'activate'.
That is why I voted for Bug 226256 which, if it were possible to implement,
would solve this problem for everybody once and for all.
Comment 44•20 years ago
|
||
Since sidebar history and bookmarks are now styled like links (blue, underlined,
single-click) they should take the typical link (hand) cursor on hover. This is
what IE does, and it makes good sense. (see comment 13)
It would also (IMHO) be more readable if the links had no underline, and applied
the underline on hover. That's seems to be the more "contemporary" link style
(doing something on hover, underline or color). Doesn't really need to be blue
either. Maybe black, then blue with underline on hover.
Comment 45•20 years ago
|
||
I totally agree with Comment #43.
Firefox does not hardcode left / right clicks and force end users to use the
mouse buttons that the programmers have explicitly specified. Instead, anyone
can go to Control Panel, swap their mouse buttons and Firefox will
automatically respond and match those swapped button settings throughout the
application. But if you use the exact same Control Panel mouse dialog to switch
between single and double click, then Firefox just ignores any change in this
setting. This is because Firefox programmers are concentrating on specifying
the actual number of clicks that should be used in certain places within the
application, rather than dealing with selection and activation (as mentioned in
the comment above).
The result is that Firefox totally ignores and over-rides the user's chosen
click activation settings (which is very bad).
I therefore vote for Bug 226256. Let each user's Operating System settings
determine how many clicks are needed to select and launch items within Firefox.
Comment 46•20 years ago
|
||
I noticed (and so did Tracy and Marcia) that ctrl+clicking in the history
sidebar now does non-continguous selection. is this expected?
in addition, ctrl+doubleclicking no longer opens history sidebar items in a new
tab --is this also expected?
Assignee | ||
Comment 47•20 years ago
|
||
yeah, expected as in Ben changed the modifiers to do multi-selection instead of
supporting the click modifiers that were implemented to match the bookmarks
sidebar. I'm not a big fan of this step.
Comment 48•20 years ago
|
||
I am a huge fan of Ben's modifier patch (as are many other people). This is
definitely a step in the right direction and much better than the ugly horrible
mess that was implemented previously (and which was thankfully backed out after
much user protest).
Assignee | ||
Comment 49•20 years ago
|
||
if the point was to make the sidebars behave consistently, then its a step in
the wrong direction. Single-click activation + multiple selection is pretty ugly.
If you really want history management then the extension that brings back
history manager from seamonkey is a vast vast improvement in terms of power and
usability. I've seen relatively little user protest, perhaps you can point me
to where this large-scale negative feedback came from.
Comment 50•20 years ago
|
||
The History functionality in FireFox is what turned me off from FireFox back to
Mozilla. I find it almost unusable; it is a complete step back from Mozilla.
Single-click activation is very unintuitive (not only for me). It is not a menu
which you open temporarily and expect to close immediately after selection. It
is a window which you open to work with, make a selection or multiple
selections, then make an action (activate/delete/copy etc.). I sometimes want to
select an item and change sorting/grouping criteria. I often want to select item
and go up and down with arrow keys. Bookmarks should also behave the same way.
The design goal to follow IE is not convincing: why go from good to bad?
Not possible to search by location. Many pages have the same title and it’s
impossible to find the right page. In addition, I often remember the location
(or part of it) and inability to search by location makes History almost
unusable to me. (I always search History by Location, never by Title)
I would prefer to have History in the separate window (or to have an option to
do so). I don’t like to open/close sidebar and every time.
In general, the design decisions in Mozilla (not only with History) were much
better.
Assignee | ||
Comment 51•20 years ago
|
||
You can do multi-select now, if you click to the left of the tree item, it does
selection, not activation.
If you want something a little more advanced/heavy-duty, there's an Enhanced
History Manager extension out there that does a much better job of "managing"
history. The sidebar panels are generally unsuited for any advanced
management-style interface, and we're discussing what we can do for 1.5 to
improve history management, or if history "management" is a generally
little-used feature and should be relegated to an extension.
Resolving, the trunk merge will get this.
Status: ASSIGNED → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
Comment 52•20 years ago
|
||
(In reply to comment #51)
> You can do multi-select now, if you click to the left of the tree item, it does
> selection, not activation.
I don't think this should be resolved. You can multiple select with the trick
above, but then you can't get a contextual menu, so you can't really do anything
with that selection (hitting the delete key doesn't do anything either). And
having different behavior when clicking on the icon/name and the empty space
next to the icon/name doesn't seem very usable.
I still think this should be a 1.0 blocking bug, but if you want to put this off
for post 1.0, fine. But you shouldn't resolve it.
(comments in bug 259658 brought me here)
Comment 53•20 years ago
|
||
verified.
This bug was meant only for the single-click vs double-click consistency of
sidebar items across bookmarks and history.
If needed, please open bugs for other items that creeped in here.
Status: RESOLVED → VERIFIED
Comment 54•19 years ago
|
||
(In reply to comment #53)
> verified.
>
> This bug was meant only for the single-click vs double-click consistency of
> sidebar items across bookmarks and history.
I hear claim that you now need to double click in the history pane.
But in 1.5b1, on linux, if I single click, and try to do a multi-selection, then
I get a window opening up on me with the url that was last clicked. This is
*definitely* contrary to my window manager policy, and makes deleting multiple
entries impossible.
Did I read something wrong, or has the behaviour reverted back to the annoying
single click behaviour, without this bug being updated?
Component: History → Bookmarks & History
QA Contact: mconnor → bookmarks
You need to log in
before you can comment on or make changes to this bug.
Description
•