Bug 20125

Summary: http://webkit.org/ missing link to key documentation for new developers.
Product: WebKit Reporter: Jack Wootton <jackwootton>
Component: WebKit WebsiteAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: ddkilzer, webkit
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
URL: http://webkit.org/
Attachments:
Description Flags
Text file containing new link to key documentation for new developers.
eric: review-
Patch file containing updated web site documentation page with link to existing Apple documentation. ddkilzer: review+

Description Jack Wootton 2008-07-21 08:51:38 PDT
http://webkit.org/ is missing link to key resource for new WebKit contributors / developers. Here is a link to WebKit documentation by Apple, this should be linked, probably in the Documentation section:

http://developer.apple.com/documentation/Cocoa/Conceptual/DisplayWebContent/DisplayWebContent.html
Comment 1 David Kilzer (:ddkilzer) 2008-11-06 11:34:31 PST
Please create a patch for the change to the web site, upload the patch file and set the "review?" flag on the attachment.

Comment 2 Jack Wootton 2008-11-06 12:15:56 PST
Created attachment 24951 [details]
Text file containing new link to key documentation for new developers.
Comment 3 Eric Seidel (no email) 2008-11-17 13:32:34 PST
Comment on attachment 24951 [details]
Text file containing new link to key documentation for new developers.

I don't think this is the file you meant to attach. This is not a patch.
Comment 4 Jack Wootton 2008-11-18 03:14:19 PST
Prior to this patch file being submitted I was
advised the only way I could request the website be updated was to
raise a bug and attach a patch file.  The conversation acting as a
precursor to this patch file is fully available on the webkit mailing
list.

(In reply to comment #3)
> (From update of attachment 24951 [details] [review])
> I don't think this is the file you meant to attach. This is not a patch.
> 
Comment 5 David Kilzer (:ddkilzer) 2008-11-21 11:10:24 PST
Jack, you need to create a patch to the HTML file that you want changed.  Describing the change using plain text is not acceptable.  This will require that you check out the "source" for the WebKit site, modify the HTML, then run "svn diff" to create the patch.

Comment 6 Jack Wootton 2008-11-25 06:42:27 PST
Created attachment 25480 [details]
Patch file containing updated web site documentation page with link to existing Apple documentation.
Comment 7 David Kilzer (:ddkilzer) 2008-11-25 10:20:19 PST
Comment on attachment 25480 [details]
Patch file containing updated web site documentation page with link to existing Apple documentation.

Thanks for being so patient, Jack.  I'm going to mark this is as reviewed and land it with a slight change.

Please include a ChangeLog entry in future patches.  See this page for more details:

http://webkit.org/coding/contributing.html
Comment 8 David Kilzer (:ddkilzer) 2008-11-25 10:25:41 PST
$ git svn dcommit
Committing to http://svn.webkit.org/repository/webkit/trunk ...
	M	WebKitSite/ChangeLog
	M	WebKitSite/projects/documentation/index.html
Committed r38753

http://trac.webkit.org/changeset/38753