Span node title attribute not (always?) rendered properly in mobile
Closed, InvalidPublic
Actions

Assigned To

None

Authored By

	Kelson
	Dec 26 2020, 10:45 AM

Description

At https://ru.wikipedia.org/wiki/%D0%92%D0%B7%D1%80%D1%8B%D0%B2_%D0%BD%D0%B5%D1%84%D1%82%D1%8F%D0%BD%D0%BE%D0%B9_%D0%BF%D0%BB%D0%B0%D1%82%D1%84%D0%BE%D1%80%D0%BC%D1%8B_Deepwater_Horizon

We have the wiki code:

{{Comment|5 миллионов [[Баррель (американский нефтяной)|баррелей]]|795 000 кубических метров}}

Which renders:

<span class="ts-comment-commentedText" title="795 000 кубических метров">5 миллионов <a href="/wiki/%D0%91%D0%B0%D1%80%D1%80%D0%B5%D0%BB%D1%8C_(%D0%B0%D0%BC%D0%B5%D1%80%D0%B8%D0%BA%D0%B0%D0%BD%D1%81%D0%BA%D0%B8%D0%B9_%D0%BD%D0%B5%D1%84%D1%82%D1%8F%D0%BD%D0%BE%D0%B9)" title="Баррель (американский нефтяной)">баррелей</a></span>

span node as a proper title attribute.

Desktop Parsoid render does it right as well:
https://ru.wikipedia.org/api/rest_v1/page/html/%D0%92%D0%B7%D1%80%D1%8B%D0%B2_%D0%BD%D0%B5%D1%84%D1%82%D1%8F%D0%BD%D0%BE%D0%B9_%D0%BF%D0%BB%D0%B0%D1%82%D1%84%D0%BE%D1%80%D0%BC%D1%8B_Deepwater_Horizon

But Parsoid Mobile does not:
https://ru.wikipedia.org/api/rest_v1/page/mobile-sections-lead/%D0%92%D0%B7%D1%80%D1%8B%D0%B2_%D0%BD%D0%B5%D1%84%D1%82%D1%8F%D0%BD%D0%BE%D0%B9_%D0%BF%D0%BB%D0%B0%D1%82%D1%84%D0%BE%D1%80%D0%BC%D1%8B_Deepwater_Horizon

which gives something like:

<span class="ts-comment-commentedText">5 миллионов <a href="/wiki/Баррель_(американский_нефтяной)" title="Баррель (американский нефтяной)">баррелей</a></span>

The title attribute has disappeared and it should not IMO.

Maybe this is a feature, but then class="ts-comment-commentedText" should be removed as well.

Report first at https://github.com/openzim/mwoffliner/issues/1132

Event Timeline

Kelson created this task.Dec 26 2020, 10:45 AM

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptDec 26 2020, 10:45 AM

Kelson updated the task description. (Show Details)Dec 26 2020, 10:47 AM

Kelson moved this task from TRIAGE to TOP on the affects-Kiwix-and-openZIM board.May 3 2021, 7:32 AM

Arlolra added a project: Parsoid.Jun 10 2021, 3:10 PM

Arlolra edited projects, added Parsoid (Tracking), Mobile-Content-Service; removed Parsoid, Parsoid-Rendering.Jun 22 2021, 9:04 PM

Restricted Application added a project: Product-Infrastructure-Team-Backlog-Deprecated. · View Herald TranscriptJun 22 2021, 9:04 PM

Desktop Parsoid render does it right as well:
...
But Parsoid Mobile does not:

Parsoid itself only produces what you're referring to as the desktop render. The mobile endpoints are built on top of that, through various transforms, by the Mobile-Content-Service and Page Content Service.

@Kelson As per docs for this template the tip should not be rendered on the mobile app. And it actually doesn't - see the shot. Not sure whether we need to do extra actions here.

@vadim-kovalenko Thx for coming back with this insightfull comment. Does that mean that the Wikipedians behind this template have chosen to do so... or does that mean this is not their choice (it's a feature/bug in Mediawiki) and they complain as well about it?

@Kelson I've just asked a question about this in the template discussion thread. I'll keep you posted if get any updates.

@vadim-kovalenko, have you seen any updates on this?

@JMcLeod_WMF I've just checked the conversation, I haven't gotten any response to my question about this issue for now.

It looks like the mobile-html response works properly according to the template docs and there is some inactivity on the conversation. Closing this one for now, feel free to reopen when we have more clarity for the issue.

• JMcLeod_WMF unsubscribed.Mar 30 2023, 1:28 PM

	F35057949: comment-template.jpg
	Apr 19 2022, 9:15 AM

Span node title attribute not (always?) rendered properly in mobileClosed, InvalidPublicActions

Description

Event Timeline

Span node title attribute not (always?) rendered properly in mobile
Closed, InvalidPublic
Actions