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

SpanArray HTML rendering interactivity and view instance tools #215

Merged
merged 7 commits into from
Jul 1, 2021

Conversation

PokkeFe
Copy link
Collaborator

@PokkeFe PokkeFe commented Jun 28, 2021

For issue #202

Added interactivity to the JS-generated HTML rendering of SpanArrays and TokenSpanArrays, including table controls like visibility toggling and highlight locking.

Removed JupyterLab-specific CSS overrides in favor of more consistent styling across themes, and made some general styling adjustments.

Added highlights as a view instance property, preserving highlight locks over multiple view renders, but isolating the locks to the instance (Multiple JPL windows of the same notebook can have separate Spans hidden or highlighted at the same time).

Compatibility verified in JupyterLab and VSCode

Copy link
Member

@frreiss frreiss left a comment

Choose a reason for hiding this comment

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

Some minor suggested tweaks/concerns -- see inline comments -- but overall this is ready to merge.

text_extensions_for_pandas/resources/span_array.js Outdated Show resolved Hide resolved
text_extensions_for_pandas/resources/span_array.js Outdated Show resolved Hide resolved
@frreiss frreiss merged commit e414fcd into CODAIT:master Jul 1, 2021
@PokkeFe PokkeFe deleted the issue-202-interactive branch July 20, 2021 17:29
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