Enable Cross Platform (Issue #30)#31
Conversation
(To Replace PBIXInspectorWinLibrary).
Changes Constants to use / path instead of \ to be platform agnostic.
…o allow for System Agnostic applications.
…rence error presents in Linux.
|
Many thanks Rhys, I will review asap :) This had been on my to-do list and I'm really grateful for your contribution. |
|
Thanks again Rhys, identified some issues i.e. 1) unit test fails due to win library rename, 2) font size in Skia wireframe pages is smaller than original, 3) as yet unidentified cause of error when rendering wireframes for a local report.json file. Shall I commit fixes to your branch first? |
Hey, ah great - thanks for spotting. Feel free to check in any already made fixes to the branch, I'll take a look into the local report.json failure as soon as I get a chance. |
|
Hi @RhysTabor-dev , FYI for now I've ported your code to PBI Inspector V2 as this is where most of my spare time is invested currently, see https://github.com/NatVanG/PBI-InspectorV2/tree/remove-win-dependencies (merge into the main branch is pending but shouldn't be long). I've used dependency injection to run the cross-platform solution with the CLI while retaining the windows-specific drawing library for the Windows Form (as there are some rendering differences and I suspect the wireframe output is mostly used with the Windows Form app rather than the CLI). Thank you very much for your contribution. |
|
Hey @NatVanG thanks for sorting this across. I'll pull it all down this side and take another look, it's been a while since I had a chance to sit down with this change. Appreciate your patience on this, and love the effort of extending to all fabric items. Lots of exciting developments from yourself and Microsoft on the analytics space. |
Changes
/instead of Back Slash\The intention of this change is in relation to Issue #30 to enable Cross-Platform output of all formats, including HTML and Images.