Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Redefine barchart tooltip: more understandable tooltip #475

Merged
merged 1 commit into from
Nov 1, 2022

Conversation

noelchen90
Copy link
Collaborator

Solves #426 bullet point 14.

  1. They (test users) were not familiar with some of the metrics. They would love to have more explanations (e.g. what is lexical diversity). (proposed by Ying)
  • adds y-axis label name
  • modify tooltip format

The look Before

截圖 2022-10-31 下午10 05 07

The look After

截圖 2022-10-31 下午9 56 54

Copy link
Collaborator

@pfliu-nlp pfliu-nlp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@noelchen90 noelchen90 merged commit 30fcc4d into main Nov 1, 2022
@noelchen90 noelchen90 deleted the refine-barchart-tooltip branch November 1, 2022 17:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants