AnnotationWindow renders appropriately

pull/10616/head
Luke Morris 2018-02-19 10:09:32 -08:00
parent fe24ba6dfc
commit b1096b2ba2
2 changed files with 12 additions and 10 deletions

View File

@ -193,21 +193,22 @@ export const annotationWindowStyle = (annotation, dygraph) => {
// TODO: export and test this function
const [startX, endX] = dygraph.xAxisRange()
const containerLeftPadding = 16
const windowEnd = Number(annotation.time) + Number(annotation.duration)
const startTime = +annotation.startTime
const endTime = +annotation.endTime
let windowStartXCoord = dygraph.toDomXCoord(annotation.time)
let windowEndXCoord = dygraph.toDomXCoord(windowEnd)
let windowStartXCoord = dygraph.toDomXCoord(startTime)
let windowEndXCoord = dygraph.toDomXCoord(endTime)
let visibility = 'visible'
if (annotation.time < startX) {
if (startTime < startX) {
windowStartXCoord = dygraph.toDomXCoord(startX)
}
if (windowEnd > endX) {
if (endTime > endX) {
windowEndXCoord = dygraph.toDomXCoord(endX)
}
if (windowEnd < startX || annotation.time > endX) {
if (endTime < startX || startTime > endX) {
visibility = 'hidden'
}

View File

@ -75,10 +75,11 @@ class Annotations extends Component {
onDeleteAnnotation={handleDeleteAnnotation}
/>
)}
{annotations.map((a, i) => {
return a.duration
? <AnnotationWindow key={i} annotation={a} dygraph={dygraph} />
: null
{annotations.filter(a => !a.id.includes('-end')).map((a, i) => {
return (
a.startTime !== a.endTime &&
<AnnotationWindow key={i} annotation={a} dygraph={dygraph} />
)
})}
</div>
)