User login

Incremental improvements make Drupal's code awesome

A fix was recently committed to both Drupal 8 and Drupal 7 to use #attached for block_admin_display_form() instead of drupal_add_css(). It only came up in my "Your issues" block on my drupal.org dashboard because, it turned out, i was the one who had initially complained about it. And that's all i did- looking for core examples to tell readers of dgd7.org how to write good code, i noticed core was in fact doing it wrong. And whined about it, but at least in the proper forum, Drupal core's issue queue.

Drupal users barbi, DjebbZ, and ultimately 10oclock stepped up to suggest the code and make the patch, and lambic reviewed it.

And now people using Drupal 7.9 who search for how to attach CSS will see the best practice way. Awesome.

As i'm writing this, in #drupal...

[16:11:40] Any novices around? Got a new one for ya: http://drupal.org/node/1297410
[16:11:41] http://drupal.org/node/1297410 => Drop remnants of #node_edit_form => Drupal core, node system, normal, active, 0 comments, 2 IRC mentions

I am so excited to have Catch a maintainer of Drupal 8 and really excited about Drupal's potential to both make the leaps and the incremental steps to continue being awesome.

Comments

Post new comment

The content of this field is kept private and will not be shown publicly.
  • You may post code using <code>...</code> (generic) or <?php ... ?> (highlighted PHP) tags.
  • You can use Markdown syntax to format and style the text. Also see Markdown Extra for tables, footnotes, and more.
  • Web page addresses and e-mail addresses turn into links automatically.
  • Allowed HTML tags: <a> <em> <strong> <cite> <code> <ul> <ol> <li> <dl> <dt> <dd> <img> <blockquote> <small> <h2> <h3> <h4> <h5> <h6> <sub> <sup> <p> <br> <strike> <table> <tr> <td> <thead> <th> <tbody> <tt> <output>
  • Lines and paragraphs break automatically.

More information about formatting options

By submitting this form, you accept the Mollom privacy policy.