Page MenuHomePhabricator

annotations should show on tab layout
Closed, ResolvedPublic8 Estimated Story Points

Description

Data annotations should show on dashiki's tab layout just like they do in the vital signs layout

Event Timeline

Milimetric triaged this task as Medium priority.Apr 10 2017, 4:13 PM
Milimetric subscribed.

Just a quick FYI: here's a bit better way to show annotations, just modifying the style:

pasted_file (310×835 px, 45 KB)

It's just an example, it should be thought through, with probably hover effects, but it seems better. This particular style was just:

width: 26px;
top: 0;
height: 100%;
opacity: 0.5;
background-color: lightgrey;

Nuria raised the priority of this task from Medium to High.Apr 10 2017, 4:16 PM

@Milimetric
Sounds good! Will try that.
It seems to me like this way would work better for long-duration anomalies, as opposed to spikes, no?

Hm, it might be tricky for ranges, because ranges right now are two annotations. So you'd have to do some CSS magic to get whatever you sandwich between two divs to stay there. I guess :after might work... can't know for sure without trying :)

For single-date annotations, maybe a dashed vertical line would work? You could do width: 0; border: 1px dashed grey; maybe

Change 348227 had a related patch set uploaded (by Mforns):
[analytics/dashiki@master] Add annotations to tabs layout

https://gerrit.wikimedia.org/r/348227

mforns set the point value for this task to 8.Apr 19 2017, 2:39 PM

Change 348227 merged by Milimetric:
[analytics/dashiki@master] Improve annotations

https://gerrit.wikimedia.org/r/348227

Change 350259 had a related patch set uploaded (by Mforns):
[analytics/analytics.wikimedia.org@master] Deploy browsers and reportcard

https://gerrit.wikimedia.org/r/350259

Change 350259 merged by Mforns:
[analytics/analytics.wikimedia.org@master] Deploy browsers and reportcard

https://gerrit.wikimedia.org/r/350259

Code has been deployed, but config can not be edited because of deployment freeze.
Will work on documentation in the meantime and leave this task in "ready to deploy" until deployment freeze finishes.

Config is broken due to Dashiki extension being broken, fixes are being deployed after freeze.