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

Adopt native adaptive serialization, add indent toggle #168

Merged
merged 7 commits into from
Jun 30, 2017

Conversation

joewiz
Copy link
Member

@joewiz joewiz commented Jun 8, 2017

Adaptive serialization with indent=yes

screen_shot_2017-06-08_at_5_11_40_pm

Adaptive serialization with indent=no

screen_shot_2017-06-08_at_5_11_59_pm

JSON with indent=yes

screen shot 2017-06-08 at 5 13 55 pm

JSON with indent=no

screen shot 2017-06-08 at 5 14 12 pm

… to use new pretty-print function signature
- This commit turns eXide’s “Output” dropdown menu into a serialization sandbox.
- It enables the user to serialize query results not just as Adaptive, JSON, XML, or “direct”, but also as Text, HTML5, XHTML, XHTML5, and MicroXML - as enumerated in https://github.com/eXist-db/exist/blob/develop/src/org/exist/util/serializer/AbstractSerializer.java.
… to use native serialization
put direct on the bottom, since it’s more a “mode” than a serialization form
@joewiz
Copy link
Member Author

joewiz commented Jun 10, 2017

With my last commits, all natively-supported serialization output methods can be selected by the user — not just Adaptive, JSON, XML, and “direct”, but also Text, HTML5, XHTML, XHTML5, and MicroXML (as enumerated in https://github.com/eXist-db/exist/blob/develop/src/org/exist/util/serializer/AbstractSerializer.java):

screen shot 2017-06-10 at 5 47 19 pm

This commit turns eXide’s “Output” dropdown menu into a serialization sandbox. Users can now experiment and see how HTML5 serialization differs from XML:

screen shot 2017-06-10 at 5 48 56 pm

screen shot 2017-06-10 at 5 49 14 pm

screen shot 2017-06-10 at 5 49 27 pm

screen shot 2017-06-10 at 5 49 40 pm

@joewiz joewiz requested a review from wolfgangmm June 29, 2017 15:07
@joewiz
Copy link
Member Author

joewiz commented Jun 29, 2017

@wolfgangmm I've tested this PR thoroughly and think it's a good candidate for including in the upcoming 3.3.0 release (scheduled for this Sunday, July 2). If you agree and merge today or tomorrow, I'm happy to prepare the release to the public-repo for inclusion in the installer.

@joewiz
Copy link
Member Author

joewiz commented Jul 1, 2017

@wolfgangmm Thank you! I've completed and uploaded the release: https://github.com/wolfgangmm/eXide/releases/tag/v2.4.0.

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