Summary: | [Patch] QWebPage/QWebView::selectAll to select all content | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Urs Wolfer <uwolfer> | ||||||||
Component: | WebKit Qt | Assignee: | Nobody <webkit-unassigned> | ||||||||
Status: | RESOLVED FIXED | ||||||||||
Severity: | Enhancement | CC: | hausmann | ||||||||
Priority: | P2 | ||||||||||
Version: | 528+ (Nightly build) | ||||||||||
Hardware: | PC | ||||||||||
OS: | Linux | ||||||||||
Bug Depends on: | |||||||||||
Bug Blocks: | 50049 | ||||||||||
Attachments: |
|
Description
Urs Wolfer
2008-12-20 03:43:07 PST
Created attachment 26167 [details]
Patch to add QWebPage/QWebView::selectAll
Comment on attachment 26167 [details]
Patch to add QWebPage/QWebView::selectAll
I agree that the change is straight forward, and it looks correct. But I think this should be a function of QWebFrame only, not QWebPage or QWebView, because the selection is a property of a frame. In a HTML frameset with many frames each frame has its own selection.
Created attachment 27018 [details]
Updated patch as suggested by Simon
Created attachment 27020 [details]
Updated as discussed with Simon on IRC
Comment on attachment 27020 [details]
Updated as discussed with Simon on IRC
r=me
|