Skip to content

Visualization: Custom legend markers and colors #678

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

Closed
moontrip opened this issue Nov 9, 2021 · 6 comments · Fixed by #698
Closed

Visualization: Custom legend markers and colors #678

moontrip opened this issue Nov 9, 2021 · 6 comments · Fixed by #698

Comments

@moontrip
Copy link
Contributor

moontrip commented Nov 9, 2021

while a prototype of custom legend works are done, it is still required to add markers at legend labels like plotly's legend. A basic square markers are already made at #637 which would work for histogram, barplot, mosaic (2x2 and RxC) Vizs but still need to consider two more cases, scatter plot and boxplot Vizs. This ticket will handle the task.

@moontrip moontrip self-assigned this Nov 9, 2021
@moontrip moontrip added this to the EDA ClinEpi Phase 2 milestone Nov 9, 2021
@moontrip moontrip changed the title Visualization: Custom legend markers Visualization: Custom legend markers and colors Nov 10, 2021
@moontrip
Copy link
Contributor Author

Boxplot Viz is done via #251

@sweverschulman
Copy link

@moontrip
Before I approve this ticket could you confirm the following things.

  1. That the open dots are the correct icon in the scatter plot legend
  2. That the clickable legend thing is supposed to be there

I am attaching photos
Screen Shot 2021-11-15 at 11 24 45 AM

@moontrip
Copy link
Contributor Author

@sweverschulman Thank you for your review and tests. Sorry that it is not clear of your question 1: The attached one is boxplot and you are saying scatter plot legend. FYI, it seems that my custom legend works for the scatter plot (and possibly the latest works from developers) are not updated correctly at qa/qa.restricted: asking Dave about this. So, if you are asking about the style of scatter plot then yes they are almost identical to what the Plotly's default legend shows. You can see my screenshot in this link, which shows what the new markers/colors of custom legend for the scatter plot look like.

As for the number 2, unlike Plotly's default legend that offers clicking texts, we are not supporting it. This is the reason why we have introduced checkbox instead, which controls on-off plot trace(s).

Hope this would be helpful for you.

@sweverschulman
Copy link

thanks @moontrip. So to clarify the custom legend this.
Screen Shot 2021-11-15 at 11 40 33 AM

@moontrip
Copy link
Contributor Author

@sweverschulman yes correct. FYI, Jamie just committed his work for overall layout of plot, custom legend, bird eye plot, and table: certainly not reflected at as site yet. Also, we will disable the Plotly's default legend sooner or later and stick to the new custom legend instead for both regular plots and faceted plot.

@sweverschulman
Copy link

the new custom legend markers and colors are present in both scatterplots and boxplots
Screen Shot 2021-11-17 at 4 02 22 PM
Screen Shot 2021-11-17 at 4 02 30 PM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants