search

DotJoshJohnson / vscode-xml

XML Tools for Visual Studio Code
MIT License
322 stars 86 forks source link

Added XSLT Transformations #247

Closed szul closed 5 years ago

szul commented 5 years ago

Addresses #93 to a certain extent.

I was in the process of building my own extension for XSLT transformations, but thought it would be better to contribute to this extension.

When inside an XML file, you can pull up the command palette, and search for "XML Tools: Run XSLT Transformation" (or just "Run XSLT Transformation"). It will open a dialog window to select an XSLT file. Once selected it will transform the XML, open the results to the right, and then open a web view to the right of that, trying to display a rendered version.

The VSCode version had to be updated in order to use the web view control and the "BESIDE" enum for positioning of the opened window.

Two things I'd like to eventually see added: 1.) a setting for whether or not to open the web view; 2.) have the XSLT selection in the command palette rather than an open dialog.

DotJoshJohnson commented 5 years ago

Nice! I'm just now jumping back in to this project after a few months away, but I look forward to seeing this in an upcoming release. Thank you!

szul commented 5 years ago

Still some work to refine it, but its a good start. The xslt-processor modules does all the heavily lifting.

stale[bot] commented 5 years ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

szul commented 5 years ago

@DotJoshJohnson Planning to move forward with this? The bot just marked it as stale. Let me know, and if not, and I can start spinning it out into a separate XSLT extension.

stale[bot] commented 5 years ago

This issue is being automatically closed because it has not had recent activity. Thank you for your contributions.

moseskt commented 5 years ago

Hi everyone. Do we yet have a way to execute XSL transformations in VS Code in this or a separate extension? @szul @DotJoshJohnson

szul commented 5 years ago

@moseskt The maintainer of this repo uses a bot to automatically close PR that he hasn’t moved to merge. I never received a response after the first message, so I’m assuming he has no interest or time to merge this. If you need this, I can spin it out as a different extension and publish it. I was hoping to simply contribute it to a larger project.