Comments on: A Question for Shashin Users

By: Mike

Mike — Sat, 21 Nov 2009 11:48:52 +0000

Hi DuckPuppy – someone else reported the same problem with the “next” link using Google Chrome and Safari. Are you using one of those browsers? It’s a bug in webkit (the rendering engine they both use)

http://archivist.incutio.com/viewlist/css-discuss/107561

It’ll take some re-writing so I’m planning to fix it in the next verison.

I haven’t heard of the other problem you described. I took a quick look at your site – I didn’t see where you’re using Shashin. If you can point me to it I’ll take a look.

By: Mike

Mike — Sat, 21 Nov 2009 11:31:45 +0000

Hi Christian – new version probably in January. I’ve had other things going on and haven’t worked on my plugins for several months, but I started on the new version last week. I’m adding Twitpic and Flickr support.

By: DuckPuppy

DuckPuppy — Fri, 20 Nov 2009 03:13:43 +0000

Love this plugin, but I do have one problem – when I limit the number of photos per page in the “Album Photos Settings” they get limited properly when I click on an album thumbnail, but the “Next>>” text is not clickable so I can’t get to the other pages.

Also, I can’t find a reliable way to reproduce this, but I can get into a state where clicking on “Return to album list” does nothing but refresh the current page (in other words, it keeps showing the items in the album rather than returning to the page). The target for the link is correct, but the only way to fix it is to return to the main page and click on the page link for my album list again.

By: Christian

Christian — Thu, 19 Nov 2009 17:46:33 +0000

No problem, I can probably fudge the formatting to accommodate the standard class to cope for now, but it would be a great feature to have. Obvious next question: What’s the likely ETA on the next version? .. great plugin otherwise btw!

By: Mike

Mike — Thu, 19 Nov 2009 17:32:21 +0000

Sorry Christian, there’s no easy hack to provide for that. It would require touching code in a number of places. But I’ve made a note to add a “custom class” option in the next version, for all the Shashin tags – it’s a good idea.

By: Christian

Christian — Thu, 19 Nov 2009 15:08:05 +0000

*almost* ..it returns the single image okay, but the img tag uses the shashin_image CSS class so it’s still subject to the same style formatting as standard galleries. How awkward is it to pass an additional parameter in the shortcode/syntax to specify a custom CSS class?

By: Mike

Mike — Thu, 19 Nov 2009 14:18:50 +0000

Hi Christian – in the next version I’ll adjust the code so that it doesn’t create a table when it’s showing just one picture. But I’m adding a lot of features to the next version, so it won’t be ready really soon.

In the meantime, if you don’t mind editing some code, here’s a hack that should do it: open ShashinPhoto.php and go to the end of the getRandomMarkup function. Replace the “return” statement on line 267 with:

if (count($photos == 1)) {
    $match['photo_key'] = $photos[0]['photo_key'];
    $photo = new ShashinPhoto();
    return $photo->getPhotoMarkup($match);
}

return ShashinPhoto::_getTableMarkup($photos, $match);

I have not tested this – if it doesn’t work, let me know, and I’ll take a closer look.

By: Christian

Christian — Thu, 19 Nov 2009 14:00:41 +0000

Hi Mike. I’m looking for something that gives me a single random image with no table structures. I can collapse the table borders Shashin creates with CSS, although the resulting object behaves strange (probably my CSS, I’m no expert). However, this breaks the CSS used by the plugin for all galleries and I’d still like to use Shashin to generate standard galleries in other parts of the site. Having a method (clean or dirty) to disable tables when i’ve entered the shortcode for a 1 row, 1 col gallery would be the goal. Any ideas?

By: Mike

Mike — Thu, 19 Nov 2009 12:06:13 +0000

Hi Geir – Shashin is internationalized, and currently comes with 3 translations – Dutch, Belarusian, and Russian. Let me know if you’d like to contribute a translation, and I can let you know how to do it.

By: Mike

Mike — Thu, 19 Nov 2009 11:59:08 +0000

Hi Christian – do you mean that you’d like to not have the photos in a table if it’s only one image, or that you’d like to not have the table at all, regardless of how many images? I haven’t found a good CSS-only, cross-browser compatible solution to handle a variable, user-controllable number of columns, which is why I’m using tables. But I could look into not using a table if there’s just one image.