Closed (fixed)
Project:
Drupal core
Component:
locale.module
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
28 Jul 2005 at 10:13 UTC
Updated:
7 Oct 2005 at 07:20 UTC
Jump to comment: Most recent file
locale.inc:
Uses the deprecated element <strike> - should be changed to <span style=\"text-decoration: line-through;\">
Uses the deprecated attribute nowrap - should be changed to style="white-space: nowrap;"
Also, method="POST" should be changed to method="post".
Applying these changes makes the 'manage strings' results page validate as XHTML 1.0 strict.
| Comment | File | Size | Author |
|---|---|---|---|
| #8 | xhtml_fixes_0.zip | 2.3 KB | wulff |
| #6 | xhtml_fixes.zip | 2.3 KB | wulff |
| #3 | drupal.css_0.patch | 424 bytes | wulff |
| #2 | locale.inc.patch | 3.05 KB | wulff |
| locale.inc.html.patch | 3.08 KB | wulff |
Comments
Comment #1
junyor commentedI'd prefer to see the styling done in drupal.css, rather than inline. +1 for the fix, -1 for the implementation.
Comment #2
wulff commentedYou are absolutely right. This patch fixes locale.inc.
Comment #3
wulff commented...and this one adds the styles to drupal.css.
Comment #4
breyten commentedBetter, but where did the CSS go? ;) Furthermore, I like semantic markup better, ie. <em class="locale-untranslated"> instead of using spans.
Comment #5
Robrecht Jacques commented+1 for the CSS/class solution.
maybe the class "locale-nowrap" could become a more general class "nowrap" as this could be used in more locations, eg in drupal:
"locale-untranslated" is locale.module dependent, so that one is ok.
Comment #6
wulff commentedbreyten: Agreed.
Updated patches with fixes for statistics.module attached.
Comment #7
Robrecht Jacques commentedYou have put 'style' => 'nowrap' instead of 'class' => 'nowrap' on all locations. Small thinko I suppose.
If that's fixed => +1 from me.
Comment #8
wulff commentedA thinko indeed. Fixed fixes attached.
Comment #9
drummI'm not sure if this is the right tag for this, but there is a del tag which might be a better alternative to strike. http://www.w3.org/TR/html401/struct/text.html#h-9.4
You can make patchfiles which include multiple files instead of making a zip file which adds an extra step for reviewers.
cvs diff -uF^f misc/drupal.css includes/locale.inc.
Comment #10
Robrecht Jacques commented"del" is not the right tag here. It is not because the default style for "del" is strike-through that it is "correct" to use it if you replacing something with "style=strikethrough". "del" (and "ins") is for document changes, and only for that. Here the strike-through doesn't mean that.
So a "span", or maybe "em" (you put emphasis on the fact that it is untranslated?) is better.
Comment #11
Steven commentedCommited to HEAD.
Comment #12
(not verified) commentedComment #13
(not verified) commentedComment #14
(not verified) commentedComment #15
(not verified) commentedComment #16
(not verified) commented