name changed to recorder
[maemo-recorder] / src / maemo-recorder-ui.c
index 0bfc029..7e7381a 100644 (file)
  *
  */
 
-
 #include <gst/gst.h>
 #include <glib/gi18n-lib.h>
 #include <libgnomevfs/gnome-vfs.h>
 #include <locale.h>
-#include <hildon-widgets/hildon-program.h>
-#include <hildon-widgets/hildon-note.h>
-#include <hildon-widgets/hildon-banner.h>
-#include <hildon-widgets/hildon-defines.h>
-#include <hildon-widgets/hildon-file-system-model.h>
-#include <hildon-widgets/hildon-file-chooser-dialog.h>
+#include <hildon/hildon-program.h>
+#include <hildon/hildon-note.h>
+#include <hildon/hildon-banner.h>
+#include <hildon/hildon-defines.h>
+#include <hildon/hildon-file-system-model.h>
+#include <hildon/hildon-file-chooser-dialog.h>
 #include <string.h>
 #include <sys/time.h>
-
+#include <libmodest-dbus-client/libmodest-dbus-client.h>
 #include "maemo-recorder.h"
 #include "maemo-recorder-ui.h"
 #include "maemo-recorder-file.h"
@@ -56,7 +55,7 @@
 #define GST_TIME_MSECS(t) \
         (guint) (((GstClockTime)(t)) % GST_SECOND)
 
-#define RECORDER_APP_TITLE "Maemo Recorder"
+#define RECORDER_APP_TITLE "Recorder"
 #define RECORDER_MSG_READY _("Ready")
 #define RECORDER_MSG_STOPPED _("Stopped")
 #define RECORDER_MSG_PAUSED _("Paused")
@@ -64,6 +63,8 @@
 #define RECORDER_MSG_RECORDING _("Recording")
 #define RECORDER_FILE_UNTITLED _("Untitled")
 
+#define RECORDER_FMT_STRING_NONE _("N/A")
+
 /* general enumerations */
     
 typedef enum
@@ -72,19 +73,6 @@ typedef enum
     DTX_ON = 1
 } DTX;
 
-/* menu codes */
-typedef enum 
-{
-    MENU_FILE_NEW = 1,
-    MENU_FILE_OPEN,
-    MENU_FILE_SAVE,
-    MENU_FILE_SAVE_AS,
-    MENU_FILE_REC,
-    MENU_FILE_PLAY,
-    MENU_FILE_STOP,
-    MENU_FILE_QUIT
-} MenuActionCode;
-
 typedef enum 
 {
     PIPELINE_PLAY = 1,
@@ -107,6 +95,7 @@ static void pipelineStateChanged (GstElement *element,
 static void seekToTime(GstElement *pipeline, gdouble secs);
 static gboolean seekToZero(AppData *data, GstElement *pipeline);
 static void setLength(AppData *data, gdouble secs);
+static void setFormatString(AppData *data, AudioFormat afmt);
 static gboolean cbStopPlayback(AppData *data);
 static void cbStop(GtkWidget* widget, AppData *data);
 static void cbPlay(GtkWidget* widget, AppData *data);
@@ -129,25 +118,11 @@ static gboolean cbUpdateRecLength(AppData *data);
 static void cbDestroy(GtkWidget* widget, GdkEvent *event, gpointer data);
 static gboolean openURI(gpointer user_data);
 static gboolean closeFile(AppData *data);
+static const gchar *getFileName(AppData *data);
 static gdouble guessMediaLength(AppData *data);
 static GstCaps *createCapsFilter(AudioFormat format);
 
-static void new_pad_cb (GstElement *wavparse, GstPad *new_pad, gpointer data)
-{
-    GstElement *sink;
-    AppData* app = (AppData*) data; 
-
-    ULOG_INFO("new pad");
-    sink = gst_element_factory_make ("dsppcmsink", "sink");
-    gst_bin_add (GST_BIN (app->playPipeline), sink);
-    if (!gst_element_link (wavparse, sink))
-      g_error ("link(wavparse, sink) failed!");
-    gst_element_sync_state_with_parent(sink); 
-}
+static gboolean lengthSet = FALSE;
 
 static gboolean createPipeline(AppData *app, PipeLineType type)
 {
@@ -157,31 +132,29 @@ static gboolean createPipeline(AppData *app, PipeLineType type)
     GstElement *queue = NULL;
     GstElement *pipeline = NULL;
     GstElement *parse = NULL;
+    GstElement *bin = NULL;
     GstCaps *caps = NULL;
 
     g_assert(NULL != app);
 
-    pipeline = gst_pipeline_new("pipeline");
-
-    gst_bus_add_watch(gst_pipeline_get_bus (GST_PIPELINE (pipeline)),
-                   cbBus, app);
-
     /* create elements */
     switch (type)
     {
         case PIPELINE_PLAY_MP3:
             ULOG_INFO("mp3 playback - queue");
-            src = gst_element_factory_make ("gnomevfssrc", "source");
-            queue = gst_element_factory_make ("queue", "queue");
-            sink = gst_element_factory_make ("dspmp3sink", "sink");
+            bin = gst_element_factory_make ("playbin2", "bin");
+            gchar* uri = g_strdup_printf("file://%s", app->openFileName);
+            g_object_set(G_OBJECT(bin), 
+                "uri", uri, 
+                NULL);
+            g_free(uri);
+            gst_bus_add_watch(gst_pipeline_get_bus (GST_PIPELINE (bin)),
+                   cbBus, app);
 
-            g_object_set(G_OBJECT (queue), 
-                "min-threshold-bytes", 131072,
-                NULL );
-            g_object_set(G_OBJECT(src), 
-                    "location", app->openFileName, 
-                    "blocksize", 65536,
-                    NULL);
+            app->playPipeline = bin;
+            app->playPipelineType = type;
+                     
+            return TRUE;
             break;
 
         case PIPELINE_PLAY:
@@ -199,12 +172,25 @@ static gboolean createPipeline(AppData *app, PipeLineType type)
                 case FORMAT_PCMU:
                 case FORMAT_PCM:
                     ULOG_INFO("using pcm sink");
-                    sink = gst_element_factory_make ("dsppcmsink", "sink");
+                    sink = gst_element_factory_make ("pulsesink", "sink");
                     break;
 
                 case FORMAT_WAV:
                     ULOG_INFO("using wavparse & pcm sink");
-                    parse = gst_element_factory_make ("wavparse", "parse");
+                    bin = gst_element_factory_make ("playbin2", "bin");
+                    gchar* uri = g_strdup_printf("file://%s", app->openFileName);
+                    g_object_set(G_OBJECT(bin),
+                        "uri", uri,
+                        NULL);
+                    g_free(uri);
+                    
+                    gst_bus_add_watch(gst_pipeline_get_bus (GST_PIPELINE (bin)),
+                        cbBus, app);
+
+                    app->playPipeline = bin;
+                    app->playPipelineType = type;
+                              
+                    return TRUE;
                     break;
                     
                 default:
@@ -229,6 +215,7 @@ static gboolean createPipeline(AppData *app, PipeLineType type)
                     src = gst_element_factory_make("dspilbcsrc", "source");
                     g_object_set(G_OBJECT(src),
                         "dtx", DTX_OFF,
+                        "mode", 1,
                         NULL);
                     break;
         
@@ -236,20 +223,16 @@ static gboolean createPipeline(AppData *app, PipeLineType type)
                 case FORMAT_PCMU:
                 case FORMAT_PCM:
                     ULOG_INFO("using pcm source");
-                    src = gst_element_factory_make("dsppcmsrc", "source");
-                    g_object_set(G_OBJECT (src), 
+                    src = gst_element_factory_make("pulsesrc", "source");
+                    /*g_object_set(G_OBJECT (src), 
                           "blocksize", DEFAULT_REC_BLOCKSIZE, 
                           "dtx", DTX_OFF,
-                        NULL);
+                        NULL);*/
                     break;
             
                 case FORMAT_WAV:
                     ULOG_INFO("using pcm source & wavenc");
-                    src = gst_element_factory_make("dsppcmsrc", "source");
-                    g_object_set(G_OBJECT (src),
-                            "blocksize", DEFAULT_REC_BLOCKSIZE,
-                            "dtx", DTX_OFF,
-                            NULL);
+                    src = gst_element_factory_make("pulsesrc", "source");
                     parse = gst_element_factory_make("wavenc", "enc");
                     break;   
             
@@ -273,17 +256,21 @@ static gboolean createPipeline(AppData *app, PipeLineType type)
 
         default:
             ULOG_ERR("Invalid pipeline type!");
-            gst_object_unref(pipeline);
             return FALSE;
     }
 
+    pipeline = gst_pipeline_new("pipeline");
+
+    gst_bus_add_watch(gst_pipeline_get_bus (GST_PIPELINE (pipeline)),
+                   cbBus, app);
+    
     if (!src || !pipeline)
     {
         ULOG_ERR("Could not create GstElement!");
         return FALSE;
     }
 
-    if (!sink && app->filter != FORMAT_WAV)
+    if (!sink && app->filter != FORMAT_WAV )
     {
         ULOG_ERR("Could not create GstElement!");
         return FALSE;        
@@ -321,7 +308,7 @@ static gboolean createPipeline(AppData *app, PipeLineType type)
                         ULOG_ERR("gst_element_link failed for src, parse and sink!");
                     }
                     break;
-         
+
                 default:
                     break;
           
@@ -349,29 +336,16 @@ static gboolean createPipeline(AppData *app, PipeLineType type)
 
                     break;
 
-                case FORMAT_WAV:
-                    gst_bin_add_many(GST_BIN(pipeline), src, parse, NULL);
-                    if (!gst_element_link_many (src, parse, NULL))
-                    {
-                        ULOG_ERR("gst_element_link failed for src, parse and sink!");
-                        return FALSE;
-                    }
-                    app->playPipeline = pipeline;
-                    g_signal_connect(parse, "pad_added",
-                                 G_CALLBACK(new_pad_cb), app);
-                    break;
-
                 default:
                     break;
             }
 
             break;
      
-        case PIPELINE_PLAY_MP3:
         default:
-            gst_bin_add_many(GST_BIN(pipeline), src, queue, sink, NULL);
+            gst_bin_add_many(GST_BIN(pipeline), src, sink, NULL);
  
-            if(!gst_element_link_many(src, queue, sink, NULL))
+            if(!gst_element_link_many(src, sink, NULL))
             {
                  ULOG_ERR("gst_element_link failed for src and sink!");
                  return FALSE;
@@ -418,6 +392,10 @@ static gboolean destroyPipeline(AppData *data, PipeLineType type)
 
     ULOG_INFO("%s() - Stopping playback/recording", G_STRFUNC);
 
+    hildon_banner_show_information(GTK_WIDGET(data->mainView), NULL, RECORDER_MSG_STOPPED);         
+    gtk_entry_set_text(GTK_ENTRY(data->mainViewData.stateEntry), 
+                         RECORDER_MSG_STOPPED);  
+
     switch (type)
     {
         case PIPELINE_REC:
@@ -524,18 +502,6 @@ static void pipelineStateChanged (GstElement *element,
                   hildon_banner_show_information(GTK_WIDGET(data->mainView), NULL, RECORDER_MSG_PAUSED);       
                   gtk_entry_set_text(GTK_ENTRY(data->mainViewData.stateEntry), 
                          RECORDER_MSG_PAUSED);
-                  /*
-                  gtk_tool_button_set_stock_id(GTK_TOOL_BUTTON(data->buttonPlay), GTK_STOCK_MEDIA_PLAY);
-                  gtk_widget_set_state(data->buttonPlay, GTK_STATE_NORMAL);
-                  */
-              }
-              else
-              {
-                  hildon_banner_show_information(GTK_WIDGET(data->mainView), NULL, RECORDER_MSG_STOPPED);       
-                  gtk_entry_set_text(GTK_ENTRY(data->mainViewData.stateEntry), 
-                         RECORDER_MSG_STOPPED);  
-                  gtk_widget_set_state(data->buttonPlay, GTK_STATE_NORMAL);
-                  gtk_tool_button_set_stock_id(GTK_TOOL_BUTTON(data->buttonPlay), GTK_STOCK_MEDIA_PLAY);
               }
               gtk_widget_set_state(data->buttonRec, GTK_STATE_NORMAL);
           }
@@ -543,9 +509,6 @@ static void pipelineStateChanged (GstElement *element,
           
         case GST_STATE_NULL:
           ULOG_INFO("%s() - Null", G_STRFUNC);
-          hildon_banner_show_information(GTK_WIDGET(data->mainView), NULL, RECORDER_MSG_STOPPED);
-          gtk_entry_set_text(GTK_ENTRY(data->mainViewData.stateEntry), 
-                     RECORDER_MSG_READY);  
           gtk_widget_set_state(data->buttonPlay, GTK_STATE_NORMAL);
           gtk_tool_button_set_stock_id(GTK_TOOL_BUTTON(data->buttonPlay), GTK_STOCK_MEDIA_PLAY);
           gtk_widget_set_state(data->buttonRec, GTK_STATE_NORMAL);
@@ -601,6 +564,10 @@ static gboolean cbBus(GstBus *bus,
                  case APPSTATE_PLAYING:
                     /* stop playback after a short break*/
                     g_timeout_add(STOP_DELAY, (GSourceFunc)cbStopPlayback, data);
+                    hildon_banner_show_information(GTK_WIDGET(app->mainView), NULL, RECORDER_MSG_STOPPED);       
+                    gtk_entry_set_text(GTK_ENTRY(app->mainViewData.stateEntry), 
+                         RECORDER_MSG_STOPPED);  
+
                     break;
 
                  case APPSTATE_RECORDING:
@@ -618,6 +585,7 @@ static gboolean cbBus(GstBus *bus,
              break;
         }
 
+        case GST_MESSAGE_ASYNC_DONE:
         case GST_MESSAGE_STATE_CHANGED: 
         {
             GstState old;
@@ -649,8 +617,8 @@ static void cbDestroy(GtkWidget* widget, GdkEvent *event, gpointer data)
 
     app = (AppData *) data;
 
-    if (!closeFile(app))
-        return;
+    ULOG_DEBUG("delete_event");
+    closeFile(app);
 
     destroyPipelines(app);
     if (app->playPipeline)
@@ -666,7 +634,6 @@ static gboolean cbCheckPosition (AppData *data)
 {
     GstFormat fmt = GST_FORMAT_TIME;
     gint64 pos = 0, len = 0;
-    static gboolean lengthSet = FALSE;
 
     g_assert(NULL != data);
 
@@ -737,8 +704,6 @@ static void cbNew(GtkWidget* widget, AppData *data)
     gtk_widget_set_sensitive(data->buttonSaveAs, FALSE);
     data->saved = TRUE;
 
-    gtk_window_set_title(GTK_WINDOW(data->mainView), RECORDER_FILE_UNTITLED);
-
     ULOG_DEBUG_F("cbNew end");
 }
 
@@ -809,13 +774,12 @@ static void cbOpen(GtkWidget* widget, AppData *data)
         basename = g_path_get_basename(selected);
 
         gtk_entry_set_text(GTK_ENTRY(data->mainViewData.fileNameEntry), selected);
-        gtk_window_set_title(GTK_WINDOW(data->mainView), basename);
         g_free(basename);
 
         g_free(data->openFileName);
         data->openFileName = tmpfile;
         data->file_format = format;
-        data->filter = format;
+        //data->filter = format;
         g_free(data->saveFileName);
         data->saveFileName = NULL;
         gtk_widget_set_sensitive(data->buttonSaveAs, TRUE);
@@ -826,6 +790,7 @@ static void cbOpen(GtkWidget* widget, AppData *data)
         else
             setLength(data, 0.0);
 
+        setFormatString(data, data->file_format);
         data->saved = TRUE;
     }
     else
@@ -882,13 +847,12 @@ openURI(gpointer user_data)
         basename = g_path_get_basename(selected);
         
         gtk_entry_set_text(GTK_ENTRY(data->mainViewData.fileNameEntry), selected);
-        gtk_window_set_title(GTK_WINDOW(data->mainView), basename);
         g_free(basename);
 
         g_free(data->openFileName);
         data->openFileName = tmpfile;
         data->file_format = format;
-        data->filter = format;
+        //data->filter = format;
         g_free(data->saveFileName);
         data->saveFileName = NULL;
         gtk_widget_set_sensitive(data->buttonSaveAs, TRUE);
@@ -899,6 +863,7 @@ openURI(gpointer user_data)
         else
             setLength(data, 0.0);
 
+        setFormatString(data, data->file_format);
         data->saved = TRUE;
     }
     else
@@ -918,6 +883,7 @@ static void openPlayPipeline( AppData *data )
     GstFormat fmt = GST_FORMAT_TIME;
     gint64 len;
     gdouble size = 0;
+    lengthSet = FALSE;
     /* create pipelines */
     /* check file type */
     switch (data->file_format)
@@ -927,14 +893,14 @@ static void openPlayPipeline( AppData *data )
         case FORMAT_PCM:
         case FORMAT_ILBC:
         case FORMAT_WAV:
-            destroyPipelines(data);
-            data->filter = data->file_format;
+            //destroyPipelines(data);
+            //data->filter = data->file_format;
             createPipeline(data, PIPELINE_PLAY);
             break;
 
         case FORMAT_MP3:
             destroyPipelines(data);
-            data->filter = data->file_format;
+            //data->filter = data->file_format;
             createPipeline(data, PIPELINE_PLAY_MP3);
             break;
 
@@ -965,13 +931,11 @@ static void openPlayPipeline( AppData *data )
             setLength(data, size);
         }
     }
-    /*
     else 
     {
-        ULOG_INFO("playSize else");
         setLength(data, 0.0);
     }
-    */
+    
 }
 
 /* returns whether the action can proceed or should be aborted */
@@ -987,7 +951,6 @@ closeFile(AppData *data)
     note = hildon_note_new_confirmation_add_buttons(GTK_WINDOW(data->mainView), _("Save recording?"),
             _("Yes"), GTK_RESPONSE_YES,
             _("No"), GTK_RESPONSE_NO,
-            _("Cancel"), GTK_RESPONSE_CANCEL,
             NULL);
 
     i = gtk_dialog_run(GTK_DIALOG(note));
@@ -1011,84 +974,49 @@ closeFile(AppData *data)
     }
     return FALSE;
 }
-     
-#if 0
-static void cbSave(GtkWidget* widget, AppData *data) 
+
+static const gchar *
+getFileName(AppData *data)
 {
-     GtkWidget* dialog = NULL;
-     const gchar *current;
-     gchar *selected = NULL;
-
-     g_assert(data);
-
-     ULOG_DEBUG("%s() - begin", G_STRFUNC);
-
-     current = gtk_entry_get_text(GTK_ENTRY(data->mainViewData.fileNameEntry));
-     if (NULL == current || strcmp(current, RECORDER_FILE_UNTITLED) == 0) 
-     {
-       hildon_banner_show_information(GTK_WIDGET(data->mainView), GTK_STOCK_DIALOG_ERROR, _("Nothing to save"));
-       return;
-     }
-
-     /* if saveFileName does not exist run saveas */
-     if (NULL == data->saveFileName)
-     {
-      /* create dialog */
-      dialog = GTK_WIDGET(hildon_file_chooser_dialog_new(
-                   GTK_WINDOW(data->mainView), 
-                   GTK_FILE_CHOOSER_ACTION_SAVE));
-      
-      /* show it */
-      gtk_widget_show_all(dialog);
-      
-      if (gtk_dialog_run(GTK_DIALOG(dialog)) == GTK_RESPONSE_OK) 
-      {
-           selected = gtk_file_chooser_get_filename(GTK_FILE_CHOOSER(dialog));
-      }
-     
-      ULOG_DEBUG("%s() - dialog finished", G_STRFUNC);
-      
-      gtk_widget_destroy(dialog);
-      
-      if (NULL != selected)
-      {
-           ULOG_INFO("%s() - selected filename = '%s'", G_STRFUNC,
-            selected);
-           g_free(data->saveFileName);
-           data->saveFileName = g_strdup_printf("%s%s", selected, getExtension(data->file_format));
-           g_free(selected);
-           selected = NULL;
-      }
-      else
-      {
-           return;
-      }
-     }
-
-     /* save the file */
-     if (doSave(gtk_entry_get_text(GTK_ENTRY(data->mainViewData.fileNameEntry)), data->saveFileName, data->file_format))
-     {
-      gchar *basename = g_path_get_basename(data->saveFileName);
-      ULOG_INFO("%s() - file succesfully saved!", G_STRFUNC);
-      g_free(data->openFileName);
-      data->openFileName = g_strdup(data->saveFileName);
-            
-      gtk_entry_set_text(GTK_ENTRY(data->mainViewData.fileNameEntry), 
-                 data->saveFileName);
-      gtk_widget_set_sensitive(data->buttonSave, FALSE);
-      gtk_window_set_title(GTK_WINDOW(data->mainView), basename);
-      g_free(basename);
-     }
-     
-     ULOG_DEBUG("%s() - end", G_STRFUNC);
+    g_assert(data);
+    return gtk_entry_get_text(GTK_ENTRY(data->mainViewData.fileNameEntry));
+
 }
-#endif
 
 static void cbSettings(GtkWidget* widget, AppData *data)
 {
     settings_edit( widget, data );
 }
 
+static void cbEmailing(GtkWidget* widget, AppData *data)
+{
+
+    gboolean result;
+    GSList *list = NULL;    
+    gchar *file = NULL;
+
+    g_assert(NULL != data);
+    
+    if (g_file_test(getFileName(data), G_FILE_TEST_EXISTS))
+    {
+        file = file2uri(getFileName(data));
+        ULOG_INFO("Emailing: %s", file);
+        list = g_slist_append(list, file);
+        result = libmodest_dbus_client_compose_mail(data->osso,
+                NULL, /*to*/
+                NULL, /*cc*/
+                NULL, /*bcc*/
+                NULL, /*body*/
+                NULL, /*subj*/
+                list);
+        if (!result)
+            hildon_banner_show_information(GTK_WIDGET(data->mainView), GTK_STOCK_DIALOG_ERROR, _("Emailing failed"));
+
+        g_slist_free(list);
+        g_free(file);
+    }
+}
+
 static void cbSaveAs(GtkWidget* widget, AppData *data) 
 {
     GtkWidget* dialog = NULL;
@@ -1099,7 +1027,7 @@ static void cbSaveAs(GtkWidget* widget, AppData *data)
 
     ULOG_DEBUG("%s() - begin", G_STRFUNC);
 
-    current = gtk_entry_get_text(GTK_ENTRY(data->mainViewData.fileNameEntry));
+    current = getFileName(data);
     if (NULL == current || strcmp(current, RECORDER_FILE_UNTITLED) == 0) 
     {
         hildon_banner_show_information(GTK_WIDGET(data->mainView), GTK_STOCK_DIALOG_ERROR, _("Nothing to save"));
@@ -1144,8 +1072,6 @@ static void cbSaveAs(GtkWidget* widget, AppData *data)
         basename = g_path_get_basename(data->saveFileName);
         ULOG_DEBUG("%s() - file '%s' succesfully saved!", G_STRFUNC, data->saveFileName);
 
-        gtk_window_set_title(GTK_WINDOW(data->mainView), basename);
-
         /* Houston, we have a kludge:
          * for AU files we need to keep the old tmpfile for playback
          * for RAW/iLBC files, we can remove the tmpfile and point openFileName to the saved file
@@ -1183,6 +1109,8 @@ static void cbRec(GtkWidget* widget, AppData *data)
     if (APPSTATE_READY != getAppState(data))
     {
         ULOG_WARN("%s() - state different than READY -> return", G_STRFUNC);
+        if (APPSTATE_RECORDING == getAppState(data))
+            cbStop(widget,data);
         return;
     }
 
@@ -1241,8 +1169,8 @@ static void cbRec(GtkWidget* widget, AppData *data)
 
         setAppState(data, APPSTATE_RECORDING);
         gtk_widget_set_sensitive(data->buttonSaveAs, TRUE);
-        gtk_window_set_title(GTK_WINDOW(data->mainView), RECORDER_FILE_UNTITLED);
         data->file_format = data->filter;
+        setFormatString(data, data->file_format);
     }
     else
     {
@@ -1250,7 +1178,9 @@ static void cbRec(GtkWidget* widget, AppData *data)
         hildon_banner_show_information(GTK_WIDGET(data->mainView), GTK_STOCK_DIALOG_ERROR, _("Could not create pipeline"));
         setAppState(data, APPSTATE_READY);
     }
+
+    gtk_widget_set_sensitive(data->buttonPlay, FALSE);
+
     ULOG_DEBUG("%s() - end", G_STRFUNC);     
 }
 
@@ -1262,7 +1192,7 @@ static void cbPlay(GtkWidget* widget, AppData *data)
 
     ULOG_DEBUG("%s() - begin", G_STRFUNC);
 
-    file = gtk_entry_get_text(GTK_ENTRY(data->mainViewData.fileNameEntry));
+    file = getFileName(data);
     if (NULL == data->openFileName || NULL == file || strcmp(file, RECORDER_FILE_UNTITLED) == 0) 
     {
         ULOG_WARN("%s() - nothing to play", G_STRFUNC);
@@ -1289,18 +1219,6 @@ static void cbPlay(GtkWidget* widget, AppData *data)
 
     ULOG_INFO("filename %s", file);
      
-    /*openPlayPipeline( data );*/
-     
-    /*if ( ! GST_IS_ELEMENT(data->playPipeline) ) 
-    {
-        if (g_strrstr(data->openFileName, EXTENSION_RAW))
-        {
-             ULOG_INFO("cbOpen() - file was raw, assuming audio/x-raw-int, 8kHz, 1 ch, 16-bit");
-             destroyPipelines(data);
-             createPipeline(data, PIPELINE_PLAY);      
-        }
-    }*/
-     
     if (! GST_IS_ELEMENT(data->playPipeline))
     {
         ULOG_WARN("%s() - playPipeline does not exist", G_STRFUNC);
@@ -1325,20 +1243,11 @@ static void cbStop(GtkWidget* widget, AppData *data)
 
     /* check if we are playing/recording */
 
-    /*
-    if (APPSTATE_PLAYING != getAppState(data) &&
-    APPSTATE_RECORDING != getAppState(data))
-    {
-      ULOG_WARN("cbStop() - state different than PLAYING or RECORDING "
-          "-> return");
-      return;
-    } 
-    */
-
     /* stop playing or recording */
     gtk_tool_button_set_stock_id(GTK_TOOL_BUTTON(data->buttonPlay), GTK_STOCK_MEDIA_PLAY);
     gtk_widget_set_state(data->buttonPlay, GTK_STATE_NORMAL);
     gtk_widget_set_state(data->buttonRec, GTK_STATE_NORMAL);
+    gtk_widget_set_sensitive(data->buttonPlay, TRUE);
 
     /* destroy related pipeline */
     switch(getAppState(data))
@@ -1417,12 +1326,20 @@ static GtkWidget* createToolBar(AppData *data)
      gtk_widget_set_sensitive(data->buttonSaveAs, FALSE);
      data->saved = TRUE;
 
+     gtk_tool_item_set_expand( GTK_TOOL_ITEM(new), TRUE );
+     gtk_tool_item_set_expand( GTK_TOOL_ITEM(open), TRUE );
+     gtk_tool_item_set_expand( GTK_TOOL_ITEM(saveas), TRUE );
+
      rec = gtk_tool_button_new_from_stock(GTK_STOCK_MEDIA_RECORD); 
      data->buttonRec = GTK_WIDGET(rec);
      play = gtk_tool_button_new_from_stock(GTK_STOCK_MEDIA_PLAY); 
      data->buttonPlay = GTK_WIDGET(play);
      stop = gtk_tool_button_new_from_stock(GTK_STOCK_MEDIA_STOP);
      
+     gtk_tool_item_set_expand( GTK_TOOL_ITEM(rec), TRUE );
+     gtk_tool_item_set_expand( GTK_TOOL_ITEM(play), TRUE );
+     gtk_tool_item_set_expand( GTK_TOOL_ITEM(stop), TRUE );
+
      /* create separator */
      sep  = gtk_separator_tool_item_new();
 
@@ -1432,9 +1349,6 @@ static GtkWidget* createToolBar(AppData *data)
      /* add items to toolbar */
      gtk_toolbar_insert(toolBar, new, -1);
      gtk_toolbar_insert(toolBar, open, -1);
-     /*
-     gtk_toolbar_insert(toolBar, save, -1);
-     */
      gtk_toolbar_insert(toolBar, saveas, -1);
      gtk_toolbar_insert(toolBar, sep,  -1);
      gtk_toolbar_insert(toolBar, rec, -1);
@@ -1448,11 +1362,6 @@ static GtkWidget* createToolBar(AppData *data)
      g_signal_connect(G_OBJECT(open), "clicked",
               G_CALLBACK(cbOpen), 
               data);
-     /*
-     g_signal_connect(G_OBJECT(save), "clicked",
-              G_CALLBACK(cbSave), 
-              data);
-     */
      g_signal_connect(G_OBJECT(saveas), "clicked",
               G_CALLBACK(cbSaveAs), 
               data);
@@ -1511,111 +1420,49 @@ static void cbItemClose(GtkWidget *widget, gpointer data)
 /* Create the menu items needed for the main view */
 static void createMenu( AppData *data )
 {
-    /* Create needed variables */
-    GSList *group = NULL;
-    GtkMenu *main_menu;
-    GtkWidget *menu_file;
-    GtkWidget *menu_others;
-    GtkWidget *item_file;
-    GtkWidget *item_file_open;
-    GtkWidget *item_file_save_as;
-    GtkWidget *item_others;
-    GtkWidget *item_pcma;
-    GtkWidget *item_pcmu;
-    GtkWidget *item_ilbc;
-    GtkWidget *item_pcm;
-    GtkWidget *item_settings;
-    /*
-    GtkWidget *item_radio_type1;
-    */
-    GtkWidget *item_close;
-    GtkWidget *item_separator;
+    HildonAppMenu *menu;
+    GtkWidget *button_email;
+
+    menu = HILDON_APP_MENU( hildon_app_menu_new() );
+    button_email = hildon_gtk_button_new(HILDON_SIZE_AUTO);
+    gtk_button_set_label( GTK_BUTTON(button_email), "Send via email");
+    hildon_app_menu_append( menu, GTK_BUTTON(button_email));
+   
+    g_signal_connect( G_OBJECT( button_email ), "clicked",
+        GTK_SIGNAL_FUNC (cbEmailing), data);
 
-    /* Get the menu from view */
-    main_menu = GTK_MENU(gtk_menu_new());
-    hildon_window_set_menu(data->mainView, main_menu);
-    
-    /* Create new submenu for "Others" */
-    menu_file = gtk_menu_new ();
-    menu_others = gtk_menu_new ();
-
-    /* Create menu items */
-    item_file = gtk_menu_item_new_with_label (_("File"));
-    item_file_open = gtk_menu_item_new_with_label(_("Open..."));
-    item_file_save_as = gtk_menu_item_new_with_label(_("Save as..."));
-    item_others = gtk_menu_item_new_with_label (_("Recording format"));
-    item_settings = gtk_menu_item_new_with_label (_("Settings"));
-    
-    item_pcma = gtk_radio_menu_item_new_with_label(
-        group, FORMAT_NAME_PCMA);
-    item_ilbc = gtk_radio_menu_item_new_with_label_from_widget(
-        GTK_RADIO_MENU_ITEM(item_pcma), FORMAT_NAME_ILBC);
-    item_pcmu = gtk_radio_menu_item_new_with_label_from_widget(
-        GTK_RADIO_MENU_ITEM(item_pcma), FORMAT_NAME_PCMU);
-    item_pcm = gtk_radio_menu_item_new_with_label_from_widget(
-        GTK_RADIO_MENU_ITEM(item_pcma), FORMAT_NAME_WAV);
     
+    gtk_widget_show_all( GTK_WIDGET(menu));
+
+    hildon_window_set_app_menu(HILDON_WINDOW(data->mainView), menu);
+
     data->filter = get_default_filter();
 
-    if (data->filter == FORMAT_ILBC)
-        gtk_check_menu_item_set_active(GTK_CHECK_MENU_ITEM(item_ilbc), TRUE);
-    else if (data->filter == FORMAT_WAV)
-        gtk_check_menu_item_set_active(GTK_CHECK_MENU_ITEM(item_pcm), TRUE);
-    else if (data->filter == FORMAT_PCMA)
-        gtk_check_menu_item_set_active(GTK_CHECK_MENU_ITEM(item_pcma), TRUE);
-    else 
-    {
-        gtk_check_menu_item_set_active(GTK_CHECK_MENU_ITEM(item_ilbc), TRUE);
-        data->filter = FORMAT_ILBC;
-    }
-    
-    data->radio_pcma = item_pcma;
-    data->radio_ilbc = item_ilbc;
-    data->radio_pcm = item_pcm;        
-    /*
-    data->radiotype = item_radio_type1;
-    */
-    item_close = gtk_menu_item_new_with_label(_("Close"));
-    item_separator = gtk_separator_menu_item_new();
-    
-    /* Add menu items to right menus */
-    gtk_menu_append( main_menu, item_file );
-    gtk_menu_append( menu_file, item_file_open );
-    gtk_menu_append( menu_file, item_file_save_as );
-    gtk_menu_append( main_menu, item_others );
-    gtk_menu_append( menu_others, item_pcm );
-    gtk_menu_append( menu_others, item_pcma );
-    gtk_menu_append( menu_others, item_ilbc);
-    
-    gtk_menu_append( main_menu, item_settings );
-    
-    gtk_menu_append( main_menu, item_close );
-
-    /* Add others submenu to the "Others" item */
-    gtk_menu_item_set_submenu(
-        GTK_MENU_ITEM(item_file), menu_file );
-    gtk_menu_item_set_submenu(
-        GTK_MENU_ITEM(item_others), menu_others );
-
-    /* Attach the callback functions to the activate signal */
-    g_signal_connect( G_OBJECT( item_file_open), "activate",
-        GTK_SIGNAL_FUNC (cbOpen), data);
-    g_signal_connect( G_OBJECT( item_file_save_as), "activate",
-        GTK_SIGNAL_FUNC (cbSaveAs), data);
-    g_signal_connect( G_OBJECT( item_settings ), "activate",
-        GTK_SIGNAL_FUNC (cbSettings), data);
-    
-    g_signal_connect( G_OBJECT( item_close ), "activate",
-        GTK_SIGNAL_FUNC (cbItemClose), data);
+}
 
-    g_signal_connect_swapped(G_OBJECT(item_pcma), "activate", G_CALLBACK(cbItemGroupChanged), data);
-    g_signal_connect_swapped(G_OBJECT(item_pcm), "activate", G_CALLBACK(cbItemGroupChanged), data);
-    
-    /* Make all menu widgets visible */
+gboolean
+evKeypress(GtkWidget *widget, GdkEventKey *ev, AppData *appdata)
+{
 
-    gtk_widget_show_all( GTK_WIDGET( main_menu ) );
+  switch (ev->keyval)
+  {
+    case GDK_Return:
+      cbRec(widget, appdata);
+      return TRUE;
+    case GDK_Right:
+      cbPlay(widget, appdata);
+      return TRUE;
+    case GDK_Escape:
+      cbStop(widget, appdata);
+      return TRUE;
+    default:
+      break;
+  }
+
+  return FALSE;
 }
 
+
 gboolean maemo_recorder_ui_new(AppData *data)
 {
     HildonProgram *app = NULL;
@@ -1627,6 +1474,7 @@ gboolean maemo_recorder_ui_new(AppData *data)
     GtkWidget *entry2 = NULL;
     GtkWidget *entry3 = NULL;
     GtkWidget *toolBar = NULL;
+    GtkWidget *infohbox = NULL;
     GtkWidget *table = NULL;
     GtkWidget *scale = NULL;
     GtkObject *adjustment = NULL;
@@ -1653,30 +1501,27 @@ gboolean maemo_recorder_ui_new(AppData *data)
     toolBar = createToolBar(data);
 
     /* create table for labels */
-    table = gtk_table_new (4, 3, FALSE);
+    table = gtk_table_new (4, 2, FALSE);
     gtk_table_set_homogeneous(GTK_TABLE(table), FALSE);
 
     gtk_table_set_row_spacings (GTK_TABLE (table), 4);
-    gtk_table_set_col_spacings (GTK_TABLE (table), 0);
+    gtk_table_set_col_spacings (GTK_TABLE (table), HILDON_MARGIN_TRIPLE);
 
     label = gtk_label_new_with_mnemonic(_("Filename:"));
-    /*
     gtk_misc_set_alignment(GTK_MISC(label), 1.0, 0.5);
-    */
     gtk_table_attach_defaults (GTK_TABLE (table),
             label,
             0, 1, 0, 1);
 
-    entry1 = gtk_entry_new ();
+    entry1 = gtk_entry_new();
     gtk_entry_set_has_frame(GTK_ENTRY(entry1), FALSE);
-    gtk_entry_set_text (GTK_ENTRY (entry1), _(RECORDER_FILE_UNTITLED));
-    gtk_table_attach_defaults (GTK_TABLE (table), entry1, 1, 3, 0, 1);
-    gtk_label_set_mnemonic_widget (GTK_LABEL (label), entry1);
+    gtk_entry_set_text(GTK_ENTRY (entry1), _(RECORDER_FILE_UNTITLED));
+    gtk_entry_set_editable(GTK_ENTRY(entry1), FALSE);
+    gtk_table_attach_defaults(GTK_TABLE (table), entry1, 1, 2, 0, 1);
+    gtk_label_set_mnemonic_widget(GTK_LABEL (label), entry1);
 
     label = gtk_label_new_with_mnemonic (_("Length:"));
-    /*
     gtk_misc_set_alignment(GTK_MISC(label), 1.0, 0.5);
-    */
     gtk_table_attach_defaults (GTK_TABLE (table),
             label,
             0, 1, 1, 2);
@@ -1684,22 +1529,39 @@ gboolean maemo_recorder_ui_new(AppData *data)
     entry2 = gtk_entry_new ();
     gtk_entry_set_has_frame(GTK_ENTRY(entry2), FALSE);
     gtk_entry_set_text (GTK_ENTRY (entry2), "0:00.00");
-    gtk_table_attach_defaults (GTK_TABLE (table), entry2, 1, 3, 1, 2);
+    gtk_entry_set_editable(GTK_ENTRY(entry2), FALSE);
+    gtk_table_attach_defaults (GTK_TABLE (table), entry2, 1, 2, 1, 2);
     gtk_label_set_mnemonic_widget (GTK_LABEL (label), entry2);
 
-    label = gtk_label_new_with_mnemonic (_("State:"));
-    /*
+    /* audio format field */
+    label = gtk_label_new_with_mnemonic(_("Format:"));
     gtk_misc_set_alignment(GTK_MISC(label), 1.0, 0.5);
-    */
     gtk_table_attach_defaults (GTK_TABLE (table),
             label,
             0, 1, 2, 3);
 
+    entry3 = gtk_entry_new();
+    gtk_entry_set_has_frame(GTK_ENTRY(entry3), FALSE);
+    gtk_entry_set_width_chars(GTK_ENTRY(entry3), 40);
+    gtk_entry_set_text (GTK_ENTRY (entry3), RECORDER_FMT_STRING_NONE);
+    gtk_entry_set_editable(GTK_ENTRY(entry3), FALSE);
+    data->mainViewData.formatEntry = GTK_WIDGET(entry3);
+    
+    gtk_table_attach_defaults (GTK_TABLE (table), entry3, 1, 2, 2, 3);
+    gtk_label_set_mnemonic_widget (GTK_LABEL (label), entry3);
+
+    label = gtk_label_new_with_mnemonic(_("State:"));
+    gtk_misc_set_alignment(GTK_MISC(label), 1.0, 0.5);
+    gtk_table_attach_defaults (GTK_TABLE (table),
+            label,
+            0, 1, 3, 4);
+
     entry3 = gtk_entry_new ();
     gtk_entry_set_has_frame(GTK_ENTRY(entry3), FALSE);
     gtk_entry_set_text (GTK_ENTRY (entry3), RECORDER_MSG_READY);
-    gtk_table_attach_defaults (GTK_TABLE (table), entry3, 1, 3, 2, 3);
-    gtk_label_set_mnemonic_widget (GTK_LABEL (label), entry3);
+    gtk_entry_set_editable(GTK_ENTRY(entry3), FALSE);
+    gtk_table_attach_defaults (GTK_TABLE (table), entry3, 1, 2, 3, 4);
+    gtk_label_set_mnemonic_widget(GTK_LABEL (label), entry3);
 
     adjustment = gtk_adjustment_new (0.00,
                   0.00,
@@ -1718,20 +1580,19 @@ gboolean maemo_recorder_ui_new(AppData *data)
     g_signal_connect(G_OBJECT(adjustment), "value-changed", G_CALLBACK(cbUserSeek), data);
     g_signal_connect(G_OBJECT(scale), "format-value", G_CALLBACK(cbFormatSeekbarValue), data);
     g_signal_connect(G_OBJECT(window), "delete_event", G_CALLBACK(cbDestroy), data);
+    g_signal_connect(G_OBJECT(window), "key-press-event",
+            G_CALLBACK(evKeypress), data);
 
     /* packing the view */
     gtk_container_add (GTK_CONTAINER(window), vbox);
-    gtk_box_pack_start (GTK_BOX(vbox), table, FALSE, TRUE, 0);
+    infohbox = gtk_hbox_new(FALSE, 0);
+    gtk_box_pack_start (GTK_BOX(infohbox), table, FALSE, TRUE, 0);
+    gtk_box_pack_start (GTK_BOX(vbox), infohbox, FALSE, TRUE, 0);
     gtk_box_pack_start (GTK_BOX(vbox), scale, FALSE, FALSE, 0);
 /*    gtk_box_pack_start (GTK_BOX(vbox), hbox, TRUE, TRUE, 0); */
     
     hildon_window_add_toolbar(window, GTK_TOOLBAR(toolBar)); 
 
-    /* initialise the ui */
-    gtk_entry_set_editable(GTK_ENTRY(entry1), FALSE);
-    gtk_entry_set_editable(GTK_ENTRY(entry2), FALSE);
-    gtk_entry_set_editable(GTK_ENTRY(entry3), FALSE);
-
     /* store needed widgets */
     data->app = app;
     data->mainView = window;
@@ -1811,6 +1672,52 @@ static gboolean seekToZero(AppData *data, GstElement *pipeline)
     return TRUE;
 }
 
+static void
+setFormatString(AppData *data, AudioFormat afmt)
+{
+    gchar *str;
+    gchar *format;
+
+    /* these are pretty much always the same */
+    gint channels = 1; 
+    gint rate = DEFAULT_RATE; /* 8000 */
+    gint bits = 8;
+
+    g_assert(data);
+    g_assert(GTK_IS_ENTRY(data->mainViewData.formatEntry));
+
+    switch (afmt)
+    {
+        case FORMAT_PCMA:
+            format = FORMAT_NAME_PCMA;
+            break;
+        case FORMAT_PCMU:
+            format = FORMAT_NAME_PCMU;
+            break;
+        case FORMAT_ILBC:
+            format = FORMAT_NAME_ILBC;
+            rate = ILBC_RATE;
+            break;
+        /* TODO: we can play wavs with many sampling rates, 2 channels */
+        /* we really should migrate to the better format spec */
+        case FORMAT_WAV:
+            format = FORMAT_NAME_WAV;
+            bits = PCM_WIDTH;
+            break;
+        case FORMAT_PCM:
+            format = FORMAT_NAME_PCM;
+            bits = PCM_WIDTH;
+            break;
+        default:
+            gtk_entry_set_text(GTK_ENTRY(data->mainViewData.formatEntry), RECORDER_FMT_STRING_NONE);
+            return;
+    }
+
+    str = g_strdup_printf("%s, %d %s, %d Hz, %d %s", format, channels, _("ch"), rate, bits, _("bits"));
+    gtk_entry_set_text(GTK_ENTRY(data->mainViewData.formatEntry), str);
+    g_free(str);
+}
+
 static void setLength(AppData *data, gdouble secs)
 {
     guint mins = 0;
@@ -1862,7 +1769,7 @@ static gdouble guessMediaLength(AppData *data)
     switch (data->file_format)
     {
         case FORMAT_ILBC:
-            bitrate = ILBC_BITRATE_20;
+            bitrate = ILBC_BITRATE_30;
             break; 
             
         case FORMAT_PCMA:
@@ -1892,7 +1799,7 @@ static GstCaps *createCapsFilter(AudioFormat format)
                 GST_TYPE_ILBC,
                 "rate", G_TYPE_INT, ILBC_RATE,
                 "channels", G_TYPE_INT, DEFAULT_CHANNELS,
-                "mode", G_TYPE_INT, 20, /* 20 ms frames */
+                "mode", G_TYPE_INT, 30, /* 30 ms frames */
                 NULL);
         case FORMAT_PCMA:
             return gst_caps_new_simple(
@@ -1929,15 +1836,7 @@ static gboolean cbStopPlayback(AppData *data)
     ULOG_INFO("Stopping playback");
    
     g_assert(data != NULL);
-
-    ret = gst_element_set_state(GST_ELEMENT(data->playPipeline), 
-                        GST_STATE_PAUSED);
-    if (seekToZero(data, GST_ELEMENT(data->playPipeline)))
-    {
-        gtk_adjustment_set_value( 
-           GTK_ADJUSTMENT(data->mainViewData.adjustment), 0);
-        gtk_adjustment_value_changed(GTK_ADJUSTMENT(data->mainViewData.adjustment));
-    }
+    destroyPipelines(data);
     setAppState(data, APPSTATE_READY);
     gtk_tool_button_set_stock_id(GTK_TOOL_BUTTON(data->buttonPlay), GTK_STOCK_MEDIA_PLAY);
     gtk_widget_set_state(data->buttonPlay, GTK_STATE_NORMAL);
@@ -2008,3 +1907,4 @@ static gboolean cbUpdateRecLength(AppData *data)
     return FALSE;
 }
 
+