We have moved to Github. Please open tickets there.

Opened 4 years ago

Closed 4 years ago

#792 closed Defect (invalid)

Elgg core produces invalid XHTML

Reported by: aszepeshazi Owned by:
Priority: high Milestone:
Component: Core Version: 1.2
Severity: major Keywords: w3c, standards compliant, web standards
Cc: brettp Difficulty:

Description

Checked a couple of Elgg pages in the W3C validator http://validator.w3.org/), with the following results:

http://elgg.skawa.hu/ (my own Elgg testsite) - 18 errors
http://elgg.skawa.hu/pg/file/use[svn:20]/read/506/raymond-cvikota - 25 Errors, 18 warnings

http://community.elgg.org/ - 35 errors, 1 warning
http://community.elgg.org/pg/groups/forum/15815/ - 194 Errors, 190 warnings
http://community.elgg.org/mod/plugins/world.php - 40 Errors, 6 warning(s)
http://community.elgg.org/mod/groups/discussions.php -165 Errors, 82 warning(s)

I realize that a number of the errors come from plugins (like the groups plugin), but the core has it's share of invalid markup as well. As I see, every single page rendered by Elgg breaks XHTML validity.

A couple of these XHTML errors are quite serious:

  • duplicated ids
  • mismatched <td> and <tr> tags
  • unencoded links having special characters (i.e. ampersand)
  • misplaced <div>, <h3>, etc tags

Sorry that I'm unable to give very specific report on where and what tags are broken, but as of the moment this seems a system-wide issue for me. My best bet would be to go through all pages in a validator, one by one.

If you're unable to address this in the near future, I will have some spare time next month, and will be happy to assist with it.

Change History (4)

comment:1 Changed 4 years ago by marcus

  • Priority changed from major to minor

comment:2 Changed 4 years ago by marcus

  • Priority changed from minor to trivial

comment:3 Changed 4 years ago by lynnpauls

  • Keywords w3c, standards compliant, web standards added
  • Priority changed from trivial to major

Why has the priority of this been downgraded from major to minor and minor to trivial. Some of these errors are not trivial at all (especially the repeat use of id's obviously). How can I help to contribute and correct them

comment:4 Changed 4 years ago by cash

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

This ticket is too general. We need specific tickets that point out the problems. For example, where in the code are there mismatched <td> and <tr> tags? I recommend creating tickets that all have "invalid XHTML" in the title that make it easier to track these items.

Note: See TracTickets for help on using tickets.