RESOLVED FIXED 98883
[EFL][WK2] Add toolbar buttons to MiniBrowser
https://bugs.webkit.org/show_bug.cgi?id=98883
Summary [EFL][WK2] Add toolbar buttons to MiniBrowser
Chris Dumez
Reported 2012-10-10 03:55:49 PDT
It would be nice to add a "back" button to MiniBrowser.
Attachments
Patch (13.79 KB, patch)
2012-10-10 10:47 PDT, Chris Dumez
no flags
Screenshot (43.29 KB, image/png)
2012-10-10 10:48 PDT, Chris Dumez
no flags
Patch (13.79 KB, patch)
2012-10-10 11:01 PDT, Chris Dumez
no flags
Patch (13.79 KB, patch)
2012-10-10 11:08 PDT, Chris Dumez
no flags
Patch (13.80 KB, patch)
2012-10-10 11:12 PDT, Chris Dumez
no flags
Chris Dumez
Comment 1 2012-10-10 10:47:51 PDT
Chris Dumez
Comment 2 2012-10-10 10:48:18 PDT
Created attachment 168032 [details] Screenshot
Chris Dumez
Comment 3 2012-10-10 11:01:48 PDT
Created attachment 168037 [details] Patch Warning fix.
Chris Dumez
Comment 4 2012-10-10 11:08:08 PDT
Created attachment 168038 [details] Patch Fixed a few typos.
Chris Dumez
Comment 5 2012-10-10 11:12:36 PDT
Kenneth Rohde Christiansen
Comment 6 2012-10-10 14:35:09 PDT
Comment on attachment 168040 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=168040&action=review > Source/WebKit2/ChangeLog:8 > + Add "back,forward,list,changed" signal on the Ewk_View Now we are talking! > Tools/MiniBrowser/efl/main.c:217 > + /* Update navigation buttons state */ EFL supports // comments now :-) Apparently Rasterman accepted that!
WebKit Review Bot
Comment 7 2012-10-10 16:36:17 PDT
Comment on attachment 168040 [details] Patch Clearing flags on attachment: 168040 Committed r130978: <http://trac.webkit.org/changeset/130978>
WebKit Review Bot
Comment 8 2012-10-10 16:36:21 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.