Handle iso-8859-1 keys nicely in e-text and e-table. Other encodings to

come...

svn path=/trunk/; revision=4972

5 files changed, 73 insertions, 8 deletions

diff --git a/widgets/e-table/ChangeLog b/widgets/e-table/ChangeLog
index 9ff6fbfe97..8429e2fabf 100644
--- a/widgets/e-table/ChangeLog
+++ b/widgets/e-table/ChangeLog
@@ -1,3 +1,8 @@
+2000-08-21  Lauris Kaplinski  <lauris@helixcode.com>
+
+	* e-cell-text.c (ect_event): Use e_utf8_from_gtk_event_key to translate
+	GDK_KEY_PRESS event to insertable UTF-8 string
+
 2000-08-22  Christopher James Lahey  <clahey@helixcode.com>
 
 	* e-cell-text.c, e-cell-toggle.c, e-cell-tree.c, e-table-item.c:
diff --git a/widgets/e-table/e-cell-text.c b/widgets/e-table/e-cell-text.c
index 279ef4975b..5a7f8d514e 100644
--- a/widgets/e-table/e-cell-text.c
+++ b/widgets/e-table/e-cell-text.c
@@ -35,6 +35,7 @@
 #include "e-cell-text.h"
 #include "e-util/e-util.h"
 #include "e-util/e-font.h"
+#include "e-util/e-unicode.h"
 #include "e-table-item.h"
 #include "e-text-event-processor.h"
 #include "e-text-event-processor-emacs-like.h"
@@ -912,11 +913,23 @@ ect_event (ECellView *ecell_view, GdkEvent *event, int model_col, int view_col,
 				e_tep_event.key.time = key.time;
 				e_tep_event.key.state = key.state;
 				e_tep_event.key.keyval = key.keyval;
+
+				/* This is probably ugly hack, but we have to handle UTF-8 input somehow */
+#if 0
 				e_tep_event.key.length = key.length;
 				e_tep_event.key.string = key.string;
+#else
+				e_tep_event.key.string = e_utf8_from_gtk_event_key (canvas, key.keyval, key.string);
+				if (e_tep_event.key.string != NULL) {
+					e_tep_event.key.length = strlen (e_tep_event.key.string);
+				} else {
+					e_tep_event.key.length = 0;
+				}
+#endif
+
 				_get_tep (edit);
-				return_val = e_text_event_processor_handle_event (edit->tep,
-										  &e_tep_event);
+				return_val = e_text_event_processor_handle_event (edit->tep, &e_tep_event);
+				if (e_tep_event.key.string) g_free (e_tep_event.key.string);
 				break;
 			}
 		}
diff --git a/widgets/e-text/e-text.c b/widgets/e-text/e-text.c
index dc28886c15..d30d1cb716 100644
--- a/widgets/e-text/e-text.c
+++ b/widgets/e-text/e-text.c
@@ -28,6 +28,7 @@
 #include <gtk/gtkinvisible.h>
 #include "e-util/e-canvas.h"
 #include "e-util/e-canvas-utils.h"
+#include "e-util/e-unicode.h"
 
 #include "e-text-event-processor-emacs-like.h"
 
@@ -2615,14 +2616,30 @@ e_text_event (GnomeCanvasItem *item, GdkEvent *event)
 	case GDK_KEY_RELEASE:
 		if (text->editing) {
 			GdkEventKey key = event->key;
+			gint ret;
+
 			e_tep_event.key.time = key.time;
 			e_tep_event.key.state = key.state;
 			e_tep_event.key.keyval = key.keyval;
+
+			/* This is probably ugly hack, but we have to handle UTF-8 input somehow */
+#if 0
 			e_tep_event.key.length = key.length;
 			e_tep_event.key.string = key.string;
+#else
+			e_tep_event.key.string = e_utf8_from_gtk_event_key (GTK_WIDGET (item->canvas), key.keyval, key.string);
+			if (e_tep_event.key.string != NULL) {
+				e_tep_event.key.length = strlen (e_tep_event.key.string);
+			} else {
+				e_tep_event.key.length = 0;
+			}
+#endif
 			_get_tep(text);
-			return e_text_event_processor_handle_event (text->tep,
-								    &e_tep_event);
+			ret = e_text_event_processor_handle_event (text->tep, &e_tep_event);
+
+			if (e_tep_event.key.string) g_free (e_tep_event.key.string);
+
+			return ret;
 		}
 		else
 			return 0;
diff --git a/widgets/table/e-cell-text.c b/widgets/table/e-cell-text.c
index 279ef4975b..5a7f8d514e 100644
--- a/widgets/table/e-cell-text.c
+++ b/widgets/table/e-cell-text.c
@@ -35,6 +35,7 @@
 #include "e-cell-text.h"
 #include "e-util/e-util.h"
 #include "e-util/e-font.h"
+#include "e-util/e-unicode.h"
 #include "e-table-item.h"
 #include "e-text-event-processor.h"
 #include "e-text-event-processor-emacs-like.h"
@@ -912,11 +913,23 @@ ect_event (ECellView *ecell_view, GdkEvent *event, int model_col, int view_col,
 				e_tep_event.key.time = key.time;
 				e_tep_event.key.state = key.state;
 				e_tep_event.key.keyval = key.keyval;
+
+				/* This is probably ugly hack, but we have to handle UTF-8 input somehow */
+#if 0
 				e_tep_event.key.length = key.length;
 				e_tep_event.key.string = key.string;
+#else
+				e_tep_event.key.string = e_utf8_from_gtk_event_key (canvas, key.keyval, key.string);
+				if (e_tep_event.key.string != NULL) {
+					e_tep_event.key.length = strlen (e_tep_event.key.string);
+				} else {
+					e_tep_event.key.length = 0;
+				}
+#endif
+
 				_get_tep (edit);
-				return_val = e_text_event_processor_handle_event (edit->tep,
-										  &e_tep_event);
+				return_val = e_text_event_processor_handle_event (edit->tep, &e_tep_event);
+				if (e_tep_event.key.string) g_free (e_tep_event.key.string);
 				break;
 			}
 		}
diff --git a/widgets/text/e-text.c b/widgets/text/e-text.c
index dc28886c15..d30d1cb716 100644
--- a/widgets/text/e-text.c
+++ b/widgets/text/e-text.c
@@ -28,6 +28,7 @@
 #include <gtk/gtkinvisible.h>
 #include "e-util/e-canvas.h"
 #include "e-util/e-canvas-utils.h"
+#include "e-util/e-unicode.h"
 
 #include "e-text-event-processor-emacs-like.h"
 
@@ -2615,14 +2616,30 @@ e_text_event (GnomeCanvasItem *item, GdkEvent *event)
 	case GDK_KEY_RELEASE:
 		if (text->editing) {
 			GdkEventKey key = event->key;
+			gint ret;
+
 			e_tep_event.key.time = key.time;
 			e_tep_event.key.state = key.state;
 			e_tep_event.key.keyval = key.keyval;
+
+			/* This is probably ugly hack, but we have to handle UTF-8 input somehow */
+#if 0
 			e_tep_event.key.length = key.length;
 			e_tep_event.key.string = key.string;
+#else
+			e_tep_event.key.string = e_utf8_from_gtk_event_key (GTK_WIDGET (item->canvas), key.keyval, key.string);
+			if (e_tep_event.key.string != NULL) {
+				e_tep_event.key.length = strlen (e_tep_event.key.string);
+			} else {
+				e_tep_event.key.length = 0;
+			}
+#endif
 			_get_tep(text);
-			return e_text_event_processor_handle_event (text->tep,
-								    &e_tep_event);
+			ret = e_text_event_processor_handle_event (text->tep, &e_tep_event);
+
+			if (e_tep_event.key.string) g_free (e_tep_event.key.string);
+
+			return ret;
 		}
 		else
 			return 0;