[WEBUI][SPARK-17342] Style of event timeline is broken

[sarutak]

What changes were proposed in this pull request?

SPARK-15373 (#13158) updated the version of vis.js to 4.16.1. As of 4.0.0, some class was renamed like 'timeline to vis-timeline' but that ticket didn't care and now style is broken.

In this PR, I've restored the style by modifying timeline-view.css and timeline-view.js.

How was this patch tested?

manual tests.

(If this patch involves UI changes, please attach a screenshot; otherwise, remove this)

• Before
  
• After
  

[robert3005]

have you seen #14791 ? Should fix the biggest offender but full clean up is definitely useful

[srowen]

@sarutak can you connect this to your JIRA in the title?

[SparkQA]

Test build #64728 has finished for PR 14900 at commit d32d1e1.

• This patch fails Spark unit tests.
• This patch merges cleanly.
• This patch adds no public classes.

[sarutak]

@robert3005 Thanks for the information. I think the PR you informed me still doesn't restore the style to what it was.

@srowen Thanks, I forgot to do it. I've connected to corresponding JIRA.

[sarutak]

retest this please.

[SparkQA]

Test build #64738 has finished for PR 14900 at commit d32d1e1.

• This patch fails PySpark unit tests.
• This patch merges cleanly.
• This patch adds no public classes.

[sarutak]

retest this please.

[SparkQA]

Test build #64748 has finished for PR 14900 at commit d32d1e1.

• This patch passes all tests.
• This patch merges cleanly.
• This patch adds no public classes.

[SparkQA]

Test build #64749 has finished for PR 14900 at commit d32d1e1.

• This patch passes all tests.
• This patch merges cleanly.
• This patch adds no public classes.

[srowen]

LGTM. In light of this change, was #14791 necessary, or at least still a valid change?

[sarutak]

@srowen Actually, I don't think that change is necessary because an existing setting encompasses that change added in #14791 .

[SparkQA]

Test build #64807 has finished for PR 14900 at commit c4ad6a1.

• This patch fails Spark unit tests.
• This patch merges cleanly.
• This patch adds no public classes.

[sarutak]

retest this please.

[SparkQA]

Test build #64820 has finished for PR 14900 at commit c4ad6a1.

• This patch passes all tests.
• This patch merges cleanly.
• This patch adds no public classes.

[srowen]

Merged to master/2.0