owncloud-archive / apps

Repo for ownCloud apps. Code here is work in progress and not intended for endusers
373 stars 336 forks source link

[gallery] Lightbox does not pop-up under Safari (Mac and iPad) when path has extended characters #1626

Open D1reW0lf opened 10 years ago

D1reW0lf commented 10 years ago

Environment: TurnKey Linux 13.0 / Debian 7.3 Wheezy Server: Apache/2.2.22 (Debian) Database: MySQL 5.5.33 Client: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_9_1) AppleWebKit/537.73.11 (KHTML, like Gecko) Version/7.0.1 Safari/537.73.11 OC-Version: 6.0.1 PHP-Version: 5.4.4-14+deb7u7 Gallery version: default OC 6.0.1 installation and git version as of 2014-01-23

Expected behavior: As soon as the imagens finish loading clicking on one of them should activate the lightbox with the bigger image and the slideshow controls.

Actual behavior: Under Safari (both Mac and iPad version) nothing happens

Steps to reproduce: Create a folder with extended characters on the name (for instance with é, ç, etc) and put some pictures inside. On Safari go to the pictures app, open the folder and click on a picture.

Note: Lightbox works fine with the same images when accessed thru the "Files" app. Both Firefox and Google Chrome (Mac versions) behave as expected and are also faster to display the initial folders. Google Chrome under iPad 2 crashes while trying to load the initial folder list.

DeepDiver1975 commented 10 years ago

Lightbox? You mean the slideshow - right?

Most probably a url encoding issue with Safari @icewind1991 can you have a look? THX

icewind1991 commented 10 years ago

I don't have access to safari to test it with

D1reW0lf commented 10 years ago

Hi all.

@DeepDiver1975 : yes, I meant the slideshow. I'm still used to call this sort of slideshows "lightbox" as it was one of the initial and on that time most used scripts. Just an old habit, sorry.

@icewind1991 Would it help if I create a virtual machine running OS X and give you access to it (VNC, LogMeIn, TeamViewer, etc)?

oparoz commented 9 years ago

@Fardilha - Try the master version if you can. It does include some encoding fixes.

D1reW0lf commented 9 years ago

@oparoz Will do, thanks! I'm not yet on OC 8 but I planning to update on the new couple of weeks so I'll try then.