Bug 33369 - Implement HTML5 <hgroup> element.
Summary: Implement HTML5 <hgroup> element.
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: DOM (show other bugs)
Version: 528+ (Nightly build)
Hardware: Other OS X 10.5
: P2 Normal
Assignee: Nobody
URL: http://www.w3.org/TR/html5/semantics....
Keywords: HTML5
Depends on:
Blocks: 32934
  Show dependency treegraph
 
Reported: 2010-01-08 01:48 PST by Kent Tamura
Modified: 2010-03-11 11:25 PST (History)
4 users (show)

See Also:


Attachments
Proposed patch (8.55 KB, patch)
2010-01-08 01:55 PST, Kent Tamura
no flags Details | Formatted Diff | Diff
Proposed patch (rev.2) (8.53 KB, patch)
2010-01-26 18:01 PST, Kent Tamura
abarth: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Kent Tamura 2010-01-08 01:48:34 PST
Implement HTML5 <hgroup> element.
Comment 1 Kent Tamura 2010-01-08 01:55:39 PST
Created attachment 46119 [details]
Proposed patch
Comment 2 WebKit Review Bot 2010-01-08 01:58:51 PST
style-queue ran check-webkit-style on attachment 46119 [details] without any errors.
Comment 3 Eric Seidel (no email) 2010-01-26 14:36:55 PST
Comment on attachment 46119 [details]
Proposed patch

The ChangeLog talks about <aside>?
Comment 4 Kent Tamura 2010-01-26 17:22:31 PST
(In reply to comment #3)
> The ChangeLog talks about <aside>?

Oh, silly copy&paste mistake. I'll update the patch...
Comment 5 Kent Tamura 2010-01-26 18:01:49 PST
Created attachment 47477 [details]
Proposed patch (rev.2)
Comment 6 Eric Seidel (no email) 2010-02-01 15:27:39 PST
Every time I see this change in the review queue I skip over it becase I don't know what this is for or why we'd want it.  I guess I just need to read the html5 spec.
Comment 7 Eric Seidel (no email) 2010-03-05 13:46:06 PST
Could you explain in the bug what <hgroup> is and why we would want it?
Comment 8 Kent Tamura 2010-03-07 16:50:50 PST
(In reply to comment #7)
> Could you explain in the bug what <hgroup> is and why we would want it?

Because it is defined in HTML5. 
http://www.w3.org/TR/html5/semantics.html#the-hgroup-element

Though I feel the benefit of <hgroup>is less than other sectioning elements such as <section> <article> we already have, the example in the specification is reasonable and <hgroup> is not harmful.
Comment 9 Adam Barth 2010-03-08 10:55:26 PST
Comment on attachment 47477 [details]
Proposed patch (rev.2)

Sorry this took so long to review.  HTML5++.
Comment 10 Kent Tamura 2010-03-08 22:27:47 PST
Landed as r55710.
Comment 11 Eric Seidel (no email) 2010-03-08 23:41:32 PST
It appears this checkin broke the Gtk bot:
http://build.webkit.org/results/GTK%20Linux%2064-bit%20Release/r55710%20(589)/results.html

(Or possibly the Gtk bot simply got itself wedged).
Comment 12 Gustavo Noronha (kov) 2010-03-11 11:25:06 PST
(In reply to comment #11)
> It appears this checkin broke the Gtk bot:
> http://build.webkit.org/results/GTK%20Linux%2064-bit%20Release/r55710%20(589)/results.html
> 
> (Or possibly the Gtk bot simply got itself wedged).

Yes, looks like a temporary failure of the bot there, should be fixed now.