RESOLVED FIXED 27937
Implement HTML5 nav element
https://bugs.webkit.org/show_bug.cgi?id=27937
Summary Implement HTML5 nav element
Maciej Stachowiak
Reported 2009-08-03 00:46:58 PDT
Created attachment 33962 [details] Preliminary patch - no tests yet. We should implement some of the simpler new elements from HTML5. Let's start with nav.
Attachments
Preliminary patch - no tests yet. (3.01 KB, patch)
2009-08-03 00:46 PDT, Maciej Stachowiak
no flags
slightly updated patch (2.41 KB, patch)
2009-08-03 01:11 PDT, Maciej Stachowiak
no flags
Patch v1 (2.52 KB, patch)
2009-08-18 19:20 PDT, Maciej Stachowiak
no flags
complete patch - with ChangeLog and tests (8.84 KB, patch)
2009-08-18 19:26 PDT, Maciej Stachowiak
sam: review+
Maciej Stachowiak
Comment 1 2009-08-03 01:11:27 PDT
Created attachment 33964 [details] slightly updated patch
Eric Seidel (no email)
Comment 2 2009-08-03 09:28:02 PDT
Comment on attachment 33964 [details] slightly updated patch Without a ChangeLog it's difficult for me to know why this is correct. :( Particularly why it should have tag-priority 5.
Maciej Stachowiak
Comment 3 2009-08-16 23:24:40 PDT
Comment on attachment 33964 [details] slightly updated patch Didn't mean to flag this.
Maciej Stachowiak
Comment 4 2009-08-18 19:20:31 PDT
Created attachment 35096 [details] Patch v1
Maciej Stachowiak
Comment 5 2009-08-18 19:26:29 PDT
Created attachment 35098 [details] complete patch - with ChangeLog and tests Curse you, bugzilla-tool.
Sam Weinig
Comment 6 2009-08-18 19:30:45 PDT
Comment on attachment 35098 [details] complete patch - with ChangeLog and tests > // Checks if a string is a valid tag for the FormatBlockCommand function of execCommand. Expects lower case strings. > bool validBlockTag(const String& blockTag) > { > + // FIXME: convert this to a HashSet Eek. Or at least AtomicStrings. > + > Index: LayoutTests/fast/html/nav-element.html > =================================================================== > --- LayoutTests/fast/html/nav-element.html (revision 0) > +++ LayoutTests/fast/html/nav-element.html (revision 0) > @@ -0,0 +1,64 @@ > +<!doctype HTML> > + > +</body> > +</style> What is this </style> about. r=me
Maciej Stachowiak
Comment 7 2009-08-18 22:10:59 PDT
Note You need to log in before you can comment on or make changes to this bug.