buggy link dialog
Links don't correctly autocomplete when adding them from the references dialog and not the main editor (see screenshot; Chrome, en.wp today).
Version: unspecified
Severity: major
Attached:
Eloquence | |
Oct 16 2014, 2:00 AM |
F32415818: image.png | |
Oct 27 2020, 12:20 AM |
F12139380: image.png | |
Dec 30 2017, 3:47 AM |
F12139527: image.png | |
Dec 30 2017, 3:47 AM |
buggy link dialog
Links don't correctly autocomplete when adding them from the references dialog and not the main editor (see screenshot; Chrome, en.wp today).
Version: unspecified
Severity: major
Attached:
Subject | Repo | Branch | Lines +/- | |
---|---|---|---|---|
[WIP] ve.ui.MWMediaDialog: Use an overlay | mediawiki/extensions/VisualEditor | master | +1 -0 |
Status | Subtype | Assigned | Task | ||
---|---|---|---|---|---|
Open | None | T74121 VisualEditor: Link inspector inside dialog is clipped against dialog boundary, which makes it too small | |||
Open | None | T266521 Allow TargetWidgets various popups (context menus, inspectors) to be placed inside an overlay |
The links are autocompleting just fine, it's just that the dropdown is so small that you only see one suggestion, and the rest can be accessed by scrolling (notice the scrollbar).
What we should really be doing though, is not clip the link inspector against the boundaries of the dialog it's in, but allow it to go beyond those boundaries. That's a bit fiddly but probably not very hard.
I need to pick Trevor's brain about this. Right now the WindowManager for inspectors is in the localOverlay, which is a sibling of the ce-surface (and so is inside the iframe). It would be interesting to explore putting this in the globalOverlay, but then that would be nasty for scrollable surfaces (or even surfaces embedded in a scrollable container; like, for example, an iframe) where the anchor moves as you scroll: in a globalOverlay situation, we'd have to manually reposition the inspector to account for scrolling, but with the localOverlay as it is we get this for free.
The link dialogue is still contained inside the basic reference form, so this is still a valid report. I don't expect a great many people use the basic reference form, so I think the priority here can stay as it is.
This is still an issue, with the minor difference that the inspector is not clipped, but rather forces the dialog to have a scrollbar:
It should look like this: (mockup)
This is still an issue today, although somehow it has changed back to clipping the inspector, like in the original report from 2014.
Change 636522 had a related patch set uploaded (by Bartosz Dziewoński; owner: Bartosz Dziewoński):
[mediawiki/extensions/VisualEditor@master] [WIP] ve.ui.MWMediaDialog: Use an overlay
Change 636522 abandoned by Bartosz Dziewoński:
[mediawiki/extensions/VisualEditor@master] [WIP] ve.ui.MWMediaDialog: Use an overlay
Reason: