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.

Sign up for GitHub

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

Jump to bottom

fix: properly handle data URIs #27

Merged
merged 6 commits into from
May 20, 2021
Merged

fix: properly handle data URIs #27

merged 6 commits into from
May 20, 2021

Commits on May 20, 2021

  1. add a failing test for resolve url with a data uri

    @gkatsev
    gkatsev committed May 20, 2021
    Configuration menu
    Copy the full SHA
    4737bc2 View commit details
    Browse the repository at this point in the history

  2. data uri as relative url is not resolved relatively but returned

    @gkatsev
    gkatsev committed May 20, 2021
    Configuration menu
    Copy the full SHA
    49f9720 View commit details
    Browse the repository at this point in the history

  3. set baseURL to window.location when data URIs are used

    @gkatsev
    gkatsev committed May 20, 2021
    Configuration menu
    Copy the full SHA
    533ba4d View commit details
    Browse the repository at this point in the history

  4. window.location.href for IE11

    @gkatsev
    gkatsev committed May 20, 2021
    Configuration menu
    Copy the full SHA
    2a24258 View commit details
    Browse the repository at this point in the history

  5. and of course do a null check

    @gkatsev
    gkatsev committed May 20, 2021
    Configuration menu
    Copy the full SHA
    f0da568 View commit details
    Browse the repository at this point in the history

  6. add a default

    @gkatsev
    gkatsev committed May 20, 2021
    Configuration menu
    Copy the full SHA
    3ceaf8c View commit details
    Browse the repository at this point in the history