This module, based on Drupal 7, create a field type that give you the ability to include as a content field a jQuery gallery importing photoset directly from Flickr.
The good stuff is to use a social resource (pretty simple and really user friendly) to update only one time your photo grouped in photoset and share them in your website only taking the photoset ID given by Flickr photoset URL.
It also make possible to insert a block with one or more Gallery always from a Flickr Photoset.
The administrator page give the chance to set different features as:

  • set height, width and margin of gallery wrapper
  • set max photos number to import for single gallery
  • set gallery background color (using a dinamic color picker)
  • set one of three different color of button to have right contrast with backgorund color
  • show or not debug gallery message (javascript side) and single photo description

Link to the project page: http://drupal.org/sandbox/marco_mazzocchi/1805388
Link to git repository: git clone http://git.drupal.org/sandbox/marco_mazzocchi/1805388.git

CommentFileSizeAuthor
Schermata 11-2456233 alle 11.59.10.png328.76 KBmarco_mazzocchi

Comments

klausi’s picture

klausi
he/him||they/them
Primary language German
Location 🇦🇹 Vienna
commented
Status: Needs review » Needs work

Welcome,

We are currently quite busy with all the project applications and I can only review projects with a review bonus. Please help me reviewing and I'll take a look at your project right away :-)

Problems raised by automated tools: http://ventral.org/pareview/httpgitdrupalorgsandboxmarcomazzocchi1805388git

serm’s picture

serm commented

Hi, Marco!

  • Make module settings form of a file odtelny in your hook_menu().
  • JavaScript and CSS i would do the same is in a separate files.
  • Your module contains many variables, use the function hook_uninstall() to remove these variables while removing the module from the website.

Correct link to your module repository is: git clone http://git.drupal.org/sandbox/marco_mazzocchi/1805388.git flickr_galleria_photoset

Best Wishes,
Andrew Podlubnyj.

jamesquinton’s picture

jamesquinton commented

Hi Marco,

Thanks for your contribution. I've installed your module and taken a look, I have a few comments below...

  • As you're still using the master branch, you will need to switch to a version specific branch and remove master, e.g 7.x-1.x. Release naming conventions.
  • It would be good to separate the javascript out of the .module file, and call separate JS files instead of having the JS code inline.
  • If the photoset URL's are a standard structure, it may be more convinient for the user to copy and paste the URL from Flickr and have your module parse this for the photoset ID.
  • It would be nice to have a few more instructions and more detailed description on the project page (I had to read comments and functions in the module files to fully understand).

Other than that, looks really good and could prove very useful!

James

marco_mazzocchi’s picture

marco_mazzocchi commented

hi! thank you both for comments i'm going to work following your advises. I'll update you on my progress on this modules. thanks

klausi’s picture

klausi
he/him||they/them
Primary language German
Location 🇦🇹 Vienna
commented
Status: Needs work » Closed (won't fix)

Closing due to lack of activity. Feel free to reopen if you are still working on this application.

If you reopen this please keep in mind that we are currently quite busy with all the project applications and I can only review projects with a review bonus. Please help me reviewing and I'll take a look at your project right away :-)

Also, you should get your friends, colleagues or other community members involved to review this application. Let them go through the review checklist and post a comment that sets this issue to "needs work" (found some problems with the project) or "reviewed & tested by the community" (found no major flaws).

klausi’s picture

klausi
he/him||they/them
Primary language German
Location 🇦🇹 Vienna
commented
Issue summary: View changes

Updated link to git repository