Maniphest T202351

Multiple issues box icon does not always match contents which might be confusing
Closed, InvalidPublic
Actions

Assigned To

None

Authored By

	Jdlrobson
	Aug 21 2018, 12:09 AM

Description

When visiting
http://reading-web-staging.wmflabs.org/w/index.php/Musical_film

The banner displays like this

Screen Shot 2018-08-20 at 5.01.48 PM.png (171×427 px, 25 KB)

but upon expecting the overlay the 2 icons inside the issues do not match:

Screen Shot 2018-08-20 at 5.02.13 PM.png (333×210 px, 31 KB)

The problem stems from the fact that

sweep
is a low issue
balance
is a medium issue
we show the generic icon (in this case medium issue)

Developer notes

Showing the balance icon alongside "This article has multiple issues. Please help improve it or discuss these issues on the talk page." would be a little confusing

A few options

Update design to show all icons
Take the icon associated with the highest priority issue (in this case the balance icon) - possibly a little confusing.
Keep as is.

Related Objects

Mentioned In: T191528: Mobile page issues - allow page issues to display different severity levels

Event Timeline

Jdlrobson created this task.Aug 21 2018, 12:09 AM

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptAug 21 2018, 12:09 AM

Jdlrobson mentioned this in T191528: Mobile page issues - allow page issues to display different severity levels.Aug 21 2018, 12:09 AM

Jdlrobson added a comment.Aug 21 2018, 12:13 AM

This comment was removed by Jdlrobson.

I did not initially notice that this is a medium issue. This is the expected behavior, imo:

Given a page with an issue with severity X and severity Y, if severity X > severity Y, we display the generic icon for severity X

The reasoning here is that the icons do not need to match the icons of the issues themselves, as displaying the icon for one issue but not the other may be misleading. Rather, the icons need to display the overall severity of the group of issues, which is the severity of the issue with highest level severity.

ovasileva triaged this task as High priority.Aug 21 2018, 8:33 AM

So this is working as expected?

Just needed confirmation for multiple cases. Confirmed the following:

Medium and low shows medium
Low and low shows low
Low and notice defaults to low

Couldn't find a sample for high and medium/low, but I think we can assume it will work based on the above. Closing this, but feel free to reopen if there's remaining concerns.

	F25188576: Screen Shot 2018-08-20 at 5.07.58 PM.png
	Aug 21 2018, 12:09 AM

	F25188546: Screen Shot 2018-08-20 at 5.02.13 PM.png
	Aug 21 2018, 12:09 AM

	F25188543: Screen Shot 2018-08-20 at 5.01.48 PM.png
	Aug 21 2018, 12:09 AM

	F25188574: Screen Shot 2018-08-20 at 5.07.56 PM.png
	Aug 21 2018, 12:09 AM

Multiple issues box icon does not always match contents which might be confusingClosed, InvalidPublicActions

Description

Developer notes

Related Objects

Event Timeline

Multiple issues box icon does not always match contents which might be confusing
Closed, InvalidPublic
Actions