I'm using this module, which I quite like. However, one of my clients was not so happy with the "Quit bugging me" wording at the bottom of the lightbox window, which prompted me to open up the module and make some changes. I wanted to be able to change wording to better accommodate the culture of the client in various parts, and so made the following alterations, which are reflected in the attached patch and other bits:

  1. Created a collapsible fieldset for 'Message text' in the settings form.
  2. Made the heading text editable, via a textfields (one for normal, one for black plugin) in the settings form, defaulting to the current text.
  3. Made the "Easily upgrade" text editable, via a textfields (one for normal, one for black plugin) in the settings form, defaulting to the current texts.
  4. Provided textareas for the Message Body for both the white and black versions, with default value being the existing texts respectively.
  5. Made the "Explore other browsers" heading editable, again via a textfield in the settings for, defaulting to the current text.
  6. Altered code in the black plugin to actually pay attention to the "which browser to suggest" radio button, with accompanying new image files.

As a result, I also changed code to incorporate these new variables into the hook_init function, and use them in the respective javascript files. The accompanying patch leverages the previous patch of Images are pulling from off-site URLs - should pull from module itself as well.

Another advantage of doing things this way I believe is now all the text is open to being translated to another language.

I've attached a patch, as well as zipped the entire sevenUp folder, which has the code, along with the subdirectories with the images for those who want to be lazy and just copy over. :-)

Comments welcome.

shawn

Comments

berenddeboer’s picture

Status: Active » Reviewed & tested by the community

The Drupal 7 port has this patch integrated: http://drupal.org/node/941128