I kinda like this, not only is a lot more clear than just a simple image, but it also adapts to themes a lot easier

+1, but only if u fix the spelling error from "Powerd" to "Powered" :P

and maybe consider running it trough a theme function anyway, just to make it easy to override so this block doesn't remain to be the most unused block that ships with core

How about this?

this adds an option to the block configuration that's just a checkbox that reads "Use textual link" (this needs some thought though, coz my current implementation sorta sucks and is prolly confusing as hell)
by default it will set this checkbox enabled, thus showing a textual link instead of the badge, but I retained the option to switch to the old badge and have all the options one had before

in addition, the textual version can be overriden by overriding theme_system_powered_by

@TODO: figure out a way to make it clear that when the "use textual link" checkbox is checked, the user understands that the badge won't be used (some book.module style ahah could help us hide that part of the form untill the checkbox is unchecked)

pft fine, attached patch gets rid of all the images, all the settings in the block, but retains the theme function for overridability

not sure how these binary changes work, but I seemed to have managed to get it included in the patch

version without binaries (this one won't attempt to delete the PNG files)

attached patch also removes the block save handler and changes the tests to reflect the changes

Big +1. Let's remove the visual cruft branding of D5-D6, embrace the minimalistic style and develop proper badges/branding solutions over "Evangelize/Share Drupal" in drupal.org.

Kill all the files and code for images.

Makes sense to me. PHP code is not the place to deal with UI images anyway.

      $image_path = 'misc/' . variable_get('drupal_badge_color', 'powered-blue') . '-' . variable_get('drupal_badge_size', '80x15') . '.png';

that seemed kinda useless in system_block_view(); now

+1 for removing the image and replacing with text.
-1 for 'Powerd'.

:)

Yeah, blue icon means it gets disabled if it clashes with the theme you want to use.

Plus “scary white eye balls staring at you through a narrow gap” doesn't really convey user-friendly software; its more of a “I'm stalking you and peeping through your mail slot.” ;-)

RTBC+1

Yes, please remove the graphic version. All those versions are silly (ever saw another one but the default?) and the text version is nicely understated and will look better across themes and more likely to survive.

what johnalbin said ;)

+1 for everytime we remove an image from core and replace it with text :)

Whoops, Bojhan set me straight, there is no 'Powerd' in the patch.

+1!

just occurred to me I didn't wrap this in proper <p></p> leaving it a regular textnode

or does any1 feel this should be a span or something?

seutje I don't think p is appropriate (semantically). I think it's fine just leaving it an anchor with a class.

or a span wrapping the whole thing makes sense I guess

okay, changed <p> into <span>

OK, committed to CVS HEAD!

You are acting on the assumption that there will be a discussion about this, and on forehand choosing a concession to that discussion? I do not think an option is appropriate here, this patch chooses a direction - if you do not agree with that direction, then fine. But we are not putting a "We can't decide" option in on forehand. Text is better, for all the mentioned above reasons - there is no reason to keep an image.

Quotes like this are sad. Our community deserve more than shutting down discussion.

As for myself, I prefered the badged versions than the plaintext version. However, now that this landed, we should also remove the Powered by Drupal block altogether.

It does not make sense to have a text only block in core anymore. Any themer know how to add a few words of text anywhere in their theme. :-)

the icons are still in the repository btw

I think the icons should stay in the repository - they were apparently there before the Powered by Drupal block ever existed, and people might find them useful? See #183332: Add an optional 'Powered by Drupal' block for the history...

However, the patch that was committed seems to need work:

1. There is code left behind in system.js related to the image switching - shouldn't that be removed?
2. Aren't we now leaving a couple stale variables (drupal_badge_color and drupal_badge_size) around in the database which should be deleted?
[3, sort of.] From history of the previous issue: The alt-text "Powered by Drupal, an open source content management system" was specifically chosen for SEO-friendliness, and now we are removing any references to "open source CMS".... I could personally care less about SEO, but bringing it up as a possible issue because some people do care :)

While I think there are some good things about this change, I also am not sure it was discussed enough. The comments above seems focused on designers, themers, etc, but that is something only high end Drupal sites can afford to worry about. Perhaps someone could comment on why (or if) this change is good for a small personal site? Having a cute little Drupal icon that they can choose to display and have some control over how it appears - to me that is similar to the Garland color wheel. It is not something a serious themer will really use, but it allows a regular person to start with stock Drupal and customize it a bit. Plenty of Drupal sites display this image now and I wonder if they lose something as a result of this change.

I think it might be nice to trust the UX-Team a bit more on these type of discussions, we know what we are doing - and we are trying to put UX and general design principles over purely opinions.

@Bojhan: I respect a lot of the work being done by the UX theme in general. However it does not mean that you know everything.

You are the ones making opinions here. A few themer discussion does not make a fact.

Also I am a UX designer, and I know when some changes are really usability related.

Please refrain saying that you know what you are doing.

@xmacinfo @bojhan, this is not going to be productive, so just stop now. Everyone here is just trying to make Drupal a better product. We may have different opinions about what that actually looks like, and a lot of those opinions are based on years of experience and yet may still be diametrically opposed.

@xmacinfo, Please don't just copy and paste snippets of postings and flame about them. I'm not going to give my opinion on whether its justified or not, because that is not the point. The fact of the matter is that it is just plain unproductive.

If you want to create a new issue to have the block completely removed, please do. You'd probably have my support, but doubtful you'd have support from the people that matter (Dries/webchick/etc). This issue appears to have support, and I think is a much better solution as you can still use CSS to pop your image in there if you want to, but you can't (easily) replace a hard coded image with text. (Come to think of it, we could have the image replacement css in there by default, and then it would be very easy for themes to just override that css).

Thanks! :)

See also: #661806: 'Powered by Drupal' block should link to drupal.com, not drupal.org

Would anyone care to make a patch to remove the last stuff mentioned in #27?

This patch removes the leftover variables and JavaScript code.

Looks great from my point of view.

Committed to 7.x and 8.x. Thanks David.

Untagging since it's been committed to 7.x