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

fix: fix alignment of colorbar labels for categorical color column in parallel coordinates & parallel categories sections #190

Merged
merged 1 commit into from
Nov 2, 2023

Conversation

mbelak-dtml
Copy link
Collaborator

@mbelak-dtml mbelak-dtml commented Nov 1, 2023

The first and tick labels for categorical columns would previously be aligned to the ends of the colorbar. This PR ensures that the edge tick labels are aligned to the center, same as the non-edge ones are aligned.

@mbelak-dtml mbelak-dtml requested a review from lukany as a code owner November 1, 2023 13:10
@mbelak-dtml mbelak-dtml self-assigned this Nov 1, 2023
@mbelak-dtml mbelak-dtml force-pushed the feat/align-categorical-colorbar branch from fba294a to 493fee3 Compare November 1, 2023 13:12
Copy link
Contributor

@lukany lukany left a comment

Choose a reason for hiding this comment

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

  1. I would add more details to the commit message as the changelog is generated from commit messages and the current text could be ambiguous. I would mention that it affects parallel coordinates and parallel categories plots in multivariate analysis.
  2. Not sure if it's feat or fix. Personally it feels more like a fix but leaving it up to you.

@mbelak-dtml mbelak-dtml changed the title feat: align colorbar for categorical color column fix: alignment of colorbar for categorical color column in parallel categories & parallel coordinates Nov 2, 2023
@mbelak-dtml mbelak-dtml changed the title fix: alignment of colorbar for categorical color column in parallel categories & parallel coordinates fix: fix alignment of colorbar labels for categorical color column in parallel coordinates & parallel categories Nov 2, 2023
@mbelak-dtml mbelak-dtml requested a review from lukany November 2, 2023 08:56
@mbelak-dtml mbelak-dtml changed the title fix: fix alignment of colorbar labels for categorical color column in parallel coordinates & parallel categories fix: fix alignment of colorbar labels for categorical color column in parallel coordinates & parallel categories sections Nov 2, 2023
@mbelak-dtml
Copy link
Collaborator Author

  1. I would add more details to the commit message as the changelog is generated from commit messages and the current text could be ambiguous. I would mention that it affects parallel coordinates and parallel categories plots in multivariate analysis.
  2. Not sure if it's feat or fix. Personally it feels more like a fix but leaving it up to you.
  1. I would add more details to the commit message as the changelog is generated from commit messages and the current text could be ambiguous. I would mention that it affects parallel coordinates and parallel categories plots in multivariate analysis.
  2. Not sure if it's feat or fix. Personally it feels more like a fix but leaving it up to you.

Updated the PR title with mention to which plots it affects.

Changed to fix. I'm also not really sure though. Could be seen either way IMO. Fix as in fixing a plot that looks bad by default. Or adding a feature on top of plotly defaults to make the plot look better.

@lukany lukany enabled auto-merge November 2, 2023 09:40
@lukany lukany added this pull request to the merge queue Nov 2, 2023
Merged via the queue into main with commit e3a00f9 Nov 2, 2023
30 checks passed
@lukany lukany deleted the feat/align-categorical-colorbar branch November 2, 2023 09:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants