- From: Richard Ishida <ishida@w3.org>
- Date: Wed, 18 Feb 2015 12:49:15 +0000
- To: Gunnar Bittersmann <gunnar@bittersmann.de>
- CC: www-i18n-comments@w3.org
On 18/02/2015 09:18, Gunnar Bittersmann wrote: >> Source file attached. > > Hi Richard! > Whenever I look into code there is danger. ;-) While dealing with the > mark-up again, I have noticed a few things. Most important first: > > » > $status = 'review'; > « > Status should be published (or notreviewed?) now: > $status = 'published'; fixed > » > takes NMTOKEN values in the HTML schema > « > No HTML here, but: > takes NMTOKEN values in the XML schema fixed > > The rest is not of high priority, but once we are at it: > > » > <div class="example"> <code translate="no"><p>Here is a list of > part numbers: > « > Tha natural text in this sample code might (should?) be translated. > Remove translate attribute: > <div class="example"> <code><p>Here is a list of part numbers: fixed > » > for the language attribute<code translate="no"></code> if > « > Wrong placed code tags. Remove: > for the language attribute if fixed > » > should use <span class="kw">und</span> if there > « > Better: > should use <code translate="no">xml:lang="<span > class="kw">und</span>"</code> if there fixed > » > as a value of <code>xml:lang</code>. > « > Use kw as elsewhere: > as a value of <span class="kw">xml:lang</span>. fixed > » > you can't use the empty string in XHTML. > « > More accurate: > you can't use the empty string in XHTML 1.x. fixed > » > declare xml:lang as CDATA > « > Wrap in span and use kw as elsewhere: > declare <span class="kw">xml:lang</span> as CDATA fixed > » > <ul id="full-links2"> > « > The list items herein are not styled the same as in <ul id="full-links"> > (as should be). Ways out: > > Add a class <ul id="full-links" class="full-links">, <ul > id="full-links2" class="full-links"> etc. and use it for styling. But > the rulesets with the ID selectors cannot be removed from the stylesheet > because other articles would still need them. > > Or add #full-links2 to the selectors in the stylesheet. (Not > recommended, it does not scale. What about potentional <ul > id="full-links3">?) > > Or use attribute selector [id^="full-links"] instead of ID selector in > the stylesheet. This would work best, I think. We do not care about > ancient IEs here, do we? I made it all one single list, as it should have been. thanks for that! ri
Received on Wednesday, 18 February 2015 12:49:24 UTC