Bug 20007 - Signature of WebView::select-all differs from TextView::select-all
Summary: Signature of WebView::select-all differs from TextView::select-all
Status: ASSIGNED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKitGTK (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC Linux
: P2 Normal
Assignee: Christian Dywan
URL:
Keywords: Gtk
Depends on:
Blocks:
 
Reported: 2008-07-11 04:44 PDT by Christian Dywan
Modified: 2017-03-11 10:46 PST (History)
1 user (show)

See Also:


Attachments
Adjust select-all as suggested (1.92 KB, patch)
2008-11-24 13:35 PST, Christian Dywan
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Christian Dywan 2008-07-11 04:44:37 PDT
WebKitWebView has an action signal "select-all" that when called selects everything in the current page. The signature is this:

void select-all (WebKitWebView* web_view, gpointer user_data)

By comparison, GtkTextView has the same signal, but with a different signature:

void select-all (GtkTextView* text_view, gboolean select, gpointer user_data)

If the boolean "select" is TRUE, everything is selected, which is what WebView currently does. But if it is FALSE, everything is unselected. It is actually possible to call "select-all" without a boolean on a TextView but the behaviour is the reverse of what WebView does.

I think we should really fix the signature of WebView to match TextView. I think it's not probable that it breaks anything because I don't think that one usually connects a callback to that signal.
Comment 1 Christian Dywan 2008-11-24 13:35:34 PST
Created attachment 25445 [details]
Adjust select-all as suggested

This adjusts select-all as suggested. Unless anyone actually connected to the signal everything will work fine, at least in the C API. I don't really know about bindings, maybe someone more experienced in that area can confirm that.
Comment 2 Eric Seidel (no email) 2008-11-25 18:02:14 PST
Comment on attachment 25445 [details]
Adjust select-all as suggested

Alp is our gtk reviewer, no?
Comment 3 Christian Dywan 2008-11-26 00:46:37 PST
Comment on attachment 25445 [details]
Adjust select-all as suggested

(In reply to comment #2)
> (From update of attachment 25445 [details] [review])
> Alp is our gtk reviewer, no?

Alp isn't the sole Gtk reviewer, granted he tends to assume that position for the most part. Either way, let's take it off the queue for the moment, a discussion on IRC made clear that it isn't undisputed.