Page MenuHomePhabricator

Differential highlighting highlights wrong lines when setup warning is ignored by lackadaisical admins
Closed, ResolvedPublic

Assigned To
None
Authored By
mattrobenolt
Apr 15 2015, 6:19 PM
Referenced Files
F377083: pasted_file
Apr 20 2015, 5:44 PM
F372344: pasted_file
Apr 15 2015, 6:40 PM
F372331: pasted_file
Apr 15 2015, 6:19 PM
Tokens
"The World Burns" token, awarded by Agatek34."Yellow Medal" token, awarded by avivey.

Description

When selecting lines inside of a diff, it seems that the yellow box to indicate which rows are highlighted is off-by-one.

pasted_file (237×514 px, 19 KB)

This is in Chrome 42.0.2311.82 beta (64-bit) on OS X 10.10.3.

Event Timeline

mattrobenolt raised the priority of this task from to Needs Triage.
mattrobenolt updated the task description. (Show Details)
mattrobenolt added a subscriber: mattrobenolt.

What browser? Can you reproduce the issue here?

@chad, updated browser/OS in description, and no, I can't reproduce here. We last updated to master yesterday, then again this morning hoping it was resolved.

Generally, we don't support non stable browsers. Does it reproduce with normal Chrome?

Yeah, I've tested in every browser I have on my computer. Chrome, Firefox, and Safari. All the same issue.

All diffs on your local install, or just specific diffs?

It seems to be all diffs. But I can confirm later today if I find any that don't cause a problem.

Are you an administrator, with an "Unresolved Setup Issues..." banner showing on every page?

I need to implement T6526 so I can award myself a MASTER SLEUTH badge.

(I'm not sure if it's actually the problem, but it seems likely.)

Yeah, confirmed. If I just delete the entire .setup-warning-callout div, it's fine.

We need T7567 so we can make the setup warning include an airraid.wav file.

(This token is meant to go on epriestley's comment, but we can't do that yet either...)

chad renamed this task from Differential highlighting highlights wrong lines to Differential highlighting highlights wrong lines when setup warning is ignored by lackadaisical admins.Apr 15 2015, 7:42 PM
chad added projects: Differential, Setup.

Just a small and unnecessary rhetorical question here: do people really leave setup warnings & ignore them? I mean - i fix every setup warning the moment they appear and assumed same is true for everybody. I really wish that @chad's wish for airraid.wav would come true ;)

I think we've chased our tails twice this week alone on the issue.

In fairness to the admins, we pushed a "deprecated Conduit methods called in the last 30 days" warning recently which often isn't easy to fix since you may have to chase people down or update things.

I can't remember exactly what the issue was, but there's something funky with work boards and setup issues as well.

Issues with Workboards and Conpherence were resolved earlier this week.

In T7830#107243, @chad wrote:

Issues with Workboards and Conpherence were resolved earlier this week.

Ah cool, I don't think I've updated recently.

In fairness to the admins, we pushed a "deprecated Conduit methods called in the last 30 days" warning recently which often isn't easy to fix since you may have to chase people down or update things.

This is exactly why we have the banner on our install. :) Normally, we don't have outstanding issues.

This has definitely fixed the issue, but introduced a new, much less annoying issue. :)

pasted_file (81×307 px, 10 KB)

It seems to be that only on the Differential view, the little sidebar divider is now hovering over the banner.

Seems if I bump the z-index to 5, it displays correctly over the divider.