Closed
Bug 1490776
Opened 7 years ago
Closed 7 years ago
Sidebar properties that are arrays render as objects.
Categories
(DevTools :: Accessibility Tools, enhancement)
DevTools
Accessibility Tools
Tracking
(firefox63 fixed, firefox64 fixed)
RESOLVED
FIXED
Firefox 64
People
(Reporter: yzen, Assigned: yzen)
Details
Attachments
(1 file)
|
46 bytes,
text/x-phabricator-request
|
gl
:
review+
pascalc
:
approval-mozilla-beta+
|
Details | Review |
We need to use a noGrip property to fix the array rendering. Right now all array props show up as objects.
| Assignee | ||
Comment 1•7 years ago
|
||
MozReview-Commit-ID: 4QAPbUBYiCW
Comment 2•7 years ago
|
||
Comment on attachment 9008502 [details]
Bug 1490776 - fix noGrip array rendering in a11y panel sidebar. r=gl
Gabriel [:gl] (ΦωΦ) has approved the revision.
Attachment #9008502 -
Flags: review+
Pushed by yura.zenevich@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/f039fc0bda93
fix noGrip array rendering in a11y panel sidebar. r=gl
| Assignee | ||
Comment 4•7 years ago
|
||
Comment on attachment 9008502 [details]
Bug 1490776 - fix noGrip array rendering in a11y panel sidebar. r=gl
[Feature/Bug causing the regression]: There was a change in reps component that renders arrays which required an update on the a11y panel side.
[User impact if declined]: Array in the sidebar will look like objects are would be very confusing to the users.
[Is this code covered by automated tests?]: No
[Has the fix been verified in Nightly?]: Not yet, locally. Queued for landing.
[Needs manual test from QE? If yes, steps to reproduce]: No
[List of other uplifts needed for the feature/fix]: None
[Is the change risky?]: No
[Why is the change risky/not risky?]: Just a small property added in place where simple JS values are rendered in the properties sidebar.
[String changes made/needed]: None
Attachment #9008502 -
Flags: approval-mozilla-beta?
Comment 5•7 years ago
|
||
| bugherder | ||
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
status-firefox64:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 64
Comment 6•7 years ago
|
||
Comment on attachment 9008502 [details]
Bug 1490776 - fix noGrip array rendering in a11y panel sidebar. r=gl
Low-risk patch for a11y, uplift approved for 63 beta 6 thanks.
Attachment #9008502 -
Flags: approval-mozilla-beta? → approval-mozilla-beta+
Comment 7•7 years ago
|
||
| bugherder uplift | ||
status-firefox63:
--- → fixed
You need to log in
before you can comment on or make changes to this bug.
Description
•