hi @eranroz - as far as I am aware there has not been specific discussion, as both apps have not used a border for a while now on thumbnail images, nor on mobile web:

Thumbnail on mobile web	Thumbnail on Android search results

However, upon reading the task description more closely, can I just confirm whether you were instead referring to when images are shown within articles that have a border defined? In that case there does appear to be a bug on Android where the border attribute is not being respected when the image appears in a quick box:

Borders show as expected in images within the article	Bug - no border when in the infobox:

Please confirm if this is accurate, and I will move this to be fixed in Bugs. Thanks!

@RHo yes, this is about images within articles. Thanks

@Sharvaniharan, the border color should be #c8ccd1 (eg. Base70). Thanks!

Thank you @cmadeo

The root cause for this is, the class "thumbborder"[as is seen in web/mobile content] is being removed from the <img> tag.

Eg as on web content now :
<img alt="Flag of Japan.svg" src="upload.wikimedia.org/wikipedia/en/thumb/9/9e/Flag_of_Japan.svg/255px-Flag_of_Japan.svg.png" width="255" height="170" class="thumbborder" srcset="upload.wikimedia.org/wikipedia/en/thumb/9/9e/Flag_of_Japan.svg/383px-Flag_of_Japan.svg.png 1.5x, //upload.wikimedia.org/wikipedia/en/thumb/9/9e/Flag_of_Japan.svg/510px-Flag_of_Japan.svg.png 2x" data-file-width="900" data-file-height="600">

The Parsoid HTML actually doesn't have this attribute to begin with. Instead it wraps the <a><img> into a <figure-inline class="mw-image-border">.

content.parsoid.less has a rule to make the border appear in the Parsoid version:

/* Same as img.thumbborder in content.css */
.mw-image-border > *:first-child {
	> img,
	> video {
		border: 1px solid #eaecf0;
	}
}

We might need to look into getting those rules added to the base CSS.

Looks like this rule only exists for Vector. Our base CSS endpoint requests Minerva, though: https://en.wikipedia.org/w/load.php?debug=false&lang=en&skin=minerva&target=mobile&only=styles&modules=ext.cite.style%7Cext.math.styles%7Cext.pygments%7Cext.timeline.styles%7Cmediawiki.page.gallery.styles%7Cmediawiki.skinning.content.parsoid%7Cmobile.app%7Cmobile.app.parsoid%7Cskins.minerva.base.reset%7Cskins.minerva.content.styles

@bearND are we proposing to duplicate only this class styles? or duplicating all the styles that are missing in Minerva? for later, we might want to see what elements will be impacted by this

@Nirzar I'm not sure what the best way to handle this is. This will affect web a little bit, too, at least by bloating up the CSS. So, I thought it's best to ask for advice first.

I think we should check what content.parsoid.less does to the content on minerva, and if it is acceptable (the file is quite small) then including that file/module to it would be a good idea, making mobile web content styles work with parsoid HTML for the future parser migration.

If it doesn't work well, we could have a minerva parsoid module where we add this kind of rules for styling parsoid content.

@phuedx can you route this to your team, to get some thoughts?

content.parsoid.less styles are already requested but not included in the response of https://en.wikipedia.org/w/load.php?debug=false&lang=en&skin=minerva&target=mobile&only=styles&modules=ext.cite.style%7Cext.math.styles%7Cext.pygments%7Cext.timeline.styles%7Cmediawiki.page.gallery.styles%7Cmediawiki.skinning.content.parsoid%7Cmobile.app%7Cmobile.app.parsoid%7Cskins.minerva.base.reset%7Cskins.minerva.content.styles.

We'd have to switch to vector instead: https://en.wikipedia.org/w/load.php?debug=false&lang=en&skin=vector&target=mobile&only=styles&modules=ext.cite.style%7Cext.math.styles%7Cext.pygments%7Cext.timeline.styles%7Cmediawiki.page.gallery.styles%7Cmediawiki.skinning.content.parsoid%7Cmobile.app%7Cmobile.app.parsoid%7Cskins.minerva.base.reset%7Cskins.minerva.content.styles

@Jdlrobson I think we're going to want the content.parsoid.less styles included in Minerva as well for this to properly work? Thoughts?

I think mediawiki.skinning.content.parsoid is what you want:
https://en.wikipedia.org/w/load.php?modules=mediawiki.skinning.content.parsoid&only=styles&useskin=minerva

That's what VE on mobile uses.

useskin=minerva

This triggers the loading of skinSyles:
https://github.com/wikimedia/mediawiki-skins-MinervaNeue/blob/master/skinStyles/mediawiki.skinning.content.parsoid/minerva.less

mediawiki.skinning.content.parsoid is not used inside the Minerva skin outside VisualEditor, so feel free to put any styles in that file above that you need! I suspect it is incomplete. Note the order of modules in your request is important, it might be beneficial to include mediawiki.skinning.content.parsoid module last. I suspect the font changes might be in a different module. When you inspect are you seeing any overrides? Knowing the exact CSS setting the font would be useful.

1. Any update?
2. What is responsible for trimming the mediawiki.skinning.content.parsoid? e.g

https://en.wikipedia.org/w/load.php?debug=false&lang=en&skin=minerva&target=mobile&only=styles&modules=mediawiki.skinning.content.parsoid
vs
https://en.wikipedia.org/w/load.php?debug=false&lang=en&skin=FAKE_STYLE&target=mobile&only=styles&modules=mediawiki.skinning.content.parsoid

On local dev setup, I'm not able to reproduce it. (though I don't have all the extensions, but I do have MobileFrontend and MobileApp)

skinstyles are specific to the skin. As described in https://phabricator.wikimedia.org/T198534#4889186 minerva screen defines its own styles and the new styles need to be added here.

In T198534#4960616, @Jdlrobson wrote:

skinstyles are specific to the skin. As described in https://phabricator.wikimedia.org/T198534#4889186 minerva screen defines its own styles and the new styles need to be added here.

I don't understand how to replicate this in local dev setup.
What extensions/skins are required to get it reproduced? (MobileFrontend?Minerva? MobileApp? all are installed and plain MW doesn't have this issue)
any special LocalSettings?

@eranroz the URL [1] you are using asks for the ResourceLoader module mediawiki.skinning.content.parsoid in mobile target mode for skin Minerva
Thus Minerva and core is all that's required to obtain this URL.

Documentation on how skinStyles works can be found here:
https://www.mediawiki.org/wiki/Manual:$wgResourceModules#Details
https://www.mediawiki.org/wiki/Manual:$wgResourceModuleSkinStyles
(although deprecated the information contained is relevant about how they work)

Practically, all skinStyles are defined here:
https://github.com/wikimedia/mediawiki-skins-MinervaNeue/tree/master/skinStyles/mediawiki.skinning.content.parsoid
This CSS code is used in both apps and VisualEditor.

[1] https://en.wikipedia.org/w/load.php?debug=false&lang=en&skin=minerva&target=mobile&only=styles&modules=mediawiki.skinning.content.parsoid

some people in hewiki are asking is there any update about this task or expected ETA?
It takes more than a 0.5 year to handle CSS issues. (I know there are tons of other tasks, but please find some time for it. If we aren't sure about other CSS rules - maybe just port only the border and create a separate task to investigate the rest of the CSS) .
[If you are curious - this disscussion is in hewiki and Google Translate is good enough to understand the last section]

Thanks!

Change 497340 had a related patch set uploaded (by Jdlrobson; owner: Jdlrobson):
[mediawiki/skins/MinervaNeue@master] Images should have borders in Android app / parsoid generated content

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

@eranroz thank you for prodding this task. Unfortunately this problem is one that spans 2 teams and 2 different skillsets, so I'm guessing the slowness/lack of an ETA is not because people don't care but because it's not clear how to facilitate this change as it's not been clear who's responsibility this is. I've used my volunteer time to submit a patch since I have the CSS expertise, which I'm hoping the Android app team can test and merge to move this along (as I can't verify this fixes the problem in apps), but that depends on what other things they have on their plate. If you can help identify people who understand the problem and have sufficient understanding of testing CSS in apps to review that patch that would also be useful!

Change 497340 had a related patch set uploaded (by Mholloway; owner: Jdlrobson):
[mediawiki/skins/MinervaNeue@master] Images should have borders in Android app / parsoid generated content

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

Change 497340 merged by jenkins-bot:
[mediawiki/skins/MinervaNeue@master] Images should have borders in Android app / parsoid generated content

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

@Jdlrobson The border doesn't cover the whole circumference of the image. In a comment for PS2 of the patch you mentioned that a MobileFrontend dependency was missing. Which was it? I suspect we might need to add this to our base CSS output.

I5c1fdfc87a48764cbb53b7a751fbffc1a4e36fc5

The styles that are needed are in mediawiki.skinning.content.parsoid and skins.minerva.content.styles - if you have local copy of the latter you may need to copy/paste.

In T198534#5105470, @bearND wrote:

@Jdlrobson The border doesn't cover the whole circumference of the image. In a comment for PS2 of the patch you mentioned that a MobileFrontend dependency was missing. Which was it? I suspect we might need to add this to our base CSS output.

The border doesn't cover the whole image yet. An editor in hewiki have indicated that this is most visible with lists with small flags as in this page: https://he.wikipedia.org/wiki/%D7%91%D7%90%D7%A8%D7%98_MRAD#%D7%9E%D7%A9%D7%AA%D7%9E%D7%A9%D7%99%D7%9D
The border have some padding around the images, while in mobile and desktop version it looks well.

Change 656436 had a related patch set uploaded (by Vadim Kovalenko; owner: Vadim Kovalenko):
[mediawiki/services/mobileapps@master] Border for article thumb image

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

Change 656436 merged by jenkins-bot:
[mediawiki/services/mobileapps@master] Border for article thumb image

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