Page MenuHomePhabricator

TypeaheadSearch: fix space between footer icon and text
Closed, ResolvedPublicBUG REPORT

Description

Steps to replicate the issue (include links if applicable):

What happens?:

There's no space between the footer icon and the text next to it

image.png (1,326×300 px, 48 KB)

What should have happened instead?:

There should be at least some space. Note that this currently looks right when thumbnails are included and the solution to this bug shouldn't affect that.

Software version: Codex 2.0.0

Event Timeline

Restricted Application added a subscriber: Aklapper. · View Herald Transcript

Thank you for tagging this task with good first task for Wikimedia newcomers!

Newcomers often may not be aware of things that may seem obvious to seasoned contributors, so please take a moment to reflect on how this task might look to somebody who has never contributed to Wikimedia projects.

A good first task is a self-contained, non-controversial task with a clear approach. It should be well-described with pointers to help a completely new contributor, for example it should clearly pointed to the codebase URL and provide clear steps to help a contributor get setup for success. We've included some guidelines at https://phabricator.wikimedia.org/tag/good_first_task/ !

Thank you for helping us drive new contributions to our projects <3

Just noting this doesn't seem to be a problem for Typeahead in Vector 2022:

image.png (451×114 px, 17 KB)

@CCiufo-WMF yeah, it's only when you don't have thumbnails enabled (which will be true on wikidata)

@AnneT I've been able to replicate this. I will start working on a patch

Change #1164117 had a related patch set uploaded (by Formasit-chijoh; author: Formasit-chijoh):

[design/codex@main] Fix spacing in TypeheadSearch footer without thumbnails

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

Change #1164117 had a related patch set uploaded (by Formasit-chijoh; author: Formasit-chijoh):

[design/codex@main] Fix spacing in TypeheadSearch footer without thumbnails

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

@CCiufo-WMF I have made a patch for this and it is ready for review, I could not assigned myself to the task so I indicated I was working on it

CCiufo-WMF moved this task from In Progress to Code Review on the Codex board.

Oops, sorry about that @Formasit-chijoh ! Re-assigned to you. I'll see if someone is available to review.

Change #1164117 merged by jenkins-bot:

[design/codex@main] TypeaheadSearch: Fix spacing in footer without thumbnails

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

Change #1171718 had a related patch set uploaded (by Catrope; author: Catrope):

[mediawiki/core@master] Update Codex from v2.2.0 to v2.2.1

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

Change #1171718 merged by jenkins-bot:

[mediawiki/core@master] Update Codex from v2.2.0 to v2.2.1

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