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

Restored x-axis scale labels for term bars #200

Merged
merged 1 commit into from Mar 23, 2021
Merged

Conversation

ZechyW
Copy link
Contributor

@ZechyW ZechyW commented Mar 16, 2021

The js changes in v3.2.2 removed the x-axis scale frequency ticks (as can be seen in the final screenshot on #194).

This PR replicates the v2 behaviour by restoring the x-axis scale labels for term bars, without tick lines and without causing the y-axis labels to disappear.

- The x-axis scale labels for term bars are now visible again, but without
  tick lines and without causing y-axis labels to disappear.
@msusol
Copy link
Collaborator

msusol commented Mar 23, 2021

Do you have a screenshot showing before/after?

@ZechyW
Copy link
Contributor Author

ZechyW commented Mar 23, 2021

Sure, before:

pyLDAvis-old

After:

pyLDAvis-new

@msusol msusol merged commit 51de40b into bmabey:master Mar 23, 2021
@msusol msusol self-assigned this Mar 23, 2021
@ZechyW
Copy link
Contributor Author

ZechyW commented Mar 24, 2021

Thanks @msusol! Any chance you could publish a patch release on PyPI as well?

@msusol
Copy link
Collaborator

msusol commented Mar 24, 2021

See v3.3.1

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.

None yet

2 participants