Bug 382770 - Eclipse.org nova theme doesn't pass the W3C Markup Validation Service
Summary: Eclipse.org nova theme doesn't pass the W3C Markup Validation Service
Status: RESOLVED FIXED
Alias: None
Product: Community
Classification: Eclipse Foundation
Component: Website (show other bugs)
Version: unspecified   Edit
Hardware: PC Windows 7
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: phoenix.ui CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-06-15 15:49 EDT by Christopher Guindon CLA
Modified: 2013-03-15 13:00 EDT (History)
2 users (show)

See Also:


Attachments
Fix w3c validation errors (4.67 KB, patch)
2012-06-18 11:14 EDT, Christopher Guindon CLA
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Christopher Guindon CLA 2012-06-15 15:49:40 EDT
Build Identifier: 

For example if we try to validate this page http://www.eclipse.org/membership/
we get 7 errors and 1 warning.

The 7 errors can easily be fix by replacing the <div> element with <span>.
We might need to update the css here.

The final warning is with the google search.

Line 57, Column 132: reference not terminated by REFC delimiter
…/brand?form=searchbox_017941334893793413703%3Asqfrdtd112s&lang=en"></script>	

I would need to test but replacing the &lang= with &amp;lang= should remove the error.

I will create a patch early next week for this.



Reproducible: Always
Comment 1 Christopher Guindon CLA 2012-06-18 11:14:39 EDT
Created attachment 217500 [details]
Fix w3c validation errors

After applying this patch the membership page will validate. 

Changes:

-Visit other eclipse site : Pop-up text is wrapped around a span instead of a div
-update css rules to check for span
-google search is now using &amp; instead of only &
Comment 2 Nathan Gervais CLA 2012-06-18 11:46:45 EDT
Applied this patch locally and all seems good 

+1
Comment 3 Denis Roy CLA 2012-06-21 14:51:25 EDT
+1
Comment 4 Christopher Guindon CLA 2013-03-15 13:00:43 EDT
Better late than never! This patch was committed today :) 

http://git.eclipse.org/c/www.eclipse.org/eclipse.org-common.git/commit/?id=3d2df796274f98f870797facae98d7e67ac8b733


The membership page was successfully checked as XHTML + RDFa:
http://validator.w3.org/check?uri=http%3A%2F%2Fwww.eclipse.org%2Fmembership%2F