1. BonusCSS is a solution for multi-site Drupal 7 installations to allow custom CSS for sites on a per-site basis that all share the same theme. The maintainers run 60+ Drupal sites on a single multi-site installation, many of which have the same core theme. Previous standard practice has been to subtheme when we needed to make a tweak, but sub-theming necessitates resetting all of the theme settings, creating the theme with a .info file, and other work that is simply unnecessary. BonusCSS will be primarily supported for Drupal 7+. For Drupal 6, an alternative to this module is Custom CSS & JS. Rather than port the linked module to Drupal 7, the original maintainer decided something more lightweight was in order. Additionally, he felt that placing the CSS in the site's "files" directory posed a potential security risk. If you need to control which pages your CSS is loaded on, look at CSS Injector. BonusCSS uses standard Drupal caching for the CSS, so performance should be better, but does not have the flexibility of CSS Injector.
  2. Screenshots not necessary.
  3. BonusCSS Sandbox - http://drupal.org/sandbox/matir/1169500
  4. username@git.drupal.org:sandbox/matir/1169500.git
  5. Drupal 7 initially, although the plan is to backport the module to Drupal 6 since the change is not extensive due to the very small size of the module.
  6. This section is not available.

Comments

willietse’s picture

willietse commented

An initial automated review, there are some problems that need to be modified.

http://ventral.org/pareview/httpgitdrupalorgsandboxmatir1169500git

luxpaparazzi’s picture

luxpaparazzi commented

I recommand adding the following as GIT-Link: http://git.drupal.org/sandbox/matir/1169500.git

I find it very difficult to read the introduction-text and would prefere something more clear ... but that's only a personal opinion, you may also check http://drupal.org/node/997024 ...

as this module only contains 1 function of 16 lines, I don't see how it could get full project status...

luxpaparazzi’s picture

luxpaparazzi commented
Status: Needs review » Needs work
patrickd’s picture

patrickd commented
Issue tags: +PAreview: single application approval

Don't know whether this implementation really makes sense, it's not really best practice approach IMHO.

Agreeing on module size:

This project is too short to approve you as git vetted user. We are currently discussing how much code we need, but everything with less than 120 lines of code or less than 5 functions cannot be seriously reviewed. However, we can promote this single project manually to a full project.

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.