aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorHolger Macht <holger@homac.de>2009-04-28 23:10:26 +0800
committerMatthew Barnes <mbarnes@redhat.com>2009-05-01 03:15:42 +0800
commitd7e7b7ddd4e86a34ffbf240fdc887b10d363dd12 (patch)
tree1422fbefb266453658f71fab1fae5b5934f89c22
parentfd94ba8cde63d761fc84a3724874529caa266f09 (diff)
downloadgsoc2013-evolution-d7e7b7ddd4e86a34ffbf240fdc887b10d363dd12.tar.gz
gsoc2013-evolution-d7e7b7ddd4e86a34ffbf240fdc887b10d363dd12.tar.zst
gsoc2013-evolution-d7e7b7ddd4e86a34ffbf240fdc887b10d363dd12.zip
External Editor rewrite by Holger Macht. Fixes #567145
-rw-r--r--composer/evolution-composer.ui3
-rw-r--r--plugins/external-editor/ChangeLog13
-rw-r--r--plugins/external-editor/Makefile.am4
-rw-r--r--plugins/external-editor/apps-evolution-external-editor.schemas.in10
-rw-r--r--plugins/external-editor/external-editor.c389
-rw-r--r--plugins/external-editor/org-gnome-external-editor.eplug.xml40
-rw-r--r--plugins/external-editor/org-gnome-external-editor.error.xml7
7 files changed, 303 insertions, 163 deletions
diff --git a/composer/evolution-composer.ui b/composer/evolution-composer.ui
index 8c422c06f4..6522679398 100644
--- a/composer/evolution-composer.ui
+++ b/composer/evolution-composer.ui
@@ -11,6 +11,9 @@
<menuitem action='save-draft'/>
<placeholder name='template-holder'/>
<separator/>
+ <placeholder name='external-editor-holder'/>
+ <separator/>
+ <separator/>
<menuitem action='print-preview'/>
<menuitem action='print'/>
<separator/>
diff --git a/plugins/external-editor/ChangeLog b/plugins/external-editor/ChangeLog
index dae79fc9a7..55ead0195b 100644
--- a/plugins/external-editor/ChangeLog
+++ b/plugins/external-editor/ChangeLog
@@ -1,3 +1,16 @@
+2009-04-28 Holger Macht <holger@homac.de>
+
+ ** Fix for bug #567145
+
+ External Editor rewrite.
+ * composer/evolution-composer.ui:
+ * plugins/external-editor/Makefile.am:
+ * plugins/external-editor/apps-evolution-external-editor.schemas.in
+ :
+ * plugins/external-editor/external-editor.c:
+ * plugins/external-editor/org-gnome-external-editor.eplug.xml:
+ * plugins/external-editor/org-gnome-external-editor.error.xml:
+
2009-04-27 Milan Crha <mcrha@redhat.com>
** Fix for bug #561188
diff --git a/plugins/external-editor/Makefile.am b/plugins/external-editor/Makefile.am
index ff64941a28..36cba20344 100644
--- a/plugins/external-editor/Makefile.am
+++ b/plugins/external-editor/Makefile.am
@@ -25,8 +25,7 @@ error_DATA = org-gnome-external-editor.error
errordir = $(privdatadir)/errors
plugin_DATA = \
- org-gnome-external-editor.eplug \
- org-gnome-external-editor.xml
+ org-gnome-external-editor.eplug
plugin_LTLIBRARIES = liborg-gnome-external-editor.la
@@ -54,7 +53,6 @@ install-data-local:
EXTRA_DIST = \
org-gnome-external-editor.eplug.xml \
org-gnome-external-editor.error.xml \
- org-gnome-external-editor.xml \
$(schema_in_files)
BUILT_SOURCES = org-gnome-external-editor.eplug \
diff --git a/plugins/external-editor/apps-evolution-external-editor.schemas.in b/plugins/external-editor/apps-evolution-external-editor.schemas.in
index 0f6d264b90..38becebdb7 100644
--- a/plugins/external-editor/apps-evolution-external-editor.schemas.in
+++ b/plugins/external-editor/apps-evolution-external-editor.schemas.in
@@ -10,6 +10,16 @@
<short>Default External Editor</short>
<long>The default command that must be used as the editor.</long>
</locale>
+
+ <key>/schemas/apps/evolution/eplugin/external-editor/launch-on-key-press</key>
+ <applyto>/apps/evolution/eplugin/external-editor/launch-on-key-press</applyto>
+ <owner>evolution-mail</owner>
+ <type>bool</type>
+ <default>false</default>
+ <locale name="C">
+ <short>Automatically launch when a new mail is edited</short>
+ <long>Automatically launch editor when key is pressed in the mail composer</long>
+ </locale>
</schema>
</schemalist>
</gconfschemafile>
diff --git a/plugins/external-editor/external-editor.c b/plugins/external-editor/external-editor.c
index a2c86af392..a07bb6e80e 100644
--- a/plugins/external-editor/external-editor.c
+++ b/plugins/external-editor/external-editor.c
@@ -11,11 +11,12 @@
* Lesser General Public License for more details.
*
* You should have received a copy of the GNU Lesser General Public
- * License along with the program; if not, see <http://www.gnu.org/licenses/>
+ * License along with the program; if not, see <http://www.gnu.org/licenses/>
*
*
* Authors:
- * Sankar P <psankar@novell.com>
+ * Holger Macht <hmacht@suse.de>
+ * based on work by Sankar P <psankar@novell.com>
*
* Copyright (C) 1999-2008 Novell, Inc. (www.novell.com)
*
@@ -31,11 +32,13 @@
#include <mail/mail-config.h>
#include <e-util/e-error.h>
#include <e-msg-composer.h>
+#include <camel/camel-mime-filter-tohtml.h>
#include <glib/gi18n-lib.h>
#include <glib-object.h>
#include <glib.h>
#include <glib/gstdio.h>
+#include <gdk/gdkkeysyms.h>
#include <sys/stat.h>
#ifdef HAVE_SYS_WAIT_H
@@ -47,56 +50,48 @@
#include <gconf/gconf-client.h>
-#define d(x)
+#define d(x)
-#define EDITOR_GCONF_KEY "/apps/evolution/eplugin/external-editor/editor-command"
+#define EDITOR_GCONF_KEY_COMMAND "/apps/evolution/eplugin/external-editor/editor-command"
+#define EDITOR_GCONF_KEY_IMMEDIATE "/apps/evolution/eplugin/external-editor/launch-on-key-press"
-void org_gnome_external_editor (EPlugin *ep, EMMenuTargetSelect *select);
-void ee_editor_command_changed (GtkWidget *textbox);
+gboolean e_plugin_ui_init (GtkUIManager *manager, EMsgComposer *composer);
GtkWidget * e_plugin_lib_get_configure_widget (EPlugin *epl);
+static void ee_editor_command_changed (GtkWidget *textbox);
+static void ee_editor_immediate_launch_changed (GtkWidget *checkbox);
+static void async_external_editor (EMsgComposer *composer);
+static gboolean editor_running (void);
+static gboolean key_press_cb(GtkWidget * widget, GdkEventKey * event, EMsgComposer *composer);
-/* Utility function to convert an email address to CamelInternetAddress.
-May be this should belong to CamelInternetAddress.h file itself. */
-static CamelInternetAddress * convert_to_camel_internet_address (char * emails)
-{
- CamelInternetAddress *cia = camel_internet_address_new();
- gchar **address_tokens = NULL;
- int i;
-
- d(printf ("\n\aconvert called with : [%s] \n\a", emails));
-
- emails = g_strstrip (emails);
-
- if (emails && strlen (emails) > 1) {
- address_tokens = g_strsplit (emails, ",", 0);
-
- if (address_tokens) {
- for (i = 0; address_tokens[i]; ++i) {
- camel_internet_address_add (cia, " ", address_tokens [i]);
- d(printf ("\nAdding camel_internet_address[%s] \n", address_tokens [i]));
- }
- g_strfreev (address_tokens);
-
- g_free (emails);
- return cia;
- }
- }
- camel_object_unref (cia);
- g_free (emails);
- return NULL;
-}
+/* used to track when the external editor is active */
+static GThread *editor_thread;
-void ee_editor_command_changed (GtkWidget *textbox)
+void
+ee_editor_command_changed (GtkWidget *textbox)
{
const char *editor;
GConfClient *gconf;
editor = gtk_entry_get_text (GTK_ENTRY(textbox));
d(printf ("\n\aeditor is : [%s] \n\a", editor));
-
+
/* gconf access for every key-press. Sucky ? */
gconf = gconf_client_get_default ();
- gconf_client_set_string (gconf, EDITOR_GCONF_KEY, editor, NULL);
+ gconf_client_set_string (gconf, EDITOR_GCONF_KEY_COMMAND, editor, NULL);
+ g_object_unref (gconf);
+}
+
+void
+ee_editor_immediate_launch_changed (GtkWidget *checkbox)
+{
+ gboolean immediately;
+ GConfClient *gconf;
+
+ immediately = gtk_toggle_button_get_active (GTK_TOGGLE_BUTTON (checkbox));
+ d(printf ("\n\aimmediate launch is : [%d] \n\a", immediately));
+
+ gconf = gconf_client_get_default ();
+ gconf_client_set_bool (gconf, EDITOR_GCONF_KEY_IMMEDIATE, immediately, NULL);
g_object_unref (gconf);
}
@@ -104,179 +99,297 @@ GtkWidget *
e_plugin_lib_get_configure_widget (EPlugin *epl)
{
GtkWidget *vbox, *textbox, *label, *help;
+ GtkWidget *checkbox;
GConfClient *gconf;
char *editor;
+ gboolean checked;
vbox = gtk_vbox_new (FALSE, 10);
textbox = gtk_entry_new ();
label = gtk_label_new (_("Command to be executed to launch the editor: "));
- help = gtk_label_new (_("For Emacs use \"xemacs\"\nFor VI use \"gvim\""));
+ help = gtk_label_new (_("For Emacs use \"xemacs\"\nFor VI use \"gvim -f\""));
gconf = gconf_client_get_default ();
- editor = gconf_client_get_string (gconf, EDITOR_GCONF_KEY, NULL);
+ editor = gconf_client_get_string (gconf, EDITOR_GCONF_KEY_COMMAND, NULL);
if (editor) {
gtk_entry_set_text (GTK_ENTRY(textbox), editor);
g_free (editor);
}
+
+ checkbox = gtk_check_button_new_with_label (
+ _("Automatically launch when a new mail is edited"));
+ checked = gconf_client_get_bool (gconf, EDITOR_GCONF_KEY_IMMEDIATE, NULL);
+ if (checked)
+ gtk_toggle_button_set_active (GTK_TOGGLE_BUTTON (checkbox), TRUE);
g_object_unref (gconf);
gtk_box_pack_start (GTK_BOX (vbox), label, FALSE, FALSE, 0);
gtk_box_pack_start (GTK_BOX (vbox), textbox, FALSE, FALSE, 0);
gtk_box_pack_start (GTK_BOX (vbox), help, FALSE, FALSE, 0);
+ gtk_box_pack_start (GTK_BOX (vbox), checkbox, FALSE, FALSE, 0);
g_signal_connect (textbox, "changed", G_CALLBACK(ee_editor_command_changed), textbox);
-
+
+ g_signal_connect (checkbox, "toggled",
+ G_CALLBACK(ee_editor_immediate_launch_changed), checkbox);
+
gtk_widget_show_all (vbox);
return vbox;
}
-static gboolean
-show_error (const char *id)
-{
- if (id)
- e_error_run (NULL, id, NULL);
- return FALSE;
-}
-
-static gboolean
-read_file (char *filename)
+static void
+enable_disable_composer (EMsgComposer *composer, gboolean enable)
{
- gchar *buf;
- CamelMimeMessage *message;
- EMsgComposer *composer;
-
- message = camel_mime_message_new ();
+ GtkhtmlEditor *editor;
+ GtkAction *action;
+ GtkActionGroup *action_group;
- if (filename && g_file_get_contents (filename, &buf, NULL, NULL)) {
- gchar **tokens;
- int i, j;
+ g_return_if_fail (E_IS_MSG_COMPOSER (composer));
- tokens = g_strsplit (buf, "###|||", 6);
+ editor = GTKHTML_EDITOR (composer);
- for (i = 1; tokens[i]; ++i) {
+ if (enable)
+ gtkhtml_editor_run_command (editor, "editable-on");
+ else
+ gtkhtml_editor_run_command (editor, "editable-off");
- for (j = 0; tokens[i][j] && tokens[i][j] != '\n'; ++j) {
- tokens [i][j] = ' ';
- }
+ action = GTKHTML_EDITOR_ACTION_EDIT_MENU (composer);
+ gtk_action_set_sensitive (action, enable);
- if (tokens[i][j] == '\n')
- tokens[i][j] = ' ';
+ action = GTKHTML_EDITOR_ACTION_FORMAT_MENU (composer);
+ gtk_action_set_sensitive (action, enable);
- d(printf ("\nstripped off token[%d] is : %s \n", i, tokens[i]));
- }
+ action = GTKHTML_EDITOR_ACTION_INSERT_MENU (composer);
+ gtk_action_set_sensitive (action, enable);
- camel_mime_message_set_recipients (message, "To", convert_to_camel_internet_address(g_strchug(g_strdup(tokens[1]))));
- camel_mime_message_set_recipients (message, "Cc", convert_to_camel_internet_address(g_strchug(g_strdup(tokens[2]))));
- camel_mime_message_set_recipients (message, "Bcc", convert_to_camel_internet_address(g_strchug(g_strdup(tokens[3]))));
- camel_mime_message_set_subject (message, tokens[4]);
- camel_mime_part_set_content ((CamelMimePart *)message, tokens [5], strlen (tokens [5]), "text/plain");
+ action_group = gtkhtml_editor_get_action_group (editor, "composer");
+ gtk_action_group_set_sensitive (action_group, enable);
+}
- /* FIXME: We need to make mail-remote working properly.
- So that we neednot invoke composer widget at all.
+static void
+enable_composer (EMsgComposer *composer)
+{
+ enable_disable_composer (composer, TRUE);
+}
- May be we can do it now itself by invoking local CamelTransport.
- But all that is not needed for the first release.
+static void
+disable_composer (EMsgComposer *composer)
+{
+ enable_disable_composer (composer, FALSE);
+}
- People might want to format mails using their editor (80 cols width etc.)
- But might want to use evolution addressbook for auto-completion etc.
- So starting the composer window anyway.
- */
+/* needed because the new thread needs to call g_idle_add () */
+static gboolean
+update_composer_text (GArray *array)
+{
+ EMsgComposer *composer;
+ gchar *text;
- composer = e_msg_composer_new_with_message (message);
+ composer = g_array_index (array, gpointer, 0);
+ text = g_array_index (array, gpointer, 1);
- gtk_widget_show (GTK_WIDGET (composer));
+ e_msg_composer_set_body_text (composer, text, -1);
- g_strfreev (tokens);
+ enable_composer (composer);
- /* We no longer need that temporary file */
- g_remove (filename);
- }
+ g_free (text);
- g_free (filename);
+ return FALSE;
+}
+/* needed because the new thread needs to call g_idle_add () */
+static gboolean
+run_error_dialog (gchar *text)
+{
+ e_error_run (NULL, text, NULL);
return FALSE;
}
-static void
-async_external_editor (GArray *array)
+
+void
+async_external_editor (EMsgComposer *composer)
{
char *filename = NULL;
- gchar *argv[5];
int status = 0;
+ GConfClient *gconf;
+ gchar *editor_cmd_line = NULL;
+ gint fd;
- argv[0] = g_array_index (array, gpointer, 0);
- argv[1] = g_array_index (array, gpointer, 1);
- argv[2] = NULL;
+ fd = g_file_open_tmp (NULL, &filename, NULL);
+ if (fd > 0) {
+ close (fd);
+ /* Push the text (if there is one) from the composer to the file */
+ g_file_set_contents (filename, gtkhtml_editor_get_text_plain(
+ GTKHTML_EDITOR(composer), NULL),
+ -1, NULL);
+ d(printf ("\n\aTemporary-file Name is : [%s] \n\a", filename));
+ } else {
+ g_warning ("Temporary file fd is null");
+ g_idle_add ((GSourceFunc) run_error_dialog,
+ "org.gnome.evolution.plugins.external-editor:no-temp-file");
+ g_idle_add ((GSourceFunc) enable_composer, composer);
+ return ;
+ }
+
+ gconf = gconf_client_get_default ();
+ editor_cmd_line = gconf_client_get_string (gconf, EDITOR_GCONF_KEY_COMMAND, NULL);
+ if (!editor_cmd_line) {
+ if (! (editor_cmd_line = g_strdup(g_getenv ("EDITOR"))) )
+ /* Make gedit the default external editor,
+ if the default schemas are not installed
+ and no $EDITOR is set. */
+ editor_cmd_line = g_strdup("gedit");
+ }
+ g_object_unref (gconf);
- filename = g_strdup (argv[1]);
+ editor_cmd_line = g_strconcat(editor_cmd_line, " ", filename, NULL);
- if (!g_spawn_sync (NULL, argv, NULL, G_SPAWN_SEARCH_PATH, NULL, NULL, NULL, NULL, &status, NULL))
+ if (!g_spawn_command_line_sync(editor_cmd_line, NULL, NULL, &status, NULL))
{
- g_warning ("Unable to launch %s: ", argv[0]);
- g_idle_add ((GSourceFunc)show_error, "org.gnome.evolution.plugins.external-editor:editor-not-launchable");
+ g_warning ("Unable to launch %s: ", editor_cmd_line);
+ g_idle_add ((GSourceFunc) run_error_dialog,
+ "org.gnome.evolution.plugins.external-editor:editor-not-launchable");
+ g_idle_add ((GSourceFunc) enable_composer, composer);
+
g_free (filename);
- return;
+ g_free (editor_cmd_line);
+ return ;
}
-
+ g_free (editor_cmd_line);
+
#ifdef HAVE_SYS_WAIT_H
if (WEXITSTATUS (status) != 0) {
#else
if (status) {
#endif
d(printf ("\n\nsome problem here with external editor\n\n"));
- g_free (filename);
- return;
+ g_idle_add ((GSourceFunc) enable_composer, composer);
+ return ;
} else {
- g_idle_add ((GSourceFunc)read_file, filename);
+ gchar *buf;
+
+ if (g_file_get_contents (filename, &buf, NULL, NULL)) {
+ gchar *htmltext;
+ GArray *array;
+
+ htmltext = camel_text_to_html(buf, CAMEL_MIME_FILTER_TOHTML_PRE, 0);
+
+ array = g_array_sized_new (TRUE, TRUE,
+ sizeof (gpointer), 2 * sizeof(gpointer));
+ array = g_array_append_val (array, composer);
+ array = g_array_append_val (array, htmltext);
+
+ g_idle_add ((GSourceFunc) update_composer_text, array);
+
+ /* We no longer need that temporary file */
+ g_remove (filename);
+ g_free (filename);
+ }
}
}
-void org_gnome_external_editor (EPlugin *ep, EMMenuTargetSelect *select)
+static void launch_editor (GtkAction *action, EMsgComposer *composer)
{
- /* The template to be used in the external editor */
+ d(printf ("\n\nexternal_editor plugin is launched \n\n"));
- /* README: I have not marked this for translation.
- As I might change this string to make it more meaningful and friendlier based on feedback. */
+ if (editor_running()) {
+ d(printf("not opening editor, because it's still running\n"));
+ return ;
+ }
- char template[] = "###|||Insert , seperated TO addresses below this line. Do not edit or delete this line. Optional field\n\n###||| Insert , seperated CC addresses below this line. Do not edit or delete this line. Optional field\n\n###|||Insert , seperated BCC addresses below this line. Do not edit or delete this line. Optional field\n\n###|||Insert SUBJECT below this line. Do not edit or delete this line. Optional field\n\n###|||Insert BODY of mail below this line. Do not edit or delete this line.\n\n";
+ disable_composer (composer);
- gint fd;
- char *filename = NULL;
- char *editor = NULL;
- GConfClient *gconf;
- GArray *array;
+ editor_thread = g_thread_create ((GThreadFunc)async_external_editor, composer, FALSE, NULL);
+}
- d(printf ("\n\nexternal_editor plugin is launched \n\n"));
+static GtkActionEntry entries[] = {
+ { "ExternalEditor",
+ GTK_STOCK_EDIT,
+ N_("Compose in External Editor"),
+ "<Shift><Control>e",
+ N_("Compose in External Editor"),
+ G_CALLBACK (launch_editor) }
+};
- fd = g_file_open_tmp (NULL, &filename, NULL);
- if (fd > 0) {
- close (fd);
- /* Push the template contents to the intermediate file */
- g_file_set_contents (filename, template, strlen (template), NULL);
- d(printf ("\n\aTemporary-file Name is : [%s] \n\a", filename));
- } else {
- g_warning ("Temporary file fd is null");
- e_error_run (NULL, "org.gnome.evolution.plugins.external-editor:no-temp-file", NULL);
- return ;
+static gboolean
+key_press_cb(GtkWidget * widget, GdkEventKey * event, EMsgComposer *composer)
+{
+ GConfClient *gconf;
+ gboolean immediately;
+
+ /* we don't want to start the editor on any modifier keys */
+ switch (event->keyval) {
+ case GDK_Alt_L:
+ case GDK_Alt_R:
+ case GDK_Super_L:
+ case GDK_Super_R:
+ case GDK_Control_L:
+ case GDK_Control_R:
+ return FALSE;
+ default:
+ break;
}
gconf = gconf_client_get_default ();
- editor = gconf_client_get_string (gconf, EDITOR_GCONF_KEY, NULL);
- if (!editor) {
+ immediately = gconf_client_get_bool (gconf, EDITOR_GCONF_KEY_IMMEDIATE, NULL);
+ g_object_unref (gconf);
+ if (!immediately)
+ return FALSE;
- if (! (editor = g_strdup(g_getenv ("EDITOR"))) )
- /* Make gedit the default external editor,
- if the default schemas are not installed
- and no $EDITOR is set. */
- editor = g_strdup("gedit");
+ launch_editor (NULL, composer);
+
+ return TRUE;
+}
+
+static void
+editor_running_thread_func (GThread *thread, gpointer running)
+{
+ if (thread == editor_thread)
+ *(gboolean*)running = TRUE;
+}
+
+/* Racy? */
+static gboolean
+editor_running (void)
+{
+ gboolean running = FALSE;
+
+ g_thread_foreach ((GFunc)editor_running_thread_func, &running);
+
+ return running;
+}
+
+static gboolean
+delete_cb (GtkWidget *widget, EMsgComposer *composer)
+{
+ if (editor_running()) {
+ e_error_run (NULL, "org.gnome.evolution.plugins.external-editor:editor-still-running", NULL);
+ return TRUE;
}
- g_object_unref (gconf);
- array = g_array_sized_new (TRUE, TRUE, sizeof (gpointer), 2 * sizeof(gpointer));
- array = g_array_append_val (array, editor);
- array = g_array_append_val (array, filename);
+ return FALSE;
+}
+
+gboolean
+e_plugin_ui_init (GtkUIManager *manager, EMsgComposer *composer)
+{
+ GtkhtmlEditor *editor;
+ GtkHTML *html;
+
+ editor = GTKHTML_EDITOR (composer);
+
+ /* Add actions to the "composer" action group. */
+ gtk_action_group_add_actions (
+ gtkhtml_editor_get_action_group (editor, "composer"),
+ entries, G_N_ELEMENTS (entries), composer);
+
+ html = gtkhtml_editor_get_html (editor);
+
+ g_signal_connect (G_OBJECT(html), "key_press_event",
+ G_CALLBACK(key_press_cb), composer);
- g_thread_create ( (GThreadFunc) async_external_editor, array, FALSE, NULL);
+ g_signal_connect (G_OBJECT(composer), "delete-event",
+ G_CALLBACK(delete_cb), composer);
- return ;
+ return TRUE;
}
diff --git a/plugins/external-editor/org-gnome-external-editor.eplug.xml b/plugins/external-editor/org-gnome-external-editor.eplug.xml
index 68c56d83de..861535d6cf 100644
--- a/plugins/external-editor/org-gnome-external-editor.eplug.xml
+++ b/plugins/external-editor/org-gnome-external-editor.eplug.xml
@@ -1,27 +1,25 @@
-<?xml version="1.0"?>
+<?xml version="1.0" encoding="UTF-8"?>
<e-plugin-list>
- <!-- the path to the shared library -->
- <e-plugin
- id="org.gnome.plugin.external.editor"
- type="shlib"
- location="@PLUGINDIR@/liborg-gnome-external-editor@SOEXT@"
- _name="External Editor">
+ <e-plugin type="shlib"
+ location="@PLUGINDIR@/liborg-gnome-external-editor@SOEXT@"
+ id="org.gnome.plugin.external.editor" name="External Editor">
+ <author name="Holger Macht" email="hmacht@suse.de"/>
<author name="Sankar P" email="sankar2u@gmail.com"/>
- <_description>A plugin for using an external editor as the composer. You can send only plain-text messages.</_description>
+ <description>A plugin for using an external editor as the composer. You can send only plain-text messages.</description>
- <hook class="org.gnome.evolution.shell.bonobomenu:1.0">
-
- <menu id="org.gnome.evolution.shell" target="shell">
- <!-- the path to the bonobo menu description -->
- <ui file="@PLUGINDIR@/org-gnome-external-editor.xml"/>
- <item
- type="item"
- verb="EPExtEditor"
- path="/commands/EPExtEditor"
- activate="org_gnome_external_editor"/>
- </menu>
-
- </hook>
+ <hook class="org.gnome.evolution.ui:1.0">
+ <ui-manager id="org.gnome.evolution.composer">
+ <menubar name='main-menu'>
+ <placeholder name='pre-edit-menu'>
+ <menu action='file-menu'>
+ <placeholder name="external-editor-holder">
+ <menuitem action="ExternalEditor"/>
+ </placeholder>
+ </menu>
+ </placeholder>
+ </menubar>
+ </ui-manager>
+ </hook>
</e-plugin>
</e-plugin-list>
diff --git a/plugins/external-editor/org-gnome-external-editor.error.xml b/plugins/external-editor/org-gnome-external-editor.error.xml
index c6f0bebfa3..efe1c4408d 100644
--- a/plugins/external-editor/org-gnome-external-editor.error.xml
+++ b/plugins/external-editor/org-gnome-external-editor.error.xml
@@ -1,6 +1,6 @@
<?xml version="1.0" encoding="UTF-8"?>
<error-list domain="org.gnome.evolution.plugins.external-editor">
-
+
<error id="editor-not-launchable" type="error">
<_primary>Editor not launchable</_primary>
<_secondary>The external editor set in your plugin preferences cannot be launched. Try setting a different editor.</_secondary>
@@ -11,4 +11,9 @@
<_secondary>Evolution is unable to create a temporary file to save your mail. Retry later.</_secondary>
</error>
+ <error id="editor-still-running" type="error">
+ <_primary>External editor still running</_primary>
+ <_secondary>The external editor is still running. The mail composer window cannot be closed as long as the editor is active.</_secondary>
+ </error>
+
</error-list>