Customized Logo provides a custom block for which the user can set from the block configuration page custom logos and the interval of time when a specific logo should be displayed on site.

After installing the module you can configure the customized logo block at example.com/admin/structure/block/manage/customized_logo/customized_logo_block/configure.
There can be added a default logo and the personalized logos with the interval in which they are active. Also, for each logo you can specify the title and alt attribute. If no image is set, the block will use the logo uploaded using the theme's global settings form or the current theme's default logo.

Dependencies
Block (Core)
Lazy Vars (Contrib)

Project page: https://drupal.org/sandbox/adinan/2169425
Repository: git clone --branch 7.x-1.x adinan@git.drupal.org:sandbox/adinan/2169425.git customized_logo

Comments

PA robot’s picture

PA robot commented
Status: Needs review » Needs work

There are some errors reported by automated review tools, did you already check them? See http://pareview.sh/pareview/httpgitdrupalorgsandboxadinan2169425git

We are currently quite busy with all the project applications and we prefer projects with a review bonus. Please help reviewing and put yourself on the high priority list, then we will 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" (they found some problems with the project) or "reviewed & tested by the community" (they found no major flaws).

I'm a robot and this is an automated message from Project Applications Scraper.

adinan’s picture

adinan commented

The Code Review errors were solved.

adinan’s picture

adinan commented
Status: Needs work » Needs review

The Code Review errors were solved.

tallosoft’s picture

tallosoft commented

Looked pretty good overall to me. I read through the code, but did not install it.

One thing I might mention is adding the dependencies to the README file. You have them on the project description page, so just add the same to the readme so all the info is right there for someone installing it and working with it! Good job!

tallosoft’s picture

tallosoft commented
Status: Needs review » Needs work
adinan’s picture

adinan commented

Hi tallosoft,

Thanks for taking the time to review my code.

I have update the README file with the module dependencies information.

Thanks again for your comment, let me know if there is something else.

adinan’s picture

adinan commented
Status: Needs work » Needs review

README file was updated with the module dependencies information.

freddybushboy’s picture

freddybushboy commented
Status: Needs review » Needs work

Hello,

I installed your module and it seems to be working fine and meets standards - though you do still need to delete your master branch - see step 6 and 7 in http://drupal.org/node/1127732.

A couple things I would suggest though:

1) Is the dependency on lazy_vars absolutely necessary? I think people would be less inclined to install your module with this dependency. I have never used lazy_vars before so this is just a thought.

2) It might be nice to be able to also set the start and end times as well as days. I agree that in most use cases of this module midnight would probably be the desired times but who knows? Maybe midnight could just be the default.

PA robot’s picture

PA robot 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 (see also the project application workflow).

I'm a robot and this is an automated message from Project Applications Scraper.