[drupal-devel] [task] improve theme system

adrian drupal-devel at drupal.org
Thu Apr 28 18:38:24 UTC 2005


Issue status update for http://drupal.org/node/21517

 Project:      Drupal
 Version:      cvs
 Component:    theme system
 Category:     tasks
 Priority:     normal
 Assigned to:  stefan nagtegaal
 Reported by:  stefan nagtegaal
 Updated by:   adrian
 Status:       patch

I completely +1 this.




adrian



Previous comments:
------------------------------------------------------------------------

April 28, 2005 - 18:21 : stefan nagtegaal

The theme system is very inconsistent and not clean. In a steady stream
of patches i'll try to make the theme system clean and let most of the
theme functions behave the same..


One of the main goals is, that when writing themes in PHPTemplate for
example, the way how things are being returned is different from theme
function to theme function..


Let's try to unify this, so theming will be more efficient, simpler and
more consistent..




------------------------------------------------------------------------

April 28, 2005 - 18:40 : stefan nagtegaal

Attachment: http://drupal.org/files/issues/cleaner_themable_help.patch (1.99 KB)

The attached patch does:
- remove unused function theme_help() and theme_error();
- added a wrapper function drupal_get_help() inside common.inc (for
consistency with drupal_get_title(), drupal_get_breadcrumb(), etc);
- reimplemented theme_help(), only this time for a more consistent way
for themers;


This patch makes it easier to theme the page specific helptexts for
drupal.


To change your PHP-based themes, you can do:

<?php
-  if ($help = menu_get_active_help()) {
-    $output .= '<small>'. $help .'</small><hr />';
-  }
+  $output .= theme('help', drupal_get_help());
?>




To change your PHP-Template based theme:

<?php
 if ($help): 
?>


<?php
 print $help; 
?>


<?php
 endif; 
?>




becomes a simple:

<?php
 print $help; 
?>




If this patch is accepted i'll try to make the functions for returning
the local tasks and the status messages behave the same..


Overriding the <div class="help">...</div> could be done through
deifinig your own theme_help() function in your theme or template.php
file.




------------------------------------------------------------------------

April 28, 2005 - 18:44 : dikini

+1 from me
it makes help text themeable
it uses semantic markup, which is good practice, or as some say the
Right Way




------------------------------------------------------------------------

April 28, 2005 - 19:00 : killes at www.drop.org

Attachment: http://drupal.org/files/issues/cleaner_themable_help_0.patch (1.99 KB)

The patch had DOS line endings and wouldn't apply.


I fully support this patch. +1







More information about the drupal-devel mailing list