Summary: | Can <th> serves as the AXTitleUIElement for <td>? | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | chris fleizach <cfleizach> | ||||
Component: | Accessibility | Assignee: | chris fleizach <cfleizach> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | ||||||
Priority: | P2 | ||||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | Mac | ||||||
OS: | OS X 10.5 | ||||||
Attachments: |
|
Description
chris fleizach
2008-12-02 14:59:53 PST
Created attachment 25687 [details]
patch to make <th> behave as title ui element
Comment on attachment 25687 [details] patch to make <th> behave as title ui element This is a nitpick, but we generally sentence-case our comments. I would appreciate it if you would do that in the comments below. (Meaning, capitalize the first letter of the first work in each sentence. > + // try to find if the first cell in this row is a <th>. if it is, > + // then it can act as the title ui element. (this is only in the > + // case when the table is not appearing as an AXTable) > + if (!m_renderer || isTableCell()) > + return 0; And here: > + // if this cell is in the first column, there is no need to continue > + int col = renderCell->col(); |