Closed
Bug 694954
Opened 13 years ago
Closed 13 years ago
[highlighter] [infobar] If a node is not visible anymore, the infobar should point to the direction of the node
Categories
(DevTools :: General, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
Firefox 12
People
(Reporter: paul, Assigned: paul)
References
Details
(Whiteboard: [good first bug][mentor=paul][lang=js][lang=css])
Attachments
(2 files, 1 obsolete file)
(deleted),
patch
|
rcampbell
:
review+
|
Details | Diff | Splinter Review |
(deleted),
patch
|
rcampbell
:
review+
|
Details | Diff | Splinter Review |
The current behavior is to move the infobar to the top-right corner. It would make more sense to keep the infobar aligned with the node, and make it point to its direction.
Assignee | ||
Updated•13 years ago
|
Whiteboard: [[good first bug][mentor=paul]
Assignee | ||
Updated•13 years ago
|
Whiteboard: [[good first bug][mentor=paul] → [good first bug][mentor=paul][lang=js/css]
Assignee | ||
Updated•13 years ago
|
Whiteboard: [good first bug][mentor=paul][lang=js/css] → [good first bug][mentor=paul][lang=js][lang=css]
Assignee | ||
Comment 1•13 years ago
|
||
Assignee | ||
Updated•13 years ago
|
Attachment #579696 -
Flags: review?(rcampbell)
Comment 2•13 years ago
|
||
Comment on attachment 579696 [details] [diff] [review]
patch v1
I see what you did there.
Is this something we need a test for?
Attachment #579696 -
Flags: review?(rcampbell) → review+
Comment 3•13 years ago
|
||
Assigning this to Paul as it appears that he is working on this.
I'm trying to keep the status of our mentored bugs up to date. Please unassign if this was done in error.
Assignee: nobody → paul
Status: NEW → ASSIGNED
Assignee | ||
Comment 4•13 years ago
|
||
with test
Assignee | ||
Updated•13 years ago
|
Attachment #582834 -
Flags: review?(rcampbell)
Updated•13 years ago
|
Attachment #582834 -
Flags: review?(rcampbell) → review+
Comment 5•13 years ago
|
||
- let rect = this._highlightRect;
- if (rect && this._highlighting) {
+ if (this._highlightRect) {
let winHeight = this.win.innerHeight * this.zoom;
let winWidth = this.win.innerWidth * this.zoom;
+
+ let rect = {top: this._highlightRect.top,
+ left: this._highlightRect.left,
+ width: this._highlightRect.width,
+ height: this._highlightRect.height};
this didn't apply cleanly. Wondering if you had some other patch installed previously, as the let winHeight = … and winWidth lines were not there.
Added them.
Comment 6•13 years ago
|
||
Whiteboard: [good first bug][mentor=paul][lang=js][lang=css] → [good first bug][mentor=paul][lang=js][lang=css][fixed-in-fx-team]
Comment 7•13 years ago
|
||
Backed out because of orange
https://hg.mozilla.org/integration/fx-team/rev/37b413b9c87b
Whiteboard: [good first bug][mentor=paul][lang=js][lang=css][fixed-in-fx-team] → [good first bug][mentor=paul][lang=js][lang=css]
Updated•13 years ago
|
Target Milestone: --- → Firefox 12
Assignee | ||
Comment 8•13 years ago
|
||
So this should fix the test issue.
https://tbpl.mozilla.org/?tree=Try&rev=9e5579f7bb5e
Assignee | ||
Updated•13 years ago
|
Attachment #582834 -
Attachment is obsolete: true
Assignee | ||
Comment 9•13 years ago
|
||
Apparently, this is working.
Assignee | ||
Updated•13 years ago
|
Attachment #583479 -
Flags: review?(rcampbell)
Comment 10•13 years ago
|
||
Comment on attachment 583479 [details] [diff] [review]
patch 2
alright, let's try this again. :)
Attachment #583479 -
Flags: review?(rcampbell) → review+
Updated•13 years ago
|
Whiteboard: [good first bug][mentor=paul][lang=js][lang=css] → [good first bug][mentor=paul][lang=js][lang=css][land-in-fx-team]
Comment 11•13 years ago
|
||
Whiteboard: [good first bug][mentor=paul][lang=js][lang=css][land-in-fx-team] → [good first bug][mentor=paul][lang=js][lang=css][fixed-in-fx-team]
Comment 12•13 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Whiteboard: [good first bug][mentor=paul][lang=js][lang=css][fixed-in-fx-team] → [good first bug][mentor=paul][lang=js][lang=css]
Updated•6 years ago
|
Product: Firefox → DevTools
You need to log in
before you can comment on or make changes to this bug.
Description
•