Page MenuHomePhabricator

[Mobile] Opened Search preview requires horizontal scrolling for all articles' snippets
Closed, ResolvedPublic

Description

Found when testing T320646

  1. On betalabs go to Special:Search
  2. Perform search and click on the arrow on the first result to open the Search preview.
  3. The Search preview will open; you may scroll horizontally to see an article's lead image, an article's description etc (CORRECT behavior)

Comparing with the AC in T320646, the following issues were found:

DONE (1) When the Search preview is open for one article, all other article snippets will occupy the same width as the article with the opened Search preview, so the vertical scrolling will be required for all articles' snippets. See the animated gif below:

horizontal_scroll.gif (755×591 px, 1010 KB)

(2) With the Search preview open, the arrow will be placed at the end of the expanded snippet for all articles on the search result page.

mockupenwiki betalabs
Screen Shot 2022-12-14 at 5.18.45 PM.png (966×440 px, 190 KB)
Screen Shot 2022-12-14 at 5.16.28 PM.png (750×364 px, 164 KB)
Screen Shot 2022-12-14 at 5.13.31 PM.png (731×335 px, 59 KB)

Event Timeline

Change 868505 had a related patch set uploaded (by Vadim Kovalenko; author: Vadim Kovalenko):

[mediawiki/extensions/SearchVue@master] Opened Search preview requires horizontal scrolling for all articles snippets

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

I tried to replicate this with Browserstack and also the mobile emulator and I cannot manage to replicate it anymore.

Due to numerous changes in the CSS, I expect this to have been fixed already. I am NOT merging the fix until we have a way to reproduce it.

i am moving this into QA so that Elena can check it again (I spoke with her directly).

Etonkovidova updated the task description. (Show Details)

I tried to replicate this with Browserstack and also the mobile emulator and I cannot manage to replicate it anymore.

Due to numerous changes in the CSS, I expect this to have been fixed already. I am NOT merging the fix until we have a way to reproduce it.

i am moving this into QA so that Elena can check it again (I spoke with her directly).

Yes, I re-checked - the issues are not present anymore, thx @SimoneThisDot.

Change 868505 abandoned by Vadim Kovalenko:

[mediawiki/extensions/SearchVue@master] Opened Search preview requires horizontal scrolling for all articles snippets

Reason:

Patch is irrelevant now.

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