Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Let's start improving our current selection of core-themes by simply removing some! We can begin with removing themes that don't use the PHPTemplate engine: Chameleon and Marvin.
- 'Everybody' uses PHPtemplate themes anyway, right?
- Taste aside, I think we can agree on the themes looking a bit tired. They are old and bland, let's just move on.
- Offering a theme that doesn't work without installing another theme engine first doesn't sound core-worthy anyway
Comment | File | Size | Author |
---|---|---|---|
#37 | drupal-remove-crufty-themes-315533-37.patch | 2.02 KB | JohnAlbin |
#31 | drupal-remove-crufty-themes-315533-31.patch | 968 bytes | JohnAlbin |
#30 | drupal-remove-crufty-themes-315533-30.patch | 1.8 KB | JohnAlbin |
#27 | drupal-remove-crufty-themes-315533-27.patch | 56.82 KB | JohnAlbin |
#24 | drupal-remove-crufty-themes-315533.patch | 70.33 KB | JohnAlbin |
Comments
Comment #1
MichelleTo be clear, you don't need another theme engine to use Chameleon and Marvin. But, other than that nit, I agree 100%. Hanging out in support as much as I do, I see tons of confused newbies who don't realize that the prevalant PHPTemplate snippets don't work on Chameleon. Keeping core to just PHPTemplate themes would help that a lot.
Michelle
Comment #2
yoroy CreditAttribution: yoroy commentedAh, sorry about that wild assumption without checking. But yeah, avoiding that confusion is a real win here.
Comment #3
XanoI vote for removing all templates but Garland and Minelli from core. They haven't been improved since Drupal 4 AFAIK (some if not all still use tables for layout purposes) and they look like themes from before the year 2000. Keeping them in core only makes core larger than necessary.
Comment #4
dvessel CreditAttribution: dvessel commentedI'm with Xano here. Replace them all. Pushbutton is phptemplate based but it's using tables and Bluemarine is just dated. Add in a few starter themes that are easy to work with and we should be fine.
Comment #5
webkenny CreditAttribution: webkenny commented+1 on this.
Comment #6
yoroy CreditAttribution: yoroy commentedyeah, I'd agree, all themes except Garland/Minelli are outdated and are doing us a disservice now. Just wanted to start small and get some stuff removed before we even have alternatives ready.
Comment #7
geerlingguy CreditAttribution: geerlingguy commentedAgreed - +1!
Comment #8
catchHow did I miss this one.
Comment #9
webchickSo. Who's offering to maintain these themes in contrib for the people who are still using them in production?
Comment #10
yoroy CreditAttribution: yoroy commentedHaha. As if…
But seriously, who is maintaining PHP3 at the moment for the people who are still using it in production?
Will you really require these to be kept alive?
Comment #11
catchWe don't provide backwards compatibility for code, only for user data. If we're removing core modules, then in 90% of cases we need to provide an upgrade path for user data, however in the case of throttle module where it had been superceded by better options (block caching etc.), and where having it in core was worse than not (all those extra checkboxes all over the place on already confusing screens) - we just took it out. I think these themes fall into that latter category.
If there really are sites out there using them in production, then those which are using hacked up versions of these themes have work to do either way - whether re-applying changes to the 'base' theme, or following the upgrade instructions for their 'custom' theme. The number of sites actually running one of these themes in any recognisable state that I've seen in the past two years I can count on one finger, and those have plenty of better alternatives to choose from in contrib.
And they aren't maintained in core either:
http://cvs.drupal.org/viewvc.py/drupal/drupal/themes/bluemarine/
http://cvs.drupal.org/viewvc.py/drupal/drupal/themes/pushbutton/
http://cvs.drupal.org/viewvc.py/drupal/drupal/themes/chameleon/
Comment #12
mr.baileysChameleon has moved to contrib and John Albin has volunteered to maintain it (see #374650: Chameleon broken by hook_page_alter).
http://drupal.org/project/Chameleon
I guess this means Chameleon can now be removed from core?
Comment #13
catchPatch (minus the binaries).
Comment #16
kika CreditAttribution: kika commented[cry] Nooooo, my loved baby! [/cry]
Ok, although she had some attraction 5 years ago, she's now adult and ready live in a countryside with her equally ugly husband Marvin.
Somebody said "taken to the pasture"?
Comment #17
JohnAlbinOk, so I own http://drupal.org/project/Chameleon
I created a request in the bluemarine-in-contrib issue queue to get access to it: #397274: Project maintainership
But, the Pushbutton contrib in CVS doesn't have a project page. And from the $Id$ tags it looks like Dries owns it. So I started #397478: Change ownership of contributions/themes/pushbutton to get that sorted.
@catch: it's strange that removing the content's of those files cause the Drupal installer to stop working. Odd.
Comment #18
JohnAlbinI now own http://drupal.org/project/bluemarine. I'll update the code in CVS shortly.
Comment #19
jrabeemer CreditAttribution: jrabeemer commentedWhat about the Stark theme? I think the title should reflect that.
Comment #20
kika CreditAttribution: kika commentedComment #21
tonyn CreditAttribution: tonyn commentedI think Garland may be referred to as a "Classic" Drupal theme to keep in there.
Good job guys.
Comment #22
alexanderpas CreditAttribution: alexanderpas commentedsad to see bluemarine removed from core.... it was a great, simple (starter) theme, very drupalish (blue), which was just outdated and needed updating to current best practices.
we need a new starter theme like bluemarine in core (another issue...)
Comment #23
XanoI don't see what was so great about Bluemarine. Garland does a far better job. Besides, we can linger in the past and decide not to remove those old themes from core, or we can remove them to prevent core from looking old and foolish for newcomers.
Comment #24
JohnAlbinI now own:
http://drupal.org/project/bluemarine
http://drupal.org/project/chameleon
http://drupal.org/project/pushbutton
I'm just waiting for the packaging scripts to create the tarballs for the -dev releases I created. :-)
Patch attached. I'm betting the testbot will fail to apply the patch because it removes image files. I'll re-roll without them if it does.
Comment #25
JohnAlbinComment #27
JohnAlbinYep. Re-rolled patch without image references. Those will have to manually deleted.
Comment #29
catchWe have contrib projects and a reliable maintainer. If the patch doesn't actually work, cvs remove will do.
Comment #30
JohnAlbinHere's a patch which just updates the CHANGELOG.txt. (catch asked for this in IRC just now.)
Comment #31
JohnAlbinDoh! Here's the correct patch.
As we add new themes to core, we can add them into this same line in the CHANGELOG.
Comment #32
kika CreditAttribution: kika commented"Replaced the bluemarine, chameleon and pushbutton themes with Stark" -- i do not see the logic ("with Stark" part).
Comment #33
geerlingguy CreditAttribution: geerlingguy commented@ kika - We don't mean that stark is replacing those three themes... for now, we're removing old cruft, with the plan of getting a few new themes into core. And hopefully a new default theme (that remains to be seen, though!).
Stark is an excellent theme for a person wanting to develop a new theme; it's terrible for half the people that download Drupal to quickly set up a new website though. That's why it will be installed with core, but not enabled. We hope there is a new killer theme for the default core theme.
Comment #34
alexanderpas CreditAttribution: alexanderpas commentedever tried learning theming with garland???
It's a lot harder because of the "unneeded files" (e.g. color module support) while bluemarine is just 5 .tpl.php files, 3 images (1 logo, 2 screenshots), 1 CSS file (+1 CSS file for RTL support ), LICENSE and README, and the .info file, while still supporting the full drupal toolkit.
this made it perfect for a new themer to get used to drupal, in an easy way,
also, since it had clear CSS rules, it was easy to understand.
but the best thing about bluemarine was, it had that nice, warm drupal feeling, unlike charmeleon and pushbutton.
i'm not cracking down on Garland (unlike charmeleon and pushbutton) here, each theme has it's own strengths and weaknesses
Comment #35
geerlingguy CreditAttribution: geerlingguy commented@ Alexander Pas - the problem, though, is that even having 5 .tpl.php files can be quite daunting for a first-time themer. The goal is to have more CSS-based themes, and have the whole theming idea be very designer-friendly, and lean on the PHP. Personally, I would hope we can make the core theme simply be a CSS file, maybe a few images in an 'images' folder, and a .info file. Especially since that would show off how amazingly designer-friendly the core output is.
It would also be nice to have a good README.txt file in the core default theme, explaining how people can easily extend and adapt the core theme to do whatever they'd like to do, including copying in a .tpl.php file or two.
Comment #36
JohnAlbinThere are some tests that check switching of themes that need to be updated as well.
Comment #37
JohnAlbinAdded updated block.test tests. And updated CHANGELOG wording.
Comment #38
catchLooks good to me, waiting for test bot.
Comment #39
catchComment #40
alexanderpas CreditAttribution: alexanderpas commentedFirst of all, I completely agree with you. bluemarine shoudn't be a starting point for theme developpers (neither should garland), but a simple CSS+images only stylesheet should be, however, when wanting to expand the theme, the transfer from CSS+images to e.g. Garland is still way to big, bluemarine would have filled thig gap perfectly.
remember that some people like to learn from example, while others learn by reading ;)
I was thinking of a learning curve like this (included stark as the "no theme")
stark >> (new CSS only theme bluesky?) >> bluemarine >> garland
Comment #41
yoroy CreditAttribution: yoroy commentedGuys, have a look at http://groups.drupal.org/node/16200 for the bigger picture on new core themes. The issue queue is not the best place to discuss "big ideas" and you'll find a lot more info, audience and context over there. Thanks!
Comment #42
JohnAlbinActually, a more updated plan is on the Community Initiatives page under Design & Theming:
http://drupal.org/community-initiatives/drupal-core/theming
And, if I can plug my DCDC session video… :-) http://dc2009.drupalcon.org/session/how-kill-blue-marine-new-core-themes...
Comment #43
Dries CreditAttribution: Dries commentedI support removing these themes, but I'll let webchick do the commit.
I think we should move these themes to contrib so people can continue to maintain and use them.
The next step could be remove support for different template engines and standardize on PHPtemplate? Just thinking up loud.
Comment #44
Michelle@Dries: JohnAlbin is already taking care of them in contrib. As much as I love PHPTemplate, there are some people firmly in the smarty camp and I think you'd have an outcry unless it could somehow be made to work from contrib.
Michelle
Comment #45
jrabeemer CreditAttribution: jrabeemer commentedWe could use a new issue for removing other template engines in Drupal. Let the smarty folks duke it out there.
Comment #46
webchick*drumroll*
I now present to you: Drupal 7, 75% less ugly than before!! Hoorayyy! :)
Great job, folks!!!
Comment #47
webchickIncidentally, for posterity, here was my case for this patch. Thanks, Dries. :)
Comment #48
MichelleGo Webchick!
Michelle
Comment #49
yoroy CreditAttribution: yoroy commentedGood news, glad we could do this without requiring new themes to replace these. Less is more!
Comment #50
rickvug CreditAttribution: rickvug commentedMy happy dance ensues! Webchick, I was going to track you down in DC to make my case for this. Looks like you were one step ahead. Thanks :)
Comment #52
dwwFYI: This (minorly) "broke" update.module: #490562: Hard-coded list of core themes for update module is now stale (and yes, I know, #253501: Tests for update.module would be nice). ;) Just something to keep in mind when folks are adding/removing core themes. Thanks.