SelectTranslationsPanel glitches when resized after opening with vertical flip mode
Categories
(Firefox :: Translations, defect, P3)
Tracking
()
Tracking | Status | |
---|---|---|
firefox-esr115 | --- | unaffected |
firefox125 | --- | unaffected |
firefox126 | --- | unaffected |
firefox127 | --- | disabled |
firefox128 | --- | fixed |
People
(Reporter: nordzilla, Assigned: nordzilla)
References
(Blocks 2 open bugs, Regression)
Details
(Keywords: regression)
Attachments
(1 file, 1 obsolete file)
Description
When the panel is opened such that the target screen location is near to the bottom edge of the screen, and the panel would not have enough room to fully open in the visible view at that location, the panel's location is flipped vertically over the horizontal axis of the target point, allowing it to have enough room to fully open.
When this happens, dragging the textarea resizer downward causes the panel both to glitch as well as grow from the top instead of the bottom.
We should just disable resizing if the panel was opened with vertical flip mode.
Steps to Reproduce
- Open a page and highlight some text.
- Open the SelectTranslationsPanel via the right-click context menu very close to the bottom edge of the screen.
Expected Behavior
When the panel opens and translates, the textarea resizer is not available.
Actual behavior
When the panel opens and translates, the textarea resizer is available and causes glitchy behavior when used that varies slightly for each operating system.
Steps to implement
- Fixing this will require propagating the mVFlip and mHFlip boolean values up through the popuppositioned event, so that we can tell if the arrow panel was opened with the vertical flip mode.
- Following that, we need to add a listener for the
popuppositioned
event on the SelectTranslationsPanel and disable the resizer if vertical flip mode was used.
Tests to implement
- This is very hard to test in automation, as it involves clicking and dragging with the mouse.
Assignee | ||
Updated•19 days ago
|
Comment 1•19 days ago
|
||
Set release status flags based on info from the regressing bug 1890490
Assignee | ||
Comment 2•19 days ago
|
||
Adds attributes to the popuppositioned event that report
whether vertical flip and/or horizontal flip modes were
used in the positioning of the popup.
Assignee | ||
Comment 3•19 days ago
|
||
Determines if the SelectTranslationsPanel was opened
with the vertical flip mode, and, if so, disables the
textarea resizer because it causes the panel to grow
from the top instead of the bottom, even though the
user is dragging the resizer downward.
Depends on D209395
Updated•19 days ago
|
Updated•19 days ago
|
Updated•19 days ago
|
Updated•9 days ago
|
Updated•9 days ago
|
Comment 4•8 days ago
|
||
Set release status flags based on info from the regressing bug 1890490
Pushed by enordin@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/aa76a8a2f193 Disable textarea resize based on popuppositioned r=translations-reviewers,desktop-theme-reviewers,gregtatum,emilio
Comment 6•6 days ago
|
||
bugherder |
Comment 7•6 days ago
|
||
The patch landed in nightly and beta is affected.
:nordzilla, is this bug important enough to require an uplift?
- If yes, please nominate the patch for beta approval.
- If no, please set
status-firefox127
towontfix
.
For more information, please visit BugBot documentation.
Assignee | ||
Comment 8•5 days ago
•
|
||
This does not need to be uplifted. The target release is now 128, and the panel is preffed off by default in 127 release build.
Assignee | ||
Updated•8 hours ago
|
Description
•