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

ampersand in url in sites.json breaks icon... #82

Closed
trip5 opened this issue Mar 27, 2023 · 5 comments
Closed

ampersand in url in sites.json breaks icon... #82

trip5 opened this issue Mar 27, 2023 · 5 comments
Labels
bug Something isn't working fixed - pending release verified This bug can be recreated

Comments

@trip5
Copy link

trip5 commented Mar 27, 2023

Not sure what's going on here and at first I thought something was wrong with my json... but simple enough to fix I hope? An ampersand in the url breaks the icon... removing the & from the link allows the icon to appear.

Here are some examples where the default icon gets displayed instead of the correct icon. BTW moving around the entries' order (ie putting the url last) doesn't change anything.

        {
            "name": "Calendar",
            "url" : "https://www.google.com/calendar/render?tab=wc&pli=1&hl=en",
            "icon": "calendar.svg",
            "tags": ["home"]
        },
        {
            "name": "Amazon",
            "url" : "http://www.amazon.com/b?ie=UTF8&node=230659011",
            "icon": "amazon.svg",
            "tags": ["home","shopping"]
        },

BTW, really like Jump.

@daledavies daledavies added the triage Currently trying to determine if this can be recreated label Mar 27, 2023
@daledavies
Copy link
Owner

Howdy, I'll take a look to see if I can recreate this.

Thanks for reaching out and for the kind words about Jump 😃

@daledavies
Copy link
Owner

Thanks for reporting this, I found the bug and fixed it. The patch will be bundled into the next release :)

@daledavies daledavies added bug Something isn't working verified This bug can be recreated fixed - pending release and removed triage Currently trying to determine if this can be recreated labels Apr 4, 2023
@trip5
Copy link
Author

trip5 commented Apr 5, 2023

Awesome. Thanks so much. Just waiting on your docker push. ^^

@daledavies
Copy link
Owner

Just have to write the docs for a few new features in the next release, hoping to do it this weekend 😃

@daledavies
Copy link
Owner

I know it has been a while but this is now available in v1.4.x so I'm closing this issue. Of course feel free to reopen if you need to. Thanks for helping out 😁

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working fixed - pending release verified This bug can be recreated
Projects
None yet
Development

No branches or pull requests

2 participants