User story: N/A
We have this hot mess:
We want this:
Qualifier element should be on top of the tabs (z-indexes, baby!)
Screenshots (if possible):
Acceptance Criteria:
- The qualifier window appears on top of all other page elements
• Ramsey-WMF | |
Mar 19 2019, 1:47 AM |
F28417600: labs-search-tabs-issue.jpg | |
Mar 19 2019, 1:47 AM |
User story: N/A
We have this hot mess:
We want this:
Qualifier element should be on top of the tabs (z-indexes, baby!)
Screenshots (if possible):
Acceptance Criteria:
Subject | Repo | Branch | Lines +/- | |
---|---|---|---|---|
Search prototype | mediawiki/extensions/WikibaseMediaInfo | master | +1 K -14 |
Odd, not the case on my environment. But let's lock those z-indices down, since default obviously can't be trusted :)
I'm going to do things a little differently.
Hiding the keyword|tags toggle behind the qualifier input window is equally problematic when the user start typing another tag when the qualifier window is still open:
Instead, I suggest we should just bring the thing that is focused to the front:
Ok?
Change 441040 had a related patch set uploaded (by Matthias Mullie; owner: Matthias Mullie):
[mediawiki/extensions/WikibaseMediaInfo@master] Search prototype
Change 441040 merged by jenkins-bot:
[mediawiki/extensions/WikibaseMediaInfo@master] Search prototype
Reassigning to confirm solution outlined in https://phabricator.wikimedia.org/T218630#5036226 is good
I'd like to confirm it, but autosuggest dropdowns aren't appearing on Labs right now :)
That might've been because of T219540. Pulled all new code on labs, should be fixed now (WFM at least) - can you try again?
(or try on beta - code was already merged)
It's working now. Still a bit weird though. I think the problem is the first run scenario where, for the first few times you add a tag, we automatically show the qualifier popover as soon as you add it. The popover ends up behind the keyword and tags buttons and is jarring.
For those first "hey, look at this!" in your face popups, can we just get those to show up on top/grab focus as soon as they appear?
I also feel like it would be better if qualifier windows simply disappeared if the input box has focus. Possible?
Closing this, as the current fix resolved this specific problem and user testing showed we definitely don't want to do the automatic display of the qualifier window on the first few runs. We may go with a different system for search qualifiers in general. TBD.