Drupal
/
drupal
mirror of https://git.drupalcode.org/project/drupal.git
1
0
Fork 0
Code Issues Projects Releases Wiki Activity

- Patch #555830 by effulgentsia: better code comments.

merge-requests/26/head
Dries Buytaert 2010-04-30 20:07:03 +00:00
parent c1bb8d1ecd
commit ca1fc09e1f
2 changed files with 22 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
includes/common.inc
View File

@ -5390,6 +5390,18 @@ function drupal_common_theme() {
'variables' => array('links' => NULL, 'attributes' => array('class' => array('links')), 'heading' => array()), 'variables' => array('links' => NULL, 'attributes' => array('class' => array('links')), 'heading' => array()),
), ),
'image' => array( 'image' => array(
// HTML 4 and XHTML 1.0 always require an alt attribute. The HTML 5 draft
// allows the alt attribute to be omitted in some cases. Therefore,
// default the alt attribute to an empty string, but allow code calling
// theme('image') to pass explicit NULL for it to be omitted. Usually,
// neither omission nor an empty string satisfies accessibility
// requirements, so it is strongly encouraged for code calling
// theme('image') to pass a meaningful value for the alt variable.
// - http://www.w3.org/TR/REC-html40/struct/objects.html#h-13.8
// - http://www.w3.org/TR/xhtml1/dtds.html
// - http://dev.w3.org/html5/spec/Overview.html#alt
// The title attribute is optional in all cases, so it is omitted by
// default.
'variables' => array('path' => NULL, 'alt' => '', 'title' => NULL, 'attributes' => array(), 'getsize' => TRUE), 'variables' => array('path' => NULL, 'alt' => '', 'title' => NULL, 'attributes' => array(), 'getsize' => TRUE),
), ),
'breadcrumb' => array( 'breadcrumb' => array(

11
includes/theme.inc
View File

@ -1470,7 +1470,16 @@ function theme_links($variables) {
* An associative array containing: * An associative array containing:
* - path: Either the path of the image file (relative to base_path()) or a * - path: Either the path of the image file (relative to base_path()) or a
* full URL. * full URL.
* - alt: The alternative text for text-based browsers. * - alt: The alternative text for text-based browsers. HTML 4 and XHTML 1.0
* always require an alt attribute. The HTML 5 draft allows the alt
* attribute to be omitted in some cases. Therefore, this variable defaults
* to an empty string, but can be set to NULL for the attribute to be
* omitted. Usually, neither omission nor an empty string satisfies
* accessibility requirements, so it is strongly encouraged for code calling
* theme('image') to pass a meaningful value for this variable.
* - http://www.w3.org/TR/REC-html40/struct/objects.html#h-13.8
* - http://www.w3.org/TR/xhtml1/dtds.html
* - http://dev.w3.org/html5/spec/Overview.html#alt
* - title: The title text is displayed when the image is hovered in some * - title: The title text is displayed when the image is hovered in some
* popular browsers. * popular browsers.
* - attributes: Associative array of attributes to be placed in the img tag. * - attributes: Associative array of attributes to be placed in the img tag.