Jump to content

Bug in default image grid mode: vertical list (verbose)


SamC
 Share

Recommended Posts

I like to use the vertical list for images at full size on the slider. I forget to fill in a description if I can't see that field and not found a way to make just this part required so at least I'd get a reminder. Anyway, I like the image nice and big:

59d8ea8f7a64e_ScreenShot2017-10-07at15_51_12.thumb.png.01d7310d0e1be61086870c67f058646f.png

...but every time I save the page, it reverts to a smaller image:

59d8ea864ec1a_ScreenShot2017-10-07at15_51_04.thumb.png.d5b54496e7efccab5f77e57ad87cabc1.png

Can anyone reproduce this? It happens to me with Reno theme and the default classic theme.

  • Like 1
Link to comment
Share on other sites

6 hours ago, SamC said:

Can anyone reproduce this?

Yep, it's been working like this ever since it was released as far as I can remember. Even worse, it does tend to shrink into 60 by 60 pixels (in the case of a square image) when it is set to smallest.

Link to comment
Share on other sites

  • 3 weeks later...

So I looked into this a bit but I'm way over my head staring at the /wire/modules/Inputfield/InputfieldImage js file. What I did notice was that after I dragged the slider fully to the right, the left column is 39% and the right column is 61% (at the bottom). These change depending on the slider value, but both together equal 100% width.

1.thumb.png.fad987315a16d232e04e8b4c85ca9b22.png

When the page is saved, the width of the left column always reverts to 260px wide.

2.thumb.png.045fd8b247febc91204ca0303b3095ee.png

I'm having no luck tracking down where this 260px comes from in the first place, so would need someone more experienced to look at this.

Link to comment
Share on other sites

13 hours ago, gmclelland said:

Don't know the fix, but here is the issue for this https://github.com/processwire/processwire-issues/issues/331

That's actually a different issue with a different cause.

But I think I've found fixes for both issues. :)

The new issue is here: https://github.com/processwire/processwire-issues/issues/419

  • Like 3
Link to comment
Share on other sites

Hmmm, I tried it and it didn't seem to fix :(

What I noticed is that when I save the page, the image starts at the correct previous size, then resizes a fraction of a second later to a smaller version, so I:

setGridSize($inputfield, size, ragged);

if($inputfield.hasClass('InputfieldImageEditAll') || mode == 'list') {
  var listSize = getCookieData($inputfield, 'listSize');
  setListSize($inputfield, listSize);
  console.log("setListSize ran");
}
else {
  setGridSize($inputfield, size, ragged);
  console.log("setGridSize ran");
}

...and the output showed that this runs twice (I never noticed it runs before the if block).

1.thumb.png.2be3f8d888d040b504415867f1e2244d.png

Link to comment
Share on other sites

Removing the first run of 'setGridSize' function above this block seems to work:

//console.log('initInputfield');
//console.log($inputfield);
//setGridSize($inputfield, size, ragged); << THIS

if($inputfield.hasClass('InputfieldImageEditAll') || mode == 'list') {
  var listSize = getCookieData($inputfield, 'listSize');
  setListSize($inputfield, listSize); // runs when 'verbose list'
  console.log("setListSize ran"); 
}
else {
  setGridSize($inputfield, size, ragged); // runs when 'square grid images' and 'proportional'
  console.log("setGridSize ran");
}

Work for all views now at my end.

Link to comment
Share on other sites

7 minutes ago, SamC said:

Removing the first run of 'setGridSize' function above this block seems to work

Yes, the suggested fix in the GitHub issue is to replace the all the code in the "problem" block with the code in the "fix" block.

Link to comment
Share on other sites

8 minutes ago, Robin S said:

Yes, the suggested fix in the GitHub issue is to replace the all the code in the "problem" block with the code in the "fix" block.

Ha, yep. Guilty of not reading the issue properly until after posting...

Thanks @Robin S this issue was really offputting for me.

Link to comment
Share on other sites

Create an account or sign in to comment

You need to be a member in order to leave a comment

Create an account

Sign up for a new account in our community. It's easy!

Register a new account

Sign in

Already have an account? Sign in here.

Sign In Now
 Share

  • Recently Browsing   0 members

    • No registered users viewing this page.
×
×
  • Create New...