Hi,

Following the discussion on dev list, I put up my table of the main hooks in the "reference" section of the developer's handbook, at http://drupal.org/node/112180 . Could someone:

  • assign it a "full html" type so the table can be displayed properly
  • publish it ?
  • maybe move it elswhere more appropriate

Thanks in advance.

Comments

pwolanin’s picture

pwolanin commented

I switched it to full HTML, but there seems to be some problems:

the table is too wide for the page.

the markup in the table didn't seem to be valid (I did a little fixing - there were several tbody elements, tbody elements within the thead element, and no tfoot element), and for some reason (maybe the theme) no borders display.

fgm’s picture

fgm
Primary language French
Location Paris, France
commented

Hi,

Thanks for looking into it. I'd gladly improve it, but I don't have permission to edit it, maybe because of the filter access permission ?

The no-border issue is normal, BTW: it's a consequence of the rules=groups clause used to group rows together, as per the XHTML 1.0 DTD (http://www.w3.org/TR/2000/REC-xhtml1-20000126/DTD/xhtml1-strict.dtd, see Use multiple tbody sections when rules are needed between groups of table rows. With these rules, rows only appear between tbody sections.

fgm’s picture

fgm
Primary language French
Location Paris, France
commented
Status: Active » Needs review

Fixed. Now, that the layour is done, all that's left is for some core hackers to check the content accuracy...

FWIW, the page displays better with CSS disabled.

fgm’s picture

fgm
Primary language French
Location Paris, France
commented

Looks like Full HTML has been removed again...

webernet’s picture

webernet commented

Killes added thead, tfoot, colspan, and other table tags that the page was using to the documentation filter, but the layout still breaks for some unknown reason. Switched back to Full HTML until the source of the problem can be found.

fgm’s picture

fgm
Primary language French
Location Paris, France
commented
Status: Needs review » Reviewed & tested by the community

It doesn't break on a fresh D6 install with the same filters as d.o. in the same order, so I think the page is good enough as it is.

The colgroup should have span="7" instead of span="8", though.

Regarding the width of the table, wrapping it in a <div style="font-size: 0.8em"> could help, but the most useful for most users would probably be to display the table full-page, without borders

webernet’s picture

webernet commented
Status: Reviewed & tested by the community » Fixed

Closing...

Anonymous’s picture

(not verified) commented
Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.