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

fix(rsg): Support relative paths in <script/> tags #474

Merged
merged 3 commits into from
Jul 27, 2020

Conversation

sjbarag
Copy link
Owner

@sjbarag sjbarag commented Jul 25, 2020

RBI allows relative paths in <script/> tags, resolved relative to the XML file that contains the element. We should support that too, because SGDEX makes heavy use of relative paths (also it's really handy :D)

fixes #471

RBI allows relative paths in <script/> tags, resolved relative to the
XML file that contains the element.  We should support that too, because
SGDEX makes heavy use of relative paths (also it's really handy :D)

fixes #471
@sjbarag sjbarag added bug Any difference between this BrightScript implementation and RBI, or otherwise unexpected behavior enhancement Any addition to this project's existing capabilities scenegraph Affects this project's implementation of the SceneGraph framework labels Jul 25, 2020
Copy link
Collaborator

@alimnios72 alimnios72 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wow that was a quick turnaround

@sjbarag sjbarag merged commit e2cf3a9 into master Jul 27, 2020
@sjbarag sjbarag deleted the support-relative-paths-in-script-uri branch July 27, 2020 16:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Any difference between this BrightScript implementation and RBI, or otherwise unexpected behavior enhancement Any addition to this project's existing capabilities scenegraph Affects this project's implementation of the SceneGraph framework
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add support for relative URIs in <script> tags
2 participants