Bug 14913

Summary: GTK API does not allow going forwards/backwards by multiple steps
Product: WebKit Reporter: George Wright <gwright>
Component: WebKit Misc.Assignee: Nobody <webkit-unassigned>
Status: RESOLVED DUPLICATE    
Severity: Normal CC: christian, jasuarez, xan.lopez
Priority: P2    
Version: 523.x (Safari 3)   
Hardware: PC   
OS: Linux   
Attachments:
Description Flags
Proposed fix
none
Proposed fix, take 2 aroben: review-

George Wright
Reported 2007-08-09 09:04:01 PDT
The WebKit API allows you to go forwards and backwards in the browser history by more than one page. The GTK API does not. This should be implemented so that browsers can utilise such features.
Attachments
Proposed fix (2.70 KB, patch)
2007-08-09 09:26 PDT, George Wright 		no flags
DetailsFormatted DiffDiff
Proposed fix, take 2 (2.38 KB, patch)
2007-08-09 09:31 PDT, George Wright 		aroben: review-
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
George Wright
Comment 1 2007-08-09 09:26:31 PDT
Created attachment 15881 [details] Proposed fix
George Wright
Comment 2 2007-08-09 09:31:34 PDT
Created attachment 15882 [details] Proposed fix, take 2 Update to the originally proposed patch, which accidentally mangled the ChangeLog.
Holger Freyther
Comment 3 2007-08-09 17:51:31 PDT
Duplicate of bug #14811
Holger Freyther
Comment 4 2007-08-09 18:41:17 PDT
Default arguments for ANSI C? I think not. I would r=- it.
Adam Roben (:aroben)
Comment 5 2007-08-13 17:44:28 PDT
Comment on attachment 15882 [details] Proposed fix, take 2 r- due to Holger's comments.
Christian Dywan
Comment 6 2007-08-13 19:29:24 PDT
In my opinion this feature should be an extra function which behaves the same as goBackOrForward, taking the widget and the steps as arguments. The currently suggested implementation is a bit confusing.
Juan A. Suarez
Comment 7 2007-08-17 08:12:31 PDT
IMHO this bug should have Gtk keyword.
Xan Lopez
Comment 8 2007-10-06 14:50:33 PDT
+1 for closing this as duplicate of bug 14811. Even if we agree on adding this API it should come as a new function, not as an extra argument (with default value!?) of the existing functions.
Christian Dywan
Comment 9 2007-12-27 14:40:59 PST
*** This bug has been marked as a duplicate of 14811 ***
Note You need to log in before you can comment on or make changes to this bug.