Bug 18477 - No resize indication is shown when moving/hovering over the frame border/splitter.
Summary: No resize indication is shown when moving/hovering over the frame border/spli...
Status: REOPENED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Frames (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Ariya Hidayat
URL: http://www.koffice.org/developer/apid...
Keywords:
: 17005 (view as bug list)
Depends on:
Blocks:
 
Reported: 2008-04-14 01:43 PDT by Benjamin Meyer
Modified: 2010-06-10 16:24 PDT (History)
4 users (show)

See Also:


Attachments
Patch to show resize cursor (1.14 KB, patch)
2008-04-17 11:51 PDT, Ariya Hidayat
no flags Details | Formatted Diff | Diff
Show resize cursor when hovering on frame border (1.07 KB, patch)
2008-04-18 05:44 PDT, Ariya Hidayat
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Benjamin Meyer 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.
Comment 1 Holger Freyther 2008-04-17 06:23:02 PDT
Hmm, the cursor does change here when hovering over the frame border.
Comment 2 Holger Freyther 2008-04-17 06:35:58 PDT
Attempt to clarify, adding the Qt keyword to Qt bugs.
Comment 3 Ariya Hidayat 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>
Comment 4 Adam Roben (:aroben) 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.
Comment 5 Ariya Hidayat 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.
Comment 6 mitz 2008-04-17 11:55:42 PDT
Comment on attachment 20632 [details]
Patch to show resize cursor

+        if (fs) {

This null check is redundant.
Comment 7 Dave Hyatt 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+.
Comment 8 Ariya Hidayat 2008-04-18 05:44:00 PDT
Created attachment 20665 [details]
Show resize cursor when hovering on frame border

Here is the updated patch.
Comment 9 Adam Roben (:aroben) 2008-04-18 06:38:57 PDT
Comment on attachment 20665 [details]
Show resize cursor when hovering on frame border

r=me
Comment 10 Simon Hausmann 2008-04-18 07:40:33 PDT
Landed in r32208
Comment 11 mitz 2008-04-18 08:29:48 PDT
*** Bug 17005 has been marked as a duplicate of this bug. ***
Comment 12 Jon@Chromium 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
Comment 13 Simon Hausmann 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.