Two suggested and similar feature proposals:

  1. the default (out of the box) CSS to more closely resemble the look and feel of sites by using the core Drupal classes / ids;
  2. the option for admins to select from pre-supplied styles which complement / copy the most widely used Drupal themes (the QuickTabs module has a similar feature.

Some untested code which needs testing (and based on v1.1, so needs to also be brought into line with v1.2) is attached.

Comments

callison’s picture

StatusFileSize
new20.75 KB

Thanks for the headstart, lliss. I have reviewed the code and made some changes. It is now working fine on 6.x and is re-rolled against v 1.2 as requested. I'm still really new to git, so sorry if I didn't do something right and please bear with me as I learn.

You can see the changes in the patch, but it's a bit long, so I'll summarize here for ease sake:

  1. Install file was added to set the initial style value
  2. CSS file has been added to extensively to provide style options and to style the settings page a bit
  3. Pullquote styles are now pulled from a variable that can be set on the settings page (admin/settings/pullquote)
  4. The settings page contains a dropdown list of available styles and a preview (with dummy text) to see what the changes will look like. Whenever the form is submitted, the preview reflects what the actual pullquotes will look like
  5. lliss: I changed a few of your function names and removed pullquote.inc from your zip above trying to simplify things. Let me know what you think.

Thanks, guys. This is looking real good.

Juan C’s picture

Awesome. Thanks for the patch.

csc4’s picture

Seems very useful - was wondering if/when this was going to be committed?

P.S. Thinking about general cases - wondering about including the font family in the other style elements? Is there any way fonts could be left out or selected separately from other styling?

lliss’s picture

Status: Needs review » Needs work

We've got this all working in 7.x. Backport is needed.

lliss’s picture

Issue summary: View changes

Fixed in D7. Closing since D6 has reached its end-of-life.

lliss’s picture

Status: Needs work » Closed (fixed)