| Summary: | No resize indication is shown when moving/hovering over the frame border/splitter. | ||||||||
|---|---|---|---|---|---|---|---|---|---|
| Product: | WebKit | Reporter: | Benjamin Meyer <ben> | ||||||
| Component: | Frames | Assignee: | Ariya Hidayat <ariya.hidayat> | ||||||
| Status: | REOPENED --- | ||||||||
| Severity: | Normal | CC: | ap, ariya.hidayat, aroben, hausmann | ||||||
| Priority: | P2 | ||||||||
| Version: | 528+ (Nightly build) | ||||||||
| Hardware: | All | ||||||||
| OS: | All | ||||||||
| URL: | http://www.koffice.org/developer/apidocs/ | ||||||||
| Attachments: |
|
||||||||
|
Description
Benjamin Meyer
2008-04-14 01:43:22 PDT
Hmm, the cursor does change here when hovering over the frame border. Attempt to clarify, adding the Qt keyword to Qt bugs. Faster testcase (and not burden KOffice server unnecessarily) is this URI: data:text/html,<frameset cols=25%,75%><frame><frame></frameset> Looks like this happen on Windows as well. The fix will presumably be in WebCore, so I'm going to make this a cross-platform bug. Created attachment 20632 [details]
Patch to show resize cursor
The attached patch solves the problem for Qt port. If it is not intended behavior on Safari (i.e. it is also a Safari bug), then I guess we can leave the #if part out.
Comment on attachment 20632 [details]
Patch to show resize cursor
+ if (fs) {
This null check is redundant.
Yeah, we want this fix everywhere, so remove the ifdef and fix the redundant null check and I'll r+. Created attachment 20665 [details]
Show resize cursor when hovering on frame border
Here is the updated patch.
Comment on attachment 20665 [details]
Show resize cursor when hovering on frame border
r=me
Landed in r32208 *** Bug 17005 has been marked as a duplicate of this bug. *** This still happens (with the original URL) in the nightly build of Safari for Windows. The nightly build of Safari for Mac does not have this problem. This problem also effects Chromium. See Chromium bug http://code.google.com/p/chromium/issues/detail?id=329 Comment on attachment 20665 [details]
Show resize cursor when hovering on frame border
Clearning review flag after the patch has been landed.
|