Page MenuHomePhabricator

Technical: Reconsider logo implementation
Closed, DeclinedPublic


In @Demian made some popular changes to the logo to simplify its rendering so that it didn't use padding + margin.

According to @nray and @Volker_E they've faced some challenges with the existing CSS and for improved maintenance would like us to pursue the changes proposed in Aron's patch.

We however agreed that this does not and should not need to be coupled with the moving of the personal tools and search in T249363

Aron also provided this patch for simplifying the logo itself (see original bug T256895) which we should definitely consider:

Event Timeline

@nray @Volker_E per our chat Monday it would be helpful to articulate what the challenges are with the status quo while fresh in your memory. Thank you!

@Jdlrobson sorry for the delay on this. I had the following concerns with the status quo:

  • The top margin on the header will make it difficult to implement a sticky header which is on the roadmap for DIP (AFAICT). The top margin will create a space between the header and the viewport e.g.

fixed-header.png (436×2 px, 91 KB)

  • Relatively minor, the top margin on the header makes the calculation for the top position of the sidebar slightly more complex [1]. As an aside and mostly unrelated, we will probably also need to move the sidebar out of the header unless we want that to be sticky as well so this concern may work itself out naturally.

Both of these factors led to me favoring the use of padding instead of margins for the header.


Jdlrobson renamed this task from Technical: Reconsider logo layout rules to Technical: Reconsider logo implementation.Jul 31 2020, 10:20 PM
Jdlrobson updated the task description. (Show Details)
Jdlrobson added a project: Vector.
Jdlrobson updated the task description. (Show Details)

Change 608255 had a related patch set uploaded (by Jdlrobson; owner: Aron Manning):
[mediawiki/skins/Vector@master] [modern] Simplify Logo layout rules: use inline-block instead of float and flex

Change 608255 abandoned by Jdlrobson:
[mediawiki/skins/Vector@master] [modern] Simplify Logo layout rules: use inline-block instead of float and flex

261 days old. Associated with most recent ticket so this is not lost.

Jdlrobson added a subscriber: Jdrewniak.

@nray @Jdrewniak would you have any concerns with me closing this? It doesn't seem like we need to do this anymore.