Closed
Bug 627207
Opened 13 years ago
Closed 13 years ago
"Text" cursor instead of "Move" in the top of tab group
Categories
(Firefox Graveyard :: Panorama, defect, P4)
Firefox Graveyard
Panorama
Tracking
(Not tracked)
VERIFIED
FIXED
Firefox 4.0b12
People
(Reporter: jk1700, Assigned: unusualtears)
References
Details
(Whiteboard: [visual][polish][good first bug])
Attachments
(2 files, 2 obsolete files)
12.93 KB,
image/jpeg
|
Details | |
2.39 KB,
patch
|
Details | Diff | Splinter Review |
User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:2.0b10pre) Gecko/20110119 Firefox/4.0b10pre Build Identifier: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:2.0b10pre) Gecko/20110119 Firefox/4.0b10pre Mouse cursor shouldn't be in "text" mode in the top part of tab group Reproducible: Always Steps to Reproduce: 1. Open panorama 2. Move mouse cursor to the level of "Name this group" (or how it is called in english version) input, but on the right side Actual Results: Mouse cursor is "text" Expected Results: Mouse cursor should be "move"
Comment 2•13 years ago
|
||
Good call, but we'll punt this to the future unless someone gives us a patch to consider.
OS: Windows 7 → All
Hardware: x86 → All
Whiteboard: [visual][polish][good first bug]
Target Milestone: --- → Future
Updated•13 years ago
|
Status: UNCONFIRMED → NEW
Ever confirmed: true
Priority: -- → P4
So, can we put this simple looking patch in b12?
Blocks: 627096
Comment 5•13 years ago
|
||
Comment on attachment 510382 [details] [diff] [review] Patches the tabview.css files to correctly apply the cursor:text style. Thanks for the patch! Please move the cursor attribute into the existing .title-shield, rather than having two entries for the same class. Once that's done, please flag me for review again and it should be good to go!
Attachment #510382 -
Flags: review-
Updated•13 years ago
|
Target Milestone: Future → ---
Thanks for the feedback.
Attachment #510382 -
Attachment is obsolete: true
Attachment #512543 -
Flags: review?(ian)
Comment 7•13 years ago
|
||
Comment on attachment 512543 [details] [diff] [review] Moves the cursor style to the existing selector. Beautiful, thanks!
Attachment #512543 -
Flags: review?(ian)
Attachment #512543 -
Flags: review+
Attachment #512543 -
Flags: approval2.0?
Comment 8•13 years ago
|
||
Comment on attachment 512543 [details] [diff] [review] Moves the cursor style to the existing selector. a=beltzner
Attachment #512543 -
Flags: approval2.0? → approval2.0+
Comment 9•13 years ago
|
||
Attachment #512543 -
Attachment is obsolete: true
Updated•13 years ago
|
Keywords: checkin-needed
Updated•13 years ago
|
Assignee: nobody → unusualtears
Status: NEW → ASSIGNED
Comment 10•13 years ago
|
||
http://hg.mozilla.org/mozilla-central/rev/58123a02d149
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 4.0b12
Updated•8 years ago
|
Product: Firefox → Firefox Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•