is not used any more and exists only for history. Post new tickets on the Github account. n'est plus utilisée, et existe uniquement pour son historique. Postez les nouveaux tickets sur le compte github.

Opened 13 years ago

Closed 13 years ago

#800 closed bug (fixed)

breadcrumb not valid "jelix"/w3c

Reported by: foxmask Owned by:
Priority: low Milestone: Jelix 1.1.2
Component: jelix:plugins:tpl Version: 1.1RC1
Severity: trivial Keywords:
Cc: Blocked By:
Blocking: Documentation needed: no
Hosting Provider: Php version:


in the plugin breadcrumb we read :

echo '<li value="'.($i+1).'"'.($i==$nb?' class="first"':($i==$leng-1?' class="end"':'')).'>';

which is not valid W3C .

echo '<li'.($i==$nb?' class="first"':($i==$leng-1?' class="end"':'')).'>';

would do the trick.

Attachments (1)

800-breadcrumb.diff (1.2 KB) - added by Surfoo 13 years ago.

Download all attachments as: .zip

Change History (9)

comment:1 Changed 13 years ago by laurentj

  • Priority changed from normal to low
  • Severity changed from normal to trivial

This attribute is valid in XHTML 1.1 transitionnal and HTML 4.01 transitionnal...

comment:2 Changed 13 years ago by foxmask

  • Summary changed from breadcrumb not valid w3c to breadcrumb not valid "jelix"/w3c

The fact is, if Jelix use a default doctype like "XHTMl Strict", the rest of the code/plugin should be compatible with this choice.

this is why i raised this little issue ;)

comment:3 Changed 13 years ago by laurentj

  • Milestone set to Jelix 1.1.2

comment:4 Changed 13 years ago by bballizlife

Indeed in xhtml 1.0 strict (the default doctype in Jelix) the "value" attribute no longer exists. So what to do here ? I suggest deleting the use of this attribute in the plugin so as we could be compliant with the default doctype of Jelix. Moreover i think the lost of information induced by this deletion is not that point important.

comment:5 Changed 13 years ago by laurentj

I agree to remove this attribute.

Changed 13 years ago by Surfoo

comment:6 Changed 13 years ago by Surfoo

  • review set to review?

comment:7 Changed 13 years ago by laurentj

  • Component changed from jelix:plugins to jelix:plugins:tpl
  • review changed from review? to review+


comment:8 Changed 13 years ago by Surfoo

  • Resolution set to fixed
  • Status changed from new to closed

commited in trunk and branch 1.1.x in r1349

Note: See TracTickets for help on using tickets.