Page MenuHomePhabricator

Wikibase Repository: Collapse button have different height of site link group header in sitelinks view
Closed, ResolvedPublicBUG REPORT

Description

Follow-up of T329167: Wikibase Repository: The collapse button in sitelinks view become invisible, overlapped by the sitelinks heading .

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

  • Check the site group link header and the collapse button

What happens?:

It looks a little not that good as the height of the toggle does not exactly match the header.

I was testing something else and it just look like that, clearly not the same height:

image.png (65×537 px, 4 KB)

Ah yes, when actively giving the parent a different background-color. I agree that it would be nice for them to be aligned, both for something like dark-mode or things like Windows High Contrast Mode (WHCM), but it might be surprisingly finicky to align them properly and consistently, especially with regard to text-only zooming.

So I think we should move forward with this change at is right now, given that this issue is entirely invisible for everyone not changing the background color, and it already provides a significant boost in value and functionality.

I'm open to follow-up patches that tackle the height difference specifically, but I don't think we should block this patch because of that.

What should have happened instead?:

The header and the collapse toggle should have the same height.

NOTE: The header might have line wrap, so it's not a fixed height.

Software version (skip for WMF-hosted wikis like Wikipedia):

Other information (browser name/version, screenshots, etc.):

Event Timeline

Change #1204657 had a related patch set uploaded (by Dumbledore; author: Dumbledore):

[mediawiki/extensions/Wikibase@master] Fix sitelink group header height in sitelinks view

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

I don’t understand this issue. Why is the slightly different height relevant? Is there any case where the different height would be visible to users without custom CSS overrides?

I agree that it would be nice for them to be aligned, both for something like dark-mode or things like Windows High Contrast Mode (WHCM)

I don’t notice anything wrong with the height in Vector 2022 dark mode. (The toggle is black instead of white, which isn’t great, but that’s not related to this issue.) I don’t have access to a Windows box to test high contrast mode.

Winston_Sung closed this task as Resolved.EditedFri, Nov 14, 3:31 PM

It appears that it's no longer a problem in both Vector 2010 and Vector 2022:

image.png (74×840 px, 13 KB)

(Although they still don't have the same height, they now under the same background-color element.)

Thanks for the investigation @Winston_Sung. I see that the visual issue is resolved from a user perspective since they're under the same background element now. However, should I still proceed with my patch to ensure they have the exact same height for semantic consistency, or is the current state acceptable? Happy to abandon if not needed.

However, should I still proceed with my patch to ensure they have the exact same height for semantic consistency, or is the current state acceptable?

I guess that would be a design decision (which would not be decided by me).

Any thoughts from @Lucas_Werkmeister_WMDE ?

IMHO the height of the elements has nothing to do with semantics. If the difference isn’t visible to users, then the patch isn’t necessary (and, since it increases the size of the styles, shouldn’t be merged).