search

contao / core

Contao 3 → see contao/contao for Contao 4
GNU Lesser General Public License v3.0
492 stars 213 forks source link

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

Closed ghost closed 12 years ago

ghost commented 12 years ago

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 commented 12 years ago

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 commented 12 years ago

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 commented 12 years ago

Behoben in c3da6b88a1c14bc39db5a3c1dc8e6598.

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

ghost commented 12 years ago

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 commented 12 years ago

Die Google-Group ist die richtige Anlaufstelle.

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

leofeyer commented 12 years ago

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