Bug 14913 - GTK API does not allow going forwards/backwards by multiple steps
Summary: GTK API does not allow going forwards/backwards by multiple steps
Status: RESOLVED DUPLICATE of bug 14811
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit Misc. (show other bugs)
Version: 523.x (Safari 3)
Hardware: PC Linux
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2007-08-09 09:04 PDT by George Wright
Modified: 2007-12-27 14:40 PST (History)
3 users (show)

See Also:


Attachments
Proposed fix (2.70 KB, patch)
2007-08-09 09:26 PDT, George Wright
no flags Details | Formatted Diff | Diff
Proposed fix, take 2 (2.38 KB, patch)
2007-08-09 09:31 PDT, George Wright
aroben: review-
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description George Wright 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.
Comment 1 George Wright 2007-08-09 09:26:31 PDT
Created attachment 15881 [details]
Proposed fix
Comment 2 George Wright 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.
Comment 3 Holger Freyther 2007-08-09 17:51:31 PDT
Duplicate of bug #14811
Comment 4 Holger Freyther 2007-08-09 18:41:17 PDT
Default arguments for ANSI C? I think not. I would r=- it.
Comment 5 Adam Roben (:aroben) 2007-08-13 17:44:28 PDT
Comment on attachment 15882 [details]
Proposed fix, take 2

r- due to Holger's comments.
Comment 6 Christian Dywan 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.
Comment 7 Juan A. Suarez 2007-08-17 08:12:31 PDT
IMHO this bug should have Gtk keyword.
Comment 8 Xan Lopez 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.
Comment 9 Christian Dywan 2007-12-27 14:40:59 PST

*** This bug has been marked as a duplicate of 14811 ***