Merge lp:~gero-bare/midori/midori-bug-1299988 into lp:midori

Proposed by Gero.Bare
Status: Merged
Approved by: Christian Dywan
Approved revision: 6840
Merged at revision: 6842
Proposed branch: lp:~gero-bare/midori/midori-bug-1299988
Merge into: lp:midori
Diff against target: 27 lines (+7/-3)
1 file modified
midori/midori-view.c (+7/-3)
To merge this branch: bzr merge lp:~gero-bare/midori/midori-bug-1299988
Reviewer Review Type Date Requested Status
Christian Dywan Approve
Review via email:

This proposal supersedes a proposal from 2014-11-19.

Commit message

Handle _NEW_WINDOW_ACTION explicitly to make _blank targets work

Description of the change

I hope this solve the error.
Now when the user makes a click in the link with "_blank" target midori opens a new tab and selects that one.

I hope not to be forgetting something.

<a href=""> An example</a>
<a href="" target="_blank"> Another example</a>

To post a comment you must log in.
Revision history for this message
Christian Dywan (kalikiana) wrote : Posted in a previous version of this proposal

../midori/midori-view.c:600:50 error: ‘request’ undeclared (first use in this function)
 if (!midori_view_get_tls_info (view, request, &tls_cert, &tls_flags, &hostname)

The line removing request seems accidental? And well, it doesn't build without it, so please fix :-)

review: Needs Fixing
Revision history for this message
Christian Dywan (kalikiana) wrote :

Nice! Thanks!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'midori/midori-view.c'
2--- midori/midori-view.c 2014-07-30 17:47:30 +0000
3+++ midori/midori-view.c 2014-11-22 18:30:31 +0000
4@@ -543,6 +543,11 @@
5 }
7 {
8+ const gchar* uri = webkit_uri_request_get_uri (
9+ webkit_navigation_policy_decision_get_request (WEBKIT_NAVIGATION_POLICY_DECISION (decision)));
10+ g_signal_emit (view, signals[NEW_TAB], 0, uri, FALSE);
11+ webkit_policy_decision_ignore(decision);
12+ return FALSE;
13 }
15 {
16@@ -553,9 +558,8 @@
17 return FALSE;
18 }
20- void* request = NULL;
21- const gchar* uri = webkit_uri_request_get_uri (
22- webkit_navigation_policy_decision_get_request (WEBKIT_NAVIGATION_POLICY_DECISION (decision)));
23+ WebKitURIRequest * request = webkit_navigation_policy_decision_get_request (WEBKIT_NAVIGATION_POLICY_DECISION (decision));
24+ const gchar* uri = webkit_uri_request_get_uri (request);
25 #else
26 const gchar* uri = webkit_network_request_get_uri (request);
27 #endif


People subscribed via source and target branches

to all changes: