[webkit-reviews] review requested: [Bug 71568] [GTK] Add a way to change web view settings in MiniBrowser : [Attachment 120663] Updated patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Dec 28 06:38:04 PST 2011


Carlos Garcia Campos <cgarcia at igalia.com> has asked  for review:
Bug 71568: [GTK] Add a way to change web view settings in MiniBrowser
https://bugs.webkit.org/show_bug.cgi?id=71568

Attachment 120663: Updated patch
https://bugs.webkit.org/attachment.cgi?id=120663&action=review

------- Additional Comments from Carlos Garcia Campos <cgarcia at igalia.com>
Several fixes in this new patch:

 - Support all setting types (boolean, string and uint)
 - Fix compile warning with the GValue stack initialization.
 - Use GtkToolItem instead of GtkMenuToolItem for the preferences toolbar
button.
 - Use the property nick instead of the property name in the dialog.
 - Show the property blurb as a tooltip.


More information about the webkit-reviews mailing list