Merge lp:~teemperor/simple-scan/elementary-ui into lp:~simple-scan-team/simple-scan/trunk

[Raphael Isemann]

I changed the UI as seen here ( https://lists.launchpad.net/elementary-dev-community/msg01568.html ) to meet the elementary-HIG.

My code shouldn't affect simple-scan on other desktops then pantheon in any way.

I get the DE-name from the DESKTOP_SESSION variable, so to test the changes from another distribution you need to run "export DESKTOP_SESSION=pantheon" before executing simple-scan (from the same shell as the export-command).

New dependencies are only the granite library.

For quick feedback i'm also available on #elementary-dev (freenode) as "teemperor".

Have a nice evening!

[Robert Ancell (robert-ancell)]

So sorry! It appears I wasn't subscribed to merge requests so I didn't see this merge (I was expecting it though). Feel free to ping me directly if you think I've missed something like this.

[Robert Ancell (robert-ancell)]

What I like here is the removal of the menubar and the cog menu design - that is good to go upstream.

Issues with the branch as it stands:
- You have a hard dependency on granite - this needs to be optional.
- In saying that, I don't see the value in us taking a dependency on granite since it's Elementary specific. So I'd expect to get the majority of the functional changes into upstream simple-scan but still need a small patch for Elementary specific functionality.
- By removing the menu you have broken the right click functionality on a page.
- The cog menu is missing important functionality like email and print.
- You don't need the crop option in the cog menu (it is in right click on the page). Though it might be worth having the whole page menu as a submenu so it is discoverable.
- The about dialog renames and points at lp:elementary-scan - if this is upstreamed code it should point at lp:simple-scan
- Coding style doesn't match existing code (whitespace, comment style etc).
- Unnecessary changes in the branch - newline at the end of a function.

What I recommend you do is make this branch:
- Remove the current menubar (keeping the right click menu)
- Add the cog menu and populate with existing functionality.

We can then look in a second patch the value of adding an optional granite dependency.

review: Needs Fixing

[Eduard Gotwig (gotwig)]

this code is outdated, and the merge request should be removed.

Unmerged revisions

606. By Raphael Isemann on 2013-02-07

removed libgee from dependencies

605. By Raphael Isemann on 2013-02-07

removed the contractor-section that was already commented out

604. By Raphael Isemann on 2013-02-07

added an optional ui that meets the elementary-guidelines

=== modified file 'configure.ac'
--- configure.ac	2013-01-19 02:45:11 +0000
+++ configure.ac	2013-02-07 18:13:26 +0000
@@ -28,6 +28,7 @@
     gdk-pixbuf-2.0
     gudev-1.0
     sqlite3
+    granite
 ])
 
 PKG_CHECK_MODULES(COLORD, [
 
=== modified file 'src/Makefile.am'
--- src/Makefile.am	2012-02-03 20:49:21 +0000
+++ src/Makefile.am	2013-02-07 18:13:26 +0000
@@ -19,7 +19,8 @@
 	--pkg=gudev-1.0 \
 	--pkg=gio-2.0 \
 	--pkg=gtk+-3.0 \
-	--pkg=sqlite3
+	--pkg=sqlite3 \
+	--pkg=granite
 
 if HAVE_COLORD
 simple_scan_VALAFLAGS += -D HAVE_COLORD
 
=== modified file 'src/ui.vala'
--- src/ui.vala	2013-01-19 02:45:11 +0000
+++ src/ui.vala	2013-02-07 18:13:26 +0000
@@ -1,7 +1,7 @@
 /*
  * Copyright (C) 2009-2011 Canonical Ltd.
- * Author: Robert Ancell <[e-mail]>
- *
+ * Authors: Robert Ancell <[e-mail]>
+ *          Raphael Isemann <[e-mail]>
  * This program is free software: you can redistribute it and/or modify it under
  * the terms of the GNU General Public License as published by the Free Software
  * Foundation, either version 3 of the License, or (at your option) any later
@@ -92,6 +92,9 @@
     private int window_height;
     private bool window_is_maximized;
 
+    //indicates if the current enviroment is the pantheon shell
+    private bool is_pantheon;
+
     public signal void start_scan (string? device, ScanOptions options);
     public signal void stop_scan ();
     public signal void email (string profile);
@@ -104,9 +107,13 @@
 
         settings = new Settings ("org.gnome.SimpleScan");
 
+        string? env = Environment.get_variable("DESKTOP_SESSION");
+        is_pantheon = (env == "pantheon");
+
         load ();
 
         autosave_manager = AutosaveManager.create (ref book);
+
     }
 
     ~UserInterface ()
@@ -1095,8 +1102,29 @@
 
         /* Description of program */
         string description = _("Simple document scanning tool");
-
-        Gtk.show_about_dialog (window,
+        if(is_pantheon) 
+            Granite.Widgets.show_about_dialog ((Gtk.Window) widget,
+                                 "program_name", "Simple Scan with elementary UI",
+                                 "version", VERSION,
+                                 "logo_icon_name", "scanner",
+
+                                "comments", description,
+                                "copyright", "2009-2011 Canonical Ltd.",
+                                "website", "https://launchpad.net/elementary-scan",
+                                "website_label", "Website",
+
+                                "authors", authors,
+                                               //"documenters", about_documenters,
+                                               //"artists", about_artists,
+                                "translator_credits", _("translator-credits"),
+                                "license", license,
+                                "license_type", Gtk.License.GPL_3_0,
+
+                                "help", "https://answers.launchpad.net/elementary-scan",
+                                "translate", "https://translations.launchpad.net/simple-scan",
+                                "bug", "https://bugs.launchpad.net/elementary-scan");
+        else 
+            Gtk.show_about_dialog (window,
                                "title", title,
                                "program-name", "Simple Scan",
                                "version", VERSION,
@@ -1109,6 +1137,7 @@
                                "license", license,
                                "wrap-license", true,
                                null);
+
     }
 
     private bool on_quit ()
@@ -1316,6 +1345,44 @@
         photo_toolbar_menuitem = (Gtk.RadioMenuItem) builder.get_object ("photo_toolbutton_menuitem");
         photo_menu_menuitem = (Gtk.RadioMenuItem) builder.get_object ("photo_menuitem");
 
+        if(is_pantheon) {
+            //hide the menubar to follow elementary-guidelines
+            var menubar = (Gtk.MenuBar) builder.get_object ("menubar");
+            menubar.hide();
+
+            //unparent menuitems that we want in the appmenu
+            var crop_menuitem = (Gtk.MenuItem) builder.get_object ("crop_menuitem");
+            crop_menuitem.unparent ();
+            var content_menuitem = (Gtk.MenuItem) builder.get_object ("help_contents_menuitem");
+            content_menuitem.unparent ();
+            var about_menuitem = (Gtk.MenuItem) builder.get_object ("about_menuitem");
+            about_menuitem.unparent ();
+            var preferences_menuitem = (Gtk.MenuItem) builder.get_object ("preferences_menuitem");
+            preferences_menuitem.unparent ();
+
+            //build the appmenu
+            var menu = new Gtk.Menu ();
+            menu.append (crop_menuitem);
+            menu.append (preferences_menuitem);
+            menu.append (new Gtk.SeparatorMenuItem ());
+            menu.append (content_menuitem);
+            menu.append (about_menuitem);
+
+            var appmenu = new Granite.Widgets.ToolButtonWithMenu.from_stock (
+                Gtk.Stock.PROPERTIES, Gtk.IconSize.MENU, _("Menu"), menu);
+            var appmenu_toolitem = new Gtk.ToolItem ();
+            appmenu_toolitem.add (appmenu);
+
+            //create a spacer to align the elementary-menuitems on the right side
+            var spacer = new Gtk.ToolItem ();
+            spacer.set_expand (true);
+
+            var toolbar = (Gtk.Toolbar) builder.get_object ("toolbar1");
+            toolbar.insert (spacer, 8);
+            toolbar.insert (appmenu_toolitem, 9);
+            toolbar.show_all ();
+        }
+
         authorize_dialog = (Gtk.Dialog) builder.get_object ("authorize_dialog");
         authorize_label = (Gtk.Label) builder.get_object ("authorize_label");
         username_entry = (Gtk.Entry) builder.get_object ("username_entry");