Column picker does not work in Page info media tab and about:preferences#privacy View certification dialog
Categories
(Core :: XUL, defect)
Tracking
()
Tracking | Status | |
---|---|---|
firefox-esr102 | --- | unaffected |
firefox105 | --- | unaffected |
firefox106 | --- | unaffected |
firefox107 | --- | verified |
People
(Reporter: alice0775, Assigned: emilio)
References
(Regression)
Details
(Keywords: nightly-community, regression)
Attachments
(2 files)
Steps To Reproduce:
- Open Page Info ( Ctrl+I )
- Click
Media
tab - Click column picker at the right side of table header
- Select a column name (e.g.
Size
) - optionally, try to repeat step 3 and 4
Actual Results:
Nothing added header.
Clicking on the column picker does not show the dropdown except the first time.
Expected Results:
The column should be added/removed.
The second and subsequent clicks on the column picker should also show/hide a dropdown.
Regression window:
https://hg.mozilla.org/integration/autoland/pushloghtml?fromchange=51423d631a9acaaabaa931809c54b083a63ca533&tochange=d43a9bd44619d8b122caa97a64cabf5c185b8623
Note:
If select a columun name in Step 4 with keyboard, the selected column will show/hide as expected.
Reporter | ||
Updated•2 years ago
|
Reporter | ||
Comment 1•2 years ago
|
||
about:preferences#privacy > View certification...
,
the Certificate Manager dialog is also affected.
Comment 2•2 years ago
|
||
:emilio, since you are the author of the regressor, bug 1790920, could you take a look? Also, could you set the severity field?
For more information, please visit auto_nag documentation.
Assignee | ||
Updated•2 years ago
|
Assignee | ||
Comment 3•2 years ago
|
||
This element had a really weird dependency on the order of the command
events from the menuitems and the button which broke in bug 1790920.
Instead of keeping this element as a XULButtonElement, have a separate
button and popup. That's simpler and less prone to breakage, and allows
to get rid of special cases in Gecko.
Assignee | ||
Updated•2 years ago
|
Pushed by ealvarez@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/f9249f760526 Fix tree column picker. r=aminomancer
Comment 7•2 years ago
|
||
Backed out for causing mochitest failures on test_struct_tree.xhtml
Assignee | ||
Comment 8•2 years ago
|
||
After the previous change, the treecolpicker element only wraps a button
and a menu, which are siblings rather than parent/child. I think that's
probably fine but it changes the shape of the a11y tree.
This patch updates the relevant tests, but let me know if you'd rather
keep the behavior. I could do that by either keeping the treecolpicker
entry listed in XULMap (I think), or via other means.
Suggestions welcome if you think this is not the right fix.
Assignee | ||
Updated•2 years ago
|
Pushed by ealvarez@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/df75f29cded6 Fix tree column picker. r=aminomancer https://hg.mozilla.org/integration/autoland/rev/c14f7934269f Fix accessible tests after previous change. r=Jamie
Comment 10•2 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/df75f29cded6
https://hg.mozilla.org/mozilla-central/rev/c14f7934269f
Updated•1 year ago
|
Comment 11•1 year ago
|
||
Verified as fixed in our latest Beta 107.0b3.
Description
•