Some improvements

Resolved ktecho
(@ktecho)

7 years, 10 months ago

Hey there,

I want to report some things so they can be fixed / improved:

You can open my test page: https://www.aalquilar.es/album/

1- On my desktop, if I click on the first picture and go clicking on the right arrow, I can see the pictures in order: 1, 2, 3, 4… But in my mobile phone it goes two at a time: 1, 3, 5, 7, 9… I think the problem is maybe that the plugin is registering two events (click and touch?) and that’s why when I click on the arrow, two pictures are advanced.

2- On mobile devices, if you have a thumbnail view, pictures open in the lightbox on ontouch event (I think), but maybe it would be better to open the lightbox with the image “onrelease”, because pictures are opening right now when I scroll the thumbnail down with the finger. You can test it in the same url mentioned above. Just open the url and start going down in the page. If accidentally you touch a picture while scrolling, the picture will open.

3- This is a minor one: if you have the black background color in the lightbox (as shown in my test), you can see that Twitter and G+ icons are ok, but Pinterest icon has white corners, so the effect is not too good. Maybe this needs to be transparent, so it fits well with black and white backgrounds.

https://wordpress.org/plugins/wp-photo-album-plus/

Viewing 15 replies - 1 through 15 (of 15 total)

Plugin Author Jacob N. Breetvelt
(@opajaap)

7 years, 10 months ago

1. Touché. Will be fixed in the next version. (6.4.18).
2. It is just an <a></a> tag around the photo <img .. /> with href="url to the photofile" that will be catched by the lightbox js.
3. Will be implemented by border-radius.

It is only a few weeks that i finally have a smartphone, so in the past it was kind of guessing where to add touch events. Now i can veryfy and fix these kind of things. Thanx for the reports. If you find more, please feel welcome.

Thread Starter ktecho
(@ktecho)

7 years, 10 months ago

Hi Jacob,

Thanks for everything. If you want me to test fixes and report, just tell me.

I didn’t understand the reply to point 2. Is it something I have wrong? I already know that there is a link on the pictures, but the link is catch when I’m scrolling, opening the lightbox. It should open just when I touch the picture and release the finger, not when I’m scrolling. Maybe this get fixed when you fix point 1.

Plugin Author Jacob N. Breetvelt
(@opajaap)

7 years, 10 months ago

No, you are not wrong, but it is not a touch event. What i mean is that the thumbnail is wrapped in an <a> tag, like a text, so it is an ordinary link. When hit, it is executed, like a link, but now it is an image.

Thread Starter ktecho
(@ktecho)

7 years, 10 months ago

Will it be fixed too?

Thread Starter ktecho
(@ktecho)

7 years, 10 months ago

@jacob, 1 and 3 have been fixed in the update from one hour ago. Thanks a lot.

I don’t know if related with those latest changes, I’ve detected a problem. It happens just in my smartphone and not in my desktop PC.

You can go to this url:

https://www.aalquilar.es/album-de-fotos-de-fuencarral/

If you touch the first picture, you can see it opens fine, but the “loading” spinner wheel keeps rolling. I was thinking it has something to do with the preload of pictures, maybe?

Plugin Author Jacob N. Breetvelt
(@opajaap)

7 years, 10 months ago

It is just loading the next image. Note that the landscape image is extremely large (4128 x 2322). This takes long to load.
Tick Table I-A2: Resize on upload, and set Table I-A3 to 1024 x 768
Then run Table VIII-A11: Remake.

Thread Starter ktecho
(@ktecho)

7 years, 10 months ago

I think that’s not the problem. I’m on fiber so load speed is very fast. I’ve been whith the “loading” wheel for more than two minutes, but I think it will stay that way forever. The picture is 3,5 MB of space, so it should take less than a second to load.

Plugin Author Jacob N. Breetvelt
(@opajaap)

7 years, 10 months ago

Try http://beta.opajaap.nl/ click on Slideshow of the Birds album, click on the slide will open lightbox.

Thread Starter ktecho
(@ktecho)

7 years, 10 months ago

In my smartphone, the wheel spins for 3 seconds, then dissapears. Using the album in my site, the wheel spings forever 🙂

Plugin Author Jacob N. Breetvelt
(@opajaap)

7 years, 10 months ago

I did a few more tests and there is indeed a problem at startup of the lightbox. Will get into this.

Thread Starter ktecho
(@ktecho)

7 years, 10 months ago

Thanks a lot Jacob.

Plugin Author Jacob N. Breetvelt
(@opajaap)

7 years, 10 months ago

Please try the current dev version 6.4.19.005

From the readme:
= 6.4.19 =

= Bug Fixes =

* The display of thumbnails of any kind of virtual album will not have upload or create subalbum links.
* The display of the upload and create links can now be set at the top or bottom of the thumbnail area. Table II-D19.

= New Features =

* Table VIII-A19.x allows you to move all photos from one album to another, without being timed out.
The move does not check for duplicates and does not update timestamps, but it does move sourcefiles if present.

= Other Changes =

* Improved behaviour of ugly browse buttons, both on pc and mobile.
* Lighbox on mobile now always runs in (padded) fullscreen mode. There is only a quit icon in the upper right corner. You can swipe left/right to browse through the images.

Thread Starter ktecho
(@ktecho)

7 years, 10 months ago

* The display of thumbnails of any kind of virtual album will not have upload or create subalbum links.
— Fixed

* The display of the upload and create links can now be set at the top or bottom of the thumbnail area. Table II-D19.
— Works ok. Would be nice to have an id in the link element, so I can apply css to it, but it works ok. Thanks!

* Table VIII-A19.x allows you to move all photos from one album to another, without being timed out.
The move does not check for duplicates and does not update timestamps, but it does move sourcefiles if present.
— I’ve not tested this one.

* Improved behaviour of ugly browse buttons, both on pc and mobile.
— I don’t know what this is.

* Lighbox on mobile now always runs in (padded) fullscreen mode. There is only a quit icon in the upper right corner. You can swipe left/right to browse through the images.
— It works ok, but have a few problems:
1- I cannot see the quit icon, so I cannot close the lightbox.
2- If I rotate the smarphone, the picture doesn’t fit. It moves out of the view.
3- At the beggining, it asks the user it he wants to let the site put fullscreen. This last one is not bad. Would be better to not have that, but is not bad.
Plugin Author Jacob N. Breetvelt
(@opajaap)

7 years, 10 months ago
* Ok

* use class .wpp-upload-thumb You can add css in Table IV-A15. Make sure you add !important to overrule other css. Example i use on theme 2016 for these links on the cover:
```
.wppa-album-cover-link {
font-size:14px !important;
font-weight:bold !important;
text-transform:uppercase !important;
text-decoration:none !important;
line-height:1.75em !important;
}
```
* Works

* Set Table II-13.1 browse arrows on normal slideshow

* 1. Top right, a grayish cross
2. It does on my Samsung android phone. What device are yu testing it on?
3. That is (a security protection) by the browser, i have no influence on it.
Thread Starter ktecho
(@ktecho)

7 years, 10 months ago

Hey there,

It seems problems were caused by .css being cached or something like that.

– Gray cross. now it appears in my smartphone.
– Rotation works ok.

Point 2 is the only one that I can see here: while scrolling down, pictures open if I touch them.

Thanks a lot for the fixups!

Viewing 15 replies - 1 through 15 (of 15 total)

The topic ‘Some improvements’ is closed to new replies.