Do Not Allow Home Button To Be Dragged and Dropped Onto Itself

RESOLVED FIXED in seamonkey2.1b3

Status

SeaMonkey
UI Design
--
minor
RESOLVED FIXED
7 years ago
6 years ago

People

(Reporter: David E. Ross, Assigned: InvisibleSmiley)

Tracking

Trunk
seamonkey2.1b3

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments, 1 obsolete attachment)

(Reporter)

Description

7 years ago
User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.1.6) Gecko/20091206 SeaMonkey/2.0.1
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.1.6) Gecko/20091206 SeaMonkey/2.0.1

When I select the Home button, I sometimes get a popup asking if I want to set my home page.  I have already set my home page to be my bookmarks file (bookmarks.html).  

If I select the Set Home Page button on the popup, the result is that my bookmarks file remains my home page.  That is, the dialogue chooses the page to which I am going, not the page from which I am leaving.  

Reproducible: Sometimes




My Home button is on my Navigation toolbar.  I moved it there from my Personal toolbar, which I normally hide.
(Reporter)

Comment 1

7 years ago
Created attachment 420652 [details]
image of popup
(Reporter)

Comment 2

7 years ago
In a newsgroup thread in mozilla.support.seamonkey, Jens Hatlak noted that the Home button can be dragged from its toolbar.  I tested that and found that I can consistently recreate this problem if my mouse moves even slightly while I'm clicking on that button.  

As already noted, I moved the Home button from my Personal toolbar to my Navigation toolbar.  However, I just tested this with the Home button restored to my Personal toolbar.  This bug remains a problem in that case.  That is, even when it is on the Preferences toolbar, the Home button can be dragged slightly and will cause the popup seen in Attachment 420652 [details].  

No button should be dragged except while the Customize Toolbar window is open.  I'm not sure if this is still a bug for the Preferences component.   

By the way, I am using SeaMonkey Default Theme 1.0.
(Reporter)

Comment 3

7 years ago
Oops!  Not clear.  

When I indicated "I'm not sure if this is still a bug for the Preferences component.", I meant that I'm not sure if this falls within that component.  It might be a bug for SeaMonkey's Themes component or for Toolkit's Toolbars and Toolbar Customization component.
(Assignee)

Comment 4

7 years ago
I think it's a valid move to drag the Home button, e.g. to open a new tab at a certain position. There is little to no sense in dropping it on itself, though--the Home Page already is equal to its location, so even clicking OK in the dialog that currently appears in that situation won't change anything (or rather should, I didn't check the multi-page case).

David, if you agree, please morph this bug into requesting to remove the possibility of dropping the Home button on itself (adapt the summary).

Moving over to UI Design (still better than General I guess).
Component: Preferences → UI Design
OS: Windows XP → All
QA Contact: preferences → ui-design
Hardware: x86 → All
Version: unspecified → Trunk
(Reporter)

Comment 5

7 years ago
I agree.  Summary changed from "Asks to Set Home Page When It Is Already Set".  

The fix should not involve any error popup dialogue or require any user response.  Instead, dragging and dropping the button onto itself should simply be ignored.  Many users likely have an unsteady hand when selecting and clicking on a button.  

When this is fixed, it needs to be tested for Home buttons on both the Navigation and Personal toolbars.
Summary: Asks to Set Home Page When It Is Already Set → Do Not Allow Home Button To Be Dragged and Dropped Onto Itself
(Assignee)

Comment 6

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

Comment 7

6 years ago
(In reply to comment #4)
> I think it's a valid move to drag the Home button, e.g. to open a new tab at a
> certain position. There is little to no sense in dropping it on itself,
> though--the Home Page already is equal to its location, so even clicking OK in
> the dialog that currently appears in that situation won't change anything (or
> rather should, I didn't check the multi-page case).
IIRC the home button only drags the first page, so you'd lose data.

Comment 8

6 years ago
(In reply to comment #7)
> (In reply to comment #4)
> > I think it's a valid move to drag the Home button, e.g. to open a new tab at a
> > certain position. There is little to no sense in dropping it on itself,
> > though--the Home Page already is equal to its location, so even clicking OK in
> > the dialog that currently appears in that situation won't change anything (or
> > rather should, I didn't check the multi-page case).
> IIRC the home button only drags the first page, so you'd lose data.
On closer inspection, drops only change the first page too, so you don't.

Comment 9

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

I was wondering whether we should make it so that the button refuses to drop on itself (much as dragging a link from a page refuses to drop into the page).
(Assignee)

Comment 10

6 years ago
Created attachment 515519 [details] [diff] [review]
patch v2 [Checkin: comment 12]

(In reply to comment #9)
> I was wondering whether we should make it so that the button refuses to drop on
> itself (much as dragging a link from a page refuses to drop into the page).

I was trying something with a canDrop method, which was actually called but didn't work. Then I found the below site and tried setting aDragSession.canDrop, which didn't work either. Then I tried changing aDragSession.dragAction, which finally worked. Madness.

https://developer.mozilla.org/en/Drag_and_Drop
Attachment #515508 - Attachment is obsolete: true
Attachment #515519 - Flags: review?(neil)
Attachment #515508 - Flags: review?(neil)
Comment on attachment 515519 [details] [diff] [review]
patch v2 [Checkin: comment 12]

Maybe the other ways would have worked with the new style drag'n'drop.
Attachment #515519 - Flags: review?(neil) → review+
(Assignee)

Comment 12

6 years ago
Comment on attachment 515519 [details] [diff] [review]
patch v2 [Checkin: comment 12]

http://hg.mozilla.org/comm-central/rev/c2a8fe26bf32
Attachment #515519 - Attachment description: patch v2 → patch v2 [Checkin: comment 12]
(Assignee)

Updated

6 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.1b3
You need to log in before you can comment on or make changes to this bug.