| Etonkovidova | |
| Mar 16 2016, 8:37 PM |
| F3645249: Screen Shot 2016-03-16 at 1.32.08 PM.png | |
| Mar 16 2016, 9:06 PM |
| F3645171: Screen Shot 2016-03-16 at 1.32.08 PM.png | |
| Mar 16 2016, 8:37 PM |
In Chrome and FF (nothing in Console) - seems to be happening only the Notification panel is rather short - does not have a vertical scroll bar
| Subject | Repo | Branch | Lines +/- | |
|---|---|---|---|---|
| Work around regression in OO.ui.FloatableElement | mediawiki/extensions/Echo | master | +3 -1 |
Also reproducible if you lower the text size ("zoom out") to the point where the scroll bar in the notifications popup goes away.
This is an OOUI regression from https://gerrit.wikimedia.org/r/#/c/276664/ (for T129521). The culprit is this.needsCustomPosition = closestScrollableOfContainer !== closestScrollableOfFloatable;. This is plainly false, because (like in our case) the floatable can be in an overlay at (0,0) while the container is somewhere else, in which case positioning is definitely needed, but depending on the scrollbar situation the closest scrollable container could be the window for both of them.
Change 277908 had a related patch set uploaded (by Catrope):
Work around regression in OO.ui.FloatableElement
Change 277908 merged by jenkins-bot:
Work around regression in OO.ui.FloatableElement