Closed Bug 372773 Opened 14 years ago Closed 13 years ago
Tab favicon should have the grab cursor on hover
User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:126.96.36.199) Gecko/20070219 Firefox/188.8.131.52 Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:184.108.40.206) Gecko/20070219 Firefox/220.127.116.11 Tabs can be dragged into bookmarks the same way the urlbar favicon is, but only the urlbar favicon has the grab cursor indicating draggability. The tab favicon should have the grab cursor too. The URL is a Stylish userstyle to implement this. Reproducible: Always Steps to Reproduce: 1. 2. 3.
See Bug 349130.
We had some builds of Bon Echo working that way for a while but decided to pull it before the release. If I recall correctly, the decision had to do with the hand cursor not really symbolizing dragging an object rather than dragging, for example a page to move around, but it would be better to be consistent.
(In reply to comment #2) > We had some builds of Bon Echo working that way for a while but decided to pull > it before the release. That will probably have to change, in case we remove the favicon from the location bar.
Assignee: nobody → dao
Status: NEW → ASSIGNED
Attachment #266040 - Attachment description: patch → patch, icon only
Comment on attachment 266345 [details] [diff] [review] patch, whole tab Not on the whole tab - the primary interaction for tabs is selection, not dragging/moving.
Attachment #266345 - Flags: ui-review?(beltzner) → ui-review-
Attachment #266040 - Flags: ui-review?(beltzner)
Comment on attachment 266040 [details] [diff] [review] patch, icon only I can't decide if this is weird or not. Might end up asking to back it out, but let's take a peek.
Attachment #266040 - Flags: ui-review?(beltzner) → ui-review+
Comment on attachment 266040 [details] [diff] [review] patch, icon only Looks good, but why are you changing the opacity?
Attachment #266040 - Flags: review?(enndeakin) → review+
I'm not changing the opacity, as there's no way how those elements could end up with an opacity below 1. I suppose those rules are left over from older (experimental?) theme versions.
Checking in toolkit/themes/pinstripe/global/browser.css; /cvsroot/mozilla/toolkit/themes/pinstripe/global/browser.css,v <-- browser.css new revision: 1.23; previous revision: 1.22 done Checking in toolkit/themes/winstripe/global/browser.css; /cvsroot/mozilla/toolkit/themes/winstripe/global/browser.css,v <-- browser.css new revision: 1.34; previous revision: 1.33 done
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Is this a bug on trunk? Bug 339964 was checked in. chrome://global/skin/browser.css doesn't exist. I feel that this patch is invalid.
So yes it looks like this patch should have been updated for tabbrowser changes. For some reason the toolkit version of browser.css still exists hence the patch applying cleanly. Dão, can you confirm and I'll back this out and check in a new patch against the browser skin instead.
Ah, yes. (No need to back out though.)
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Attachment #266040 - Attachment is obsolete: true
Checking in browser/themes/pinstripe/browser/browser.css; /cvsroot/mozilla/browser/themes/pinstripe/browser/browser.css,v <-- browser.css new revision: 1.66; previous revision: 1.65 done Checking in browser/themes/winstripe/browser/browser.css; /cvsroot/mozilla/browser/themes/winstripe/browser/browser.css,v <-- browser.css new revision: 1.78; previous revision: 1.77 done
Status: REOPENED → RESOLVED
Closed: 13 years ago → 13 years ago
Resolution: --- → FIXED
in-litmus+ http://litmus.mozilla.org/show_test.cgi?id=4699 Verified FIXED using Mozilla/5.0 (Windows; U; Windows NT 6.0; en-US; rv:1.9a9pre) Gecko/2007092705 Minefield/3.0a9pre
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.