search

advanced-rest-client / arc-electron

Advanced REST Client - Desktop application
Other
1.42k stars 224 forks source link

Intent to deprecate and remove: xml-viewer July 2019 #139

Closed jarrodek closed 5 years ago

jarrodek commented 5 years ago

xml-viewer component renders XML responses in ARC and API Console. In last couple of months I have received many requests to remove use of web worker. Because the component parses received data and generates a DOM structure to be rendered, and because the response can be of any size no data parsing will be performed in main thread process This most probably would freeze the UI for the time of processing data. This is an intent to deprecate the component and replace it with currently used for other media types Prism library. The only compatibility issue is that the Prism rendering do not have option to toggle nodes.

The "response-body-view" and "response-view" components that uses this component will be changed around July 2019. After this date new version will be released without additional support for XML responses other than Prism highlighting.

jarrodek commented 5 years ago

The xml-viewer was removed from current version of the response view

jagadeesh-mri commented 4 years ago

Hi @jarrodek I am still getting This view is retired and will be replaced around June 2019. message when used API console to test an API Specification designed using RAML Spec for responses of type XML. Is there any future plan to create a new view for displaying XML responses? Or this warning message is expected to be like this for upcoming months?

jarrodek commented 4 years ago

Hi @jagadeesh-mri Technically the XML view was already replaced with syntax highlighter (provided by Prism library). However, it takes much, much longer to release new version of the console. It's been a complete rewrite of the console and the teams are testing the console in multiple environments with different APIs. We also had a major change in the model the console is using to render the documentation which caused even more delays. So the ticket is closed because the next version 6 of the console has this view removed but it is not released just yet. We are planning to do this in coming weeks.