Closed
Bug 963067
Opened 11 years ago
Closed 11 years ago
'Cut' in the cut, copy, paste menu is always active after a browser launch
Categories
(Firefox for Metro Graveyard :: Browser, defect, P1)
Tracking
(Not tracked)
VERIFIED
FIXED
Firefox 29
People
(Reporter: jimm, Assigned: azasypkin)
References
Details
(Keywords: polish, Whiteboard: p=2 s=it-30c-29a-28b.1 r=ff29)
Attachments
(1 file)
2.89 KB,
patch
|
jimm
:
review+
|
Details | Diff | Splinter Review |
STR:
1) launch browser
2) visible buzilla.mozilla.org
3) type some text in a text edit
4) double click to select
5) right-click text
'Cut' will be highlighted (orange background). Select 'Copy', then the menu works as expected from that point on.
This is a good [release] polish bug.
Updated•11 years ago
|
Blocks: metrov1backlog
Whiteboard: [triage] → [triage] [defect] p=0
Updated•11 years ago
|
Whiteboard: [triage] [defect] p=0 → [release28] [defect] p=0
Updated•11 years ago
|
Whiteboard: [release28] [defect] p=0 → [triage] [defect] p=0
Updated•11 years ago
|
Whiteboard: [triage] [defect] p=0 → [release28] [defect] p=0
Updated•11 years ago
|
Whiteboard: [release28] [defect] p=0 → [release28] [defect] p=2
Assignee | ||
Updated•11 years ago
|
Assignee: nobody → azasypkin
Assignee | ||
Updated•11 years ago
|
Status: NEW → ASSIGNED
Assignee | ||
Comment 1•11 years ago
|
||
Richlistbox sets current index to "0" if nothing is selected by default here: http://mxr.mozilla.org/mozilla-central/source/toolkit/content/widgets/richlistbox.xml#413. That triggers the following update chain: this.currentIndex -> this.currentItem -> this.selectedItem(through assignment in richlistbox "this.selectedItem = this.currentItem") and finally we have the first item selected for non-multiple richlistbox. Unfortunately it's not mentioned in the official docs: https://developer.mozilla.org/en-US/docs/XUL/richlistbox. To be honest, it looks like a bug in richlistbox for me(I think -1 should be default index), but nobody is complaining that convince me that behavior is expected.
Try is still running here: https://tbpl.mozilla.org/?tree=Try&rev=faa731efcd23
Attachment #8369406 -
Flags: review?(jmathies)
Reporter | ||
Updated•11 years ago
|
Attachment #8369406 -
Flags: review?(jmathies) → review+
Comment 3•11 years ago
|
||
Flags: in-testsuite+
Keywords: checkin-needed
Whiteboard: [release28] [defect] p=2 → [release28] [defect] p=2[fixed-in-fx-team]
Updated•11 years ago
|
Priority: -- → P1
QA Contact: jbecerra
Whiteboard: [release28] [defect] p=2[fixed-in-fx-team] → [release28] [defect] p=2[fixed-in-fx-team] s=it-30c-29a-28b.1
Comment 4•11 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Whiteboard: [release28] [defect] p=2[fixed-in-fx-team] s=it-30c-29a-28b.1 → [release28] [defect] p=2 s=it-30c-29a-28b.1
Target Milestone: --- → Firefox 29
Updated•11 years ago
|
Whiteboard: [release28] [defect] p=2 s=it-30c-29a-28b.1 → [defect] p=2 s=it-30c-29a-28b.1
Updated•11 years ago
|
Whiteboard: [defect] p=2 s=it-30c-29a-28b.1 → p=2 s=it-30c-29a-28b.1 r=ff29
Comment 5•11 years ago
|
||
Mozilla/5.0 (Windows NT 6.3; WOW64; rv:29.0) Gecko/20100101 Firefox/29.0
Verified as fixed on latest Aurora (build ID: 20140218004001) using the provided STR.
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•