-
Notifications
You must be signed in to change notification settings - Fork 33
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 local_art in Art tab and allow PNG downloads. #8
base: master
Are you sure you want to change the base?
Conversation
@@ -3808,7 +3808,11 @@ async function addTextbox(textboxType) { | |||
} | |||
//ART TAB | |||
function uploadArt(imageSource, otherParams) { | |||
art.src = imageSource; | |||
if (!imageSource.includes('http')) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The real fix for this is to actually update /creator/index.html
line 303 to use onchange='imageURL(this.value, uploadArt, "autoFit");'
instead of the uploadArt function.
@@ -698,13 +698,13 @@ <h5 class='padding input-description'>Automatically update frame</h5> | |||
<div class='readable-background padding'> | |||
<h3 class='download padding' onclick='downloadCard();'>Download your card</h3> | |||
<h5 onclick='downloadCard(false, true);' id='downloadJpg' href='' target='_blank' class='padding download input-description' style='text-align: left;'>Click here to download as JPEG</h5> | |||
<h5 onclick='downloadCard(true);' id='downloadAlt' href='' target='_blank' class='padding download input-description' style='text-align: left;'>Click here for an alternative download</h5> | |||
<h5 onclick='downloadCard(false, false);' id='downloadPng' href='' target='_blank' class='padding download input-description' style='text-align: left;'>Click here to download as PNG</h5> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This change makes it functionally the same as line 699 here.
Is this still an issue? I think this was fixed elsewhere at some point. |
This is definitely still broken, I opened #52 to fix it based on this comment |
Friend asked me to look at the local_art not working in the art tab. Was a simple oversight and has been fixed.