Skip to content
This repository was archived by the owner on Oct 8, 2021. It is now read-only.

Image link in dialog rewritten incorrectly #1557

Closed
mathieucarbou opened this issue May 3, 2011 · 7 comments
Closed

Image link in dialog rewritten incorrectly #1557

mathieucarbou opened this issue May 3, 2011 · 7 comments

Comments

@mathieucarbou
Copy link

I have a dialog page containing in the HTML result of the ajax call:

<img src="http://192.168.100.2:8080/service/contests/net.playtouch.contests.sample/1/img/icon_2.jpg" class="floatleft" alt=""/>

when the dialog is displayed, this is transformed into:

<img alt="" class="floatleft" src="/service/contests/net.playtouch.contests.sample/1/skin/desktop/service/contests/net.playtouch.contests.sample/1/img/icon_2.jpg">

which is incorrect. This is the same domain but a different web application.

How can I say to jquery mobiel to not change the image link ?

Mathieu.

@toddparker
Copy link
Contributor

The framework needs to re-write the paths for assets because the relative paths are different when we Ajax this into the original HTML document. We're re-factoring the URL and base tag code this week so keep an eye on this and let us know if our final code works for you.

@toddparker
Copy link
Contributor

@mathieucarbou
Copy link
Author

Thanks a lot for your fast response we really appreciate :-)

@pauln
Copy link
Contributor

pauln commented May 4, 2011

This sounds very similar to issue 1551, which only affects Firefox 3.6 (other browsers, including the mobile browsers which I have access to, are unaffected) - can you please confirm whether it's the same issue? Maybe even try the fix there and see if it fixes your problem?

#1551

@mathieucarbou
Copy link
Author

Hi,
we had the issue also in google chrome. I've applied your patch and it works well.
Thank you !

@pauln
Copy link
Contributor

pauln commented May 19, 2011

Could you please check the patch in Pull Request #1668 to see if it works instead? Rather than removing the functionality, it fixes the URL which it bases the new URL on, thereby resulting in links which work (at least in the case of Issue #1551). The change is essentially just reinstating the line(s) removed by the other patch and changing the definition of newPath from:
var newPath = path.get( fileUrl );
to:
var newPath = path.clean( path.origin );
a few lines earlier in the file.

@mathieucarbou
Copy link
Author

Hi,
It works for us!

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

3 participants