Page MenuHomePhabricator

Highlight an item on hover on recent changes and other cases
Open, Needs TriagePublic

Description

Initial report.

Issue
As a user, I can't easily determine the beginning and end of a recent change item. It often has to struggle to see where the entry ends when there are a lot of very verbose changes.

Proposed solution
Add css to highlight the specific line on mouse hover, e.g.:

.mw-changeslist-line:hover {
    background: lightgray;
}

Event Timeline

Restricted Application added a project: Collaboration-Team-Triage. · View Herald TranscriptNov 29 2017, 6:54 PM
Restricted Application added a subscriber: Aklapper. · View Herald Transcript

Seems fine to me. @Pginer-WMF , thoughts?

Seems fine to me. @Pginer-WMF , thoughts?

Although it may seem a small change, I'd like to check (a) the potential for distraction, and (b) how to make it work with the current highlight system we have for filters.

As it is mentioned in the discussion, the underlying issue seems to be that the current list is not easy to process visually, organising the list better seems to be a more promising approach than adding more signals on top of the current one.

In T181664#3833530, @Pginer-WMF wrote:

...As it is mentioned in the discussion, the underlying issue seems to be that the current list is not easy to process visually, organising the list better seems to be a more promising approach than adding more signals on top of the current one

.I think what you say about the results being hard to read is correct. But the triage team approved of this change only because, while its benefit may be marginal, the effort involved is equally minimal.

There is no appetite at this time for taking on the job of reorganizing the Recent Changes results, so I wouldn't spend a lot of time thinking that through (unless you are just interested as an exercise, for the future).

In T181664#3833530, @Pginer-WMF wrote:

...As it is mentioned in the discussion, the underlying issue seems to be that the current list is not easy to process visually, organising the list better seems to be a more promising approach than adding more signals on top of the current one

.I think what you say about the results being hard to read is correct. But the triage team approved of this change only because, while its benefit may be marginal, the effort involved is equally minimal.

What I was trying to point to is that it may not be that minimal (it requires to figure out what to do with the existing highlighting, when hovering a red row), and it may have side effects in terms of distraction and confusion (e.g., users expecting the whole row to be interactive, items being highlighted as the ser just scrolls down the page, etc.).

There is no appetite at this time for taking on the job of reorganizing the Recent Changes results, so I wouldn't spend a lot of time thinking that through (unless you are just interested as an exercise, for the future).

Makes sense. I agree that's outside of the current scope of the project.

Pginer-WMF wrote:

What I was trying to point to is that it may not be that minimal (it requires to figure out what to do with the existing highlighting, when hovering a red row), and it may have side effects in terms of distraction and confusion (e.g., users expecting the whole row to be interactive, items being highlighted as the ser just scrolls down the page, etc.).

Ah. Makes sense. Moving this to Freezer.

Idea: not highlight a line, but frame it?

@Pginer-WMF, would Trizek's solution work for you? If so, we can move this back to be triaged again.

Restricted Application added a project: Growth-Team. · View Herald TranscriptNov 21 2018, 2:17 PM