Skip to content
This repository has been archived by the owner on Sep 7, 2024. It is now read-only.

Default HTML Template shouldn't use %5C (backslash) to reference images in pages subdirectory #512

Closed
prikhi opened this issue Mar 13, 2015 · 2 comments

Comments

@prikhi
Copy link
Owner

prikhi commented Mar 13, 2015

From bigsteve...@gmail.com on November 13, 2012 18:09:59

What steps will reproduce the problem? 1. Create any Pencil project.
2. Export the project using the default HTML template.
3. Host the generated HTML (index.html, plus Resources and pages directories) on a web server such as Apache.
4. Use a browser to access the generated index.html hosted by the web server. What is the expected output? What do you see instead? The expected output is the same as if you accessed index.html by opening it from the Windows file system using a web browser. What you see instead is a web page with all broken images. This is because pages\pencil_page.png (for example) is not a valid URL for the src attribute of an HTML tag. The generated HTML should not be using the Windows path component separator (i.e., backslash), but rather it should be using the HTML path component separator (i.e., slash). When I manually changed all the occurrences in index.html of %5C associated with the src attribute to /, then the images were correctly display both when index.html was hosted by a web server, and when accessed by a browser directly from the Windows file system. What version of the product are you using? On what operating system? Evolus Pencil 2.0.2 on Windows XP Service Pack 3 Please provide any additional information below. Hopefully, no additional information is necessary :-) It sounds like a pretty simple fix.

Original issue: http://code.google.com/p/evoluspencil/issues/detail?id=512

@prikhi
Copy link
Owner Author

prikhi commented Mar 13, 2015

From bigsteve...@gmail.com on November 13, 2012 15:14:54

Looks to be a duplicate of 222, 225, and 237.

prikhi added a commit that referenced this issue Sep 21, 2015
Fix baseRasterizedExporter.js to write correct image URLs

Fixes #225 #222 #642 #512
@prikhi
Copy link
Owner Author

prikhi commented Sep 21, 2015

Fixed by #737

@prikhi prikhi closed this as completed Sep 21, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

1 participant