[Webkit-unassigned] [Bug 34146] New: _selectedOrMainFrame should consider caret selection in determing a target frame

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=34146

           Summary: _selectedOrMainFrame should consider caret selection
                    in determing a target frame
           Product: WebKit
           Version: 528+ (Nightly build)
          Platform: Macintosh
        OS/Version: Mac OS X 10.6
            Status: UNCONFIRMED
          Severity: Normal
          Priority: P2
         Component: WebKit API
        AssignedTo: webkit-unassigned at lists.webkit.org
        ReportedBy: jalkut at red-sweater.com


Currently _selectedOrMainFrame relies only on the official "selectedFrame"
method on WebView, which considers a frame selected only if it has a
non-zero-length selection.

Many of the methods that invoke _selectedOrMainFrame to determine the
"selection" would behave more meaningfully in a non-firstResponder state, if
frames that had caret selection were considered in the search for the most
appropriate target frame.

Consider for example a WYSIWYG editor window where a non-editable mainFrame
contains a contentEditable frame. While the view is not first responder, it may
still retain a caret insertion selection that is a legitimate target for many
methods such as replaceSelectionWithMarkupString:, etc.

I do not believe that including caret selection frames in this private method's
search criteria would cause any negative impact for any existing code. Because
most of these methods are simply (infuratingly ;) ) failing now to do what the
developer wishes they would do, I think that expanding the target criteria for
frames would only result in an increase in "desirable behavior" when
determining the implicit selection.

I will submit a patch for this if the notion of extending
_selectedOrMainFrame's behavior isn't met with immediate dismissal ;)

Daniel

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.