-
Notifications
You must be signed in to change notification settings - Fork 34
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
Added styling enchancements to SpanArray text highlighting and rendering #205
Added styling enchancements to SpanArray text highlighting and rendering #205
Conversation
…ed javascript rendering engine with new styling
…PokkeFe/text-extensions-for-pandas into issue-201-spanarray-visual-enhancements
…eritance, and organized JS code.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looking good. Some potential issues around documents with HTML and problems displaying two notebooks at the same time to work through, but otherwise great.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
Thanks for the updates, @PokkeFe. I think this set of changes is in good enough state to merge into master. One remaining issue that we should address ASAP in a follow-on PR: Cell 17 of our intro notebook throws an exception after these changes, because displaying multiple documents is not currently implemented. It would be better to generate an HTML error message (or generate an HTML warning followed by a display of the first document), so that this example notebook runs all the way to the end. |
Converted SpanArray HTML generator to load scripts and style sheets from a dedicated resource module. Integrates base JavaScript for future dynamic behavior. Added new styles for nested and compound highlight regions.
Related to issue #201