Skip to content
This repository has been archived by the owner on Jul 30, 2022. It is now read-only.

Misaligned Icons in VisualEditor #87

Open
chrom11 opened this issue Jul 31, 2019 · 7 comments
Open

Misaligned Icons in VisualEditor #87

chrom11 opened this issue Jul 31, 2019 · 7 comments

Comments

@chrom11
Copy link

chrom11 commented Jul 31, 2019

Setup

  • Browsers and versions: Chrome 75
  • MediaWiki version: 1.32.3
  • Database version: MySQL 5.7.26
  • PHP version: PHP 7.2.19
  • Pivot version: 2.1.0
  • Extension version (if connected to the issue): VisualEditor REL1_32 and Parsoid 0.9.0

Issue

The icons are misaligned when using VisualEditor on the latest version of Pivot. Strangely the issue is inconsistent, it affects certain icons but leaves others alone.

misaligned

@kghbln
Copy link
Contributor

kghbln commented Jul 31, 2019

That's the hot fix for your wiki: CSS diff

@s0rin
Copy link

s0rin commented Aug 20, 2019

That's great, the hot fix works, but the icons from the dropmenu to toggle from visual to source editing are still misaligned.

mw_visual_editor_dropdown

@kghbln
Copy link
Contributor

kghbln commented Aug 20, 2019

Well, the fix in one location creates the issue in another one. You can use this for better results but be aware that this is still not ideal. However, I do not have more time for this now, meaning you are welcome to fiddle with CSS to get this back on track for a 100%.

/* Fix edit bar of VisualEditor at least for MediaWiki 1.31 and later */
.oo-ui-tool .oo-ui-tool-link {
    padding-top: 0;
}
.oo-ui-popupToolGroup .oo-ui-tool-link .oo-ui-tool-accel,
.oo-ui-popupToolGroup .oo-ui-tool-link .oo-ui-tool-title {
    padding-top: 1em;
}

@BitBangingBytes
Copy link

BitBangingBytes commented May 1, 2020

Had this same issue myself, if you just comment out the following lines in /pivot/assets/stylesheets/pivot.css it fixes the issue.

.oo-ui-barToolGroup.oo-ui-widget-enabled > .oo-ui-toolGroup-tools > .oo-ui-tool > .oo-ui-tool-link,
.oo-ui-barToolGroup > .oo-ui-toolGroup-tools > .oo-ui-tool.oo-ui-widget-disabled > .oo-ui-tool-link  {
    display: inline !important;
}

@kghbln
Copy link
Contributor

kghbln commented May 1, 2020

@BitBangingBytes Thanks for your solution! So this does not affect other parts of Pivot? Will be great if you could confirm.

If no the CCS should probably just be removed from Pivot as soon as the old version of MW is no longer supported by Pivot.

@BitBangingBytes
Copy link

I have been checking all around the wiki and don't see any issues, all the drop downs related to toolbar are fine. If I come across any issues related to this I will post a follow-up comment.

@fredvik
Copy link

fredvik commented Jul 17, 2020

Thanks @BitBangingBytes it works great for me!
I can just add that there are two occurences of the text and both need to be deleted/commented out for the fix to work.

The same issue is present in the Pivot skin and it can be solved in the same way.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants