Bug 18477

Summary: No resize indication is shown when moving/hovering over the frame border/splitter.
Product: WebKit Reporter: Benjamin Meyer <ben>
Component: FramesAssignee: 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/
See Also: https://bugs.webkit.org/show_bug.cgi?id=268248
Attachments:
Description Flags
Patch to show resize cursor
none
Show resize cursor when hovering on frame border none

Benjamin Meyer
Reported 2008-04-14 01:43:22 PDT
Put your mouse over the frame border which you can drag to resize the frames. The cursor should change, but it does not.
Attachments
Patch to show resize cursor (1.14 KB, patch)
2008-04-17 11:51 PDT, Ariya Hidayat 		no flags
DetailsFormatted DiffDiff
Show resize cursor when hovering on frame border (1.07 KB, patch)
2008-04-18 05:44 PDT, Ariya Hidayat 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Holger Freyther
Comment 1 2008-04-17 06:23:02 PDT
Hmm, the cursor does change here when hovering over the frame border.
Holger Freyther
Comment 2 2008-04-17 06:35:58 PDT
Attempt to clarify, adding the Qt keyword to Qt bugs.
Ariya Hidayat
Comment 3 2008-04-17 06:57:54 PDT
Faster testcase (and not burden KOffice server unnecessarily) is this URI: data:text/html,<frameset cols=25%,75%><frame><frame></frameset>
Adam Roben (:aroben)
Comment 4 2008-04-17 11:03:20 PDT
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.
Ariya Hidayat
Comment 5 2008-04-17 11:51:24 PDT
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.
mitz
Comment 6 2008-04-17 11:55:42 PDT
Comment on attachment 20632 [details] Patch to show resize cursor + if (fs) { This null check is redundant.
Dave Hyatt
Comment 7 2008-04-17 12:17:52 PDT
Yeah, we want this fix everywhere, so remove the ifdef and fix the redundant null check and I'll r+.
Ariya Hidayat
Comment 8 2008-04-18 05:44:00 PDT
Created attachment 20665 [details] Show resize cursor when hovering on frame border Here is the updated patch.
Adam Roben (:aroben)
Comment 9 2008-04-18 06:38:57 PDT
Comment on attachment 20665 [details] Show resize cursor when hovering on frame border r=me
Simon Hausmann
Comment 10 2008-04-18 07:40:33 PDT
Landed in r32208
mitz
Comment 11 2008-04-18 08:29:48 PDT
*** Bug 17005 has been marked as a duplicate of this bug. ***
Jon@Chromium
Comment 12 2008-11-11 13:15:00 PST
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
Simon Hausmann
Comment 13 2008-12-10 01:17:12 PST
Comment on attachment 20665 [details] Show resize cursor when hovering on frame border Clearning review flag after the patch has been landed.
Note You need to log in before you can comment on or make changes to this bug.