Last modified: 2014-09-04 01:02:27 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T72115, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 70115 - OOjs UI: MenuWidget does not resize correctly when being clipped
OOjs UI: MenuWidget does not resize correctly when being clipped
Status: RESOLVED FIXED
Product: OOjs UI
Classification: Unclassified
General (Other open bugs)
unspecified
All All
: Normal normal
: ---
Assigned To: Roan Kattouw
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2014-08-28 00:13 UTC by Roan Kattouw
Modified: 2014-09-04 01:02 UTC (History)
1 user (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description Roan Kattouw 2014-08-28 00:13:04 UTC
1. Open the link inspector
2. Type something that will get a lot of suggestions (e.g. "M")
3. Resize the window so that the suggestions menu clips
4. Type more characters so that you will get very few suggestions (e.g. "Mfdsafsdfads")

The new list of suggestions is now short enough that it doesn't need to clip any more, but the menu doesn't shrink. If you do this while the menu isn't clipping, it shrinks just fine.

AIUI the core of the issue is that we don't have a way to tell ClippableElement that the natural size of the element changed and that it should reevaluate the clipping state.
Comment 1 Gerrit Notification Bot 2014-08-29 18:50:47 UTC
Change 157202 had a related patch set uploaded by Catrope:
Make menus reevaluate their clipping when they change

https://gerrit.wikimedia.org/r/157202
Comment 2 Gerrit Notification Bot 2014-09-04 00:38:32 UTC
Change 157202 merged by jenkins-bot:
Make menus reevaluate their clipping when they change

https://gerrit.wikimedia.org/r/157202

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links