Project:Flickr
Version:6.x-1.x-dev
Component:flickr (main module)
Category:feature request
Priority:normal
Assigned:Unassigned
Status:patch (to be ported)

Issue Summary

Since I am new in the world of Drupal, I might have missed it. I think it would nice to have some layout options to the Flickr blocks or than the picture size:

- spacing
- alignment (centre, left, right)
- number of columns
- ...

Comments

#1

Status:active» postponed

I think this not something that will be worked on in the near future so i will change the status to postponed.

Best, Paul

#2

Version:6.x-1.0-alpha1» 6.x-1.x-dev

#3

Status:postponed» closed (fixed)

There does not look to be any interest here so i will close this feature request, at least for now

#4

I must say that I am a bit disappointed by this decision. Leaving it as postponed seemed more appropriate to me. Or maybe explain to me what I had missed about the feature or a pointer to a documentation, maybe.

Patrick

#5

Status:closed (fixed)» postponed

Sorry Patrick :-)

#6

Status:postponed» needs review

I think it would be undrupaly to add that sort of layout options to the flickr module
the proper way to do layout stuff in Drupal is with theming.

Unfortunately, the flickr module does not set a CSS class for the photos in a block, so theming the flickr blocks is a bit hard.

The patch in attachment adds the CSS class "flickr_photo" to theme_flickr_photo(), to make the theming more easy.

AttachmentSize
270843_phototheming.patch 925 bytes

#7

I'd vote for this. If it was just as simple as letting content authors float images left or right using a simple parameter, that would be great. You can't predict how people want their images layed out in the theme layer ;)

#8

I too am looking for some sort of alignment options for insertion of Flickr images in a text body (align image right or left).

#9

Will this patch allow me to space the image thumbnails apart a bit when they display on the node?

How/where do I install the patch?

#10

Status:needs review» patch (to be ported)

I'll get this patch committed to the development branch later today. Changing the status of this issue to "patch (to be ported)" to ensure that i pick this up again with our efforts to migrate this module to Drupal 7

Best , Paul