Mediabox ignoriert .jpeg beim URL.match und oeffnet daher dann im iframe #1728

Closed
issue-bot opened this Issue Nov 29, 2011 · 6 comments

Comments

Projects
None yet
2 participants
@issue-bot
Collaborator

issue-bot commented Nov 29, 2011

in mediabox_src.js wird auf Zeile 321 die file extension .jpeg (welche richtigerweise in der dist config von TYPOlight als valid file extension gelistet ist) nicht im URL matching beruecksichtigt.
Das Resultat ist, dass die lightbox ein iframe beinhaltet, welches seinerseits das image beinhaltet, welches im FF dann sogar noch gestaucht wird.

Fix:
plugins/mediabox/js/mediabox_src.js:321

-           if (URL.match(/\.gif|\.jpg|\.png|twitpic\.com/i) || mediaType == 'image') {
+           if (URL.match(/\.gif|\.jpeg|\.jpg|\.png|twitpic\.com/i) || mediaType == 'image') {

Man koennte auch nachdenken, ob man .jpe (truncated .jpeg extension) auch noch aufnimmt.

--- Originally created by xtra on March 17th, 2010, at 03:09am (ID 1728)

@ghost ghost assigned leofeyer Nov 29, 2011

@issue-bot

This comment has been minimized.

Show comment Hide comment
@issue-bot

issue-bot Nov 29, 2011

Collaborator

Grad noch gesehen, selbes Spiel in Zeile 867 und 868

        if ((prevImage >= 0) && (images[prevImage][0].match(/\.gif|\.jpeg|\.jpg|\.png|twitpic\.com/i))) preloadPrev.src = images[prevImage][0].replace(/twitpic\.com/i, "twitpic.com/show/full");
        if ((nextImage >= 0) && (images[nextImage][0].match(/\.gif|\.jpeg|\.jpg|\.png|twitpic\.com/i))) preloadNext.src = images[nextImage][0].replace(/twitpic\.com/i, "twitpic.com/show/full");

Damit die prev und next links sauber preloaden.

--- Originally created by xtra on March 17th, 2010, at 03:18am

Collaborator

issue-bot commented Nov 29, 2011

Grad noch gesehen, selbes Spiel in Zeile 867 und 868

        if ((prevImage >= 0) && (images[prevImage][0].match(/\.gif|\.jpeg|\.jpg|\.png|twitpic\.com/i))) preloadPrev.src = images[prevImage][0].replace(/twitpic\.com/i, "twitpic.com/show/full");
        if ((nextImage >= 0) && (images[nextImage][0].match(/\.gif|\.jpeg|\.jpg|\.png|twitpic\.com/i))) preloadNext.src = images[nextImage][0].replace(/twitpic\.com/i, "twitpic.com/show/full");

Damit die prev und next links sauber preloaden.

--- Originally created by xtra on March 17th, 2010, at 03:18am

@leofeyer

This comment has been minimized.

Show comment Hide comment
@leofeyer

leofeyer Nov 29, 2011

Owner

Ist das innerhalb eines meiner Patches? Andernfalls solltest Du es unbedingt auch auf der Mediabox-Webseite melden, damit wir die Änderungen nicht nach jedem Update neu einfügen müssen.

--- Originally created on March 17th, 2010, at 11:44am

Owner

leofeyer commented Nov 29, 2011

Ist das innerhalb eines meiner Patches? Andernfalls solltest Du es unbedingt auch auf der Mediabox-Webseite melden, damit wir die Änderungen nicht nach jedem Update neu einfügen müssen.

--- Originally created on March 17th, 2010, at 11:44am

@leofeyer

This comment has been minimized.

Show comment Hide comment
@leofeyer

leofeyer Nov 29, 2011

Owner

Behoben in c3da6b8.

--- Originally created on March 17th, 2010, at 11:46am

Owner

leofeyer commented Nov 29, 2011

Behoben in c3da6b8.

--- Originally created on March 17th, 2010, at 11:46am

@issue-bot

This comment has been minimized.

Show comment Hide comment
@issue-bot

issue-bot Nov 29, 2011

Collaborator

Habe es in der google group gemeldet.
Mal sehen wann sie den post freischalten.

Kennst du noch einen anderen tracker der mediabox? Der issue tracker aug google code scheint mir naemlich verwaist zu sein und auf der Projektseite verweisen sie explizit auf die group, in welcher du seinerzeit auch gepostet hattest.

--- Originally created by xtra on March 17th, 2010, at 12:24pm

Collaborator

issue-bot commented Nov 29, 2011

Habe es in der google group gemeldet.
Mal sehen wann sie den post freischalten.

Kennst du noch einen anderen tracker der mediabox? Der issue tracker aug google code scheint mir naemlich verwaist zu sein und auf der Projektseite verweisen sie explizit auf die group, in welcher du seinerzeit auch gepostet hattest.

--- Originally created by xtra on March 17th, 2010, at 12:24pm

@leofeyer

This comment has been minimized.

Show comment Hide comment
@leofeyer

leofeyer Nov 29, 2011

Owner

Die Google-Group ist die richtige Anlaufstelle.

--- Originally created on March 17th, 2010, at 12:34pm

Owner

leofeyer commented Nov 29, 2011

Die Google-Group ist die richtige Anlaufstelle.

--- Originally created on March 17th, 2010, at 12:34pm

@leofeyer

This comment has been minimized.

Show comment Hide comment
@leofeyer

leofeyer Nov 29, 2011

Owner

--- Originally completed on March 17th, 2010, at 11:46am

Owner

leofeyer commented Nov 29, 2011

--- Originally completed on March 17th, 2010, at 11:46am

@leofeyer leofeyer closed this Nov 29, 2011

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment