The first thing in T165535: Block notices on mobile web for logged-in users provide insufficient information about the block that should be done is fix the HTML escaping of the block reason.
Description
Description
Details
Details
| Subject | Repo | Branch | Lines +/- | |
|---|---|---|---|---|
| Block reason messages escape HTML | mediawiki/skins/MinervaNeue | master | +2 -1 |
| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| Resolved | None | T190946 Epic: Improve the mobile block experiences | |||
| Open | None | T193173 Add the 'Need Help?' Link to the Block Message drawer | |||
| Resolved | matmarex | T194566 Parse the links in the block reason on IP User block notice on mobile | |||
| Declined | None | T194530 Add the ability to return the parsed block reason to API:Userinfo | |||
| Resolved | matmarex | T193754 Block notices for IP Users on mobile web should use block notice drawer | |||
| Resolved | ABorbaWMF | T165535 Block notices on mobile web for logged-in users provide insufficient information about the block | |||
| Resolved | dbarratt | T190901 Mobile web block reason fails to display a hyperlink properly, displaying the raw HTML code for a link instead |
Event Timeline
Comment Actions
Change 422363 had a related patch set uploaded (by Dbarratt; owner: Dbarratt):
[mediawiki/skins/MinervaNeue@master] Block reason messages escape HTML
Comment Actions
@Nirzar the mobile toast should never have HTML inside it right? The above patch does this but the toast does allow for it.
Comment Actions
Another solution would be to strip the HTML so it renders in plain text (sans the escaped HTML)
Comment Actions
@Jdlrobson yes, no html inside toasts. also as per task, i thought we would be using drwer not a toast?
Comment Actions
Change 422363 merged by jenkins-bot:
[mediawiki/skins/MinervaNeue@master] Block reason messages escape HTML