Remove the spell checking preference as it's a talkatu feature and most are unlikely to disable it

Tue, 06 Oct 2020 20:44:33 -0500

author
Gary Kramlich <grim@reaperworld.com>
date
Tue, 06 Oct 2020 20:44:33 -0500
changeset 40542
8079731c8ef9
parent 40541
9ceb8d25d4d9
child 40543
42fd60e3fbe8

Remove the spell checking preference as it's a talkatu feature and most are unlikely to disable it

Testing Done:
Compiled, verified the preference window still works, and sent some messages via bonjour.

Reviewed at https://reviews.imfreedom.org/r/143/

pidgin/gtkconv.c file | annotate | diff | comparison | revisions
pidgin/gtkprefs.c file | annotate | diff | comparison | revisions
pidgin/resources/Prefs/prefs.ui file | annotate | diff | comparison | revisions
--- a/pidgin/gtkconv.c	Sat Oct 03 05:58:21 2020 -0500
+++ b/pidgin/gtkconv.c	Tue Oct 06 20:44:33 2020 -0500
@@ -5646,27 +5646,6 @@
 }
 
 static void
-spellcheck_pref_cb(const char *name, PurplePrefType type,
-				   gconstpointer value, gpointer data)
-{
-	GList *cl;
-	PurpleConversation *conv;
-
-	for (cl = purple_conversations_get_all(); cl != NULL; cl = cl->next) {
-		PidginConversation *gtkconv = NULL;
-
-		conv = (PurpleConversation *)cl->data;
-
-		if (!PIDGIN_IS_PIDGIN_CONVERSATION(conv))
-			continue;
-
-		gtkconv = PIDGIN_CONVERSATION(conv);
-
-# warning toggle spell checking when talkatu #60 is done.
-	}
-}
-
-static void
 tab_side_pref_cb(const char *name, PurplePrefType type,
 				 gconstpointer value, gpointer data)
 {
@@ -6199,7 +6178,6 @@
 	purple_prefs_add_bool(PIDGIN_PREFS_ROOT "/conversations/send_italic", FALSE);
 	purple_prefs_add_bool(PIDGIN_PREFS_ROOT "/conversations/send_underline", FALSE);
 	purple_prefs_add_bool(PIDGIN_PREFS_ROOT "/conversations/send_strike", FALSE);
-	purple_prefs_add_bool(PIDGIN_PREFS_ROOT "/conversations/spellcheck", TRUE);
 	purple_prefs_add_bool(PIDGIN_PREFS_ROOT "/conversations/show_incoming_formatting", TRUE);
 	/* TODO: it's about *remote* smileys, not local ones */
 	purple_prefs_add_bool(PIDGIN_PREFS_ROOT "/conversations/resize_custom_smileys", TRUE);
@@ -6254,8 +6232,6 @@
 								close_on_tabs_pref_cb, NULL);
 	purple_prefs_connect_callback(handle, PIDGIN_PREFS_ROOT "/conversations/show_formatting_toolbar",
 								show_formatting_toolbar_pref_cb, NULL);
-	purple_prefs_connect_callback(handle, PIDGIN_PREFS_ROOT "/conversations/spellcheck",
-								spellcheck_pref_cb, NULL);
 	purple_prefs_connect_callback(handle, PIDGIN_PREFS_ROOT "/conversations/tab_side",
 								tab_side_pref_cb, NULL);
 
--- a/pidgin/gtkprefs.c	Sat Oct 03 05:58:21 2020 -0500
+++ b/pidgin/gtkprefs.c	Tue Oct 06 20:44:33 2020 -0500
@@ -121,7 +121,6 @@
 			GtkWidget *animate_buddy_icons;
 			GtkWidget *send_typing;
 		} im;
-		GtkWidget *spellcheck;
 		GtkWidget *use_smooth_scrolling;
 		struct {
 			GtkWidget *blink_im;
@@ -1606,8 +1605,6 @@
 
 	pidgin_prefs_bind_checkbox("/purple/conversations/im/send_typing",
 			win->conversations.im.send_typing);
-	pidgin_prefs_bind_checkbox(PIDGIN_PREFS_ROOT "/conversations/spellcheck",
-			win->conversations.spellcheck);
 
 	pidgin_prefs_bind_checkbox(PIDGIN_PREFS_ROOT "/conversations/use_smooth_scrolling",
 			win->conversations.use_smooth_scrolling);
@@ -2876,9 +2873,6 @@
 			conversations.im.send_typing);
 	gtk_widget_class_bind_template_child(
 			widget_class, PidginPrefsWindow,
-			conversations.spellcheck);
-	gtk_widget_class_bind_template_child(
-			widget_class, PidginPrefsWindow,
 			conversations.use_smooth_scrolling);
 	gtk_widget_class_bind_template_child(
 			widget_class, PidginPrefsWindow,
--- a/pidgin/resources/Prefs/prefs.ui	Sat Oct 03 05:58:21 2020 -0500
+++ b/pidgin/resources/Prefs/prefs.ui	Tue Oct 06 20:44:33 2020 -0500
@@ -718,21 +718,6 @@
                                   </packing>
                                 </child>
                                 <child>
-                                  <object class="GtkCheckButton" id="conversations.spellcheck">
-                                    <property name="label" translatable="yes">Highlight _misspelled words</property>
-                                    <property name="visible">True</property>
-                                    <property name="can_focus">True</property>
-                                    <property name="receives_default">False</property>
-                                    <property name="use_underline">True</property>
-                                    <property name="draw_indicator">True</property>
-                                  </object>
-                                  <packing>
-                                    <property name="expand">False</property>
-                                    <property name="fill">True</property>
-                                    <property name="position">6</property>
-                                  </packing>
-                                </child>
-                                <child>
                                   <object class="GtkCheckButton" id="conversations.use_smooth_scrolling">
                                     <property name="label" translatable="yes">Use smooth-scrolling</property>
                                     <property name="visible">True</property>
@@ -743,7 +728,7 @@
                                   <packing>
                                     <property name="expand">False</property>
                                     <property name="fill">True</property>
-                                    <property name="position">7</property>
+                                    <property name="position">6</property>
                                   </packing>
                                 </child>
                                 <child>
@@ -758,7 +743,7 @@
                                   <packing>
                                     <property name="expand">False</property>
                                     <property name="fill">True</property>
-                                    <property name="position">8</property>
+                                    <property name="position">7</property>
                                   </packing>
                                 </child>
                                 <child>
@@ -809,7 +794,7 @@
                                   <packing>
                                     <property name="expand">False</property>
                                     <property name="fill">True</property>
-                                    <property name="position">9</property>
+                                    <property name="position">8</property>
                                   </packing>
                                 </child>
                                 <child>
@@ -847,7 +832,7 @@
                                   <packing>
                                     <property name="expand">False</property>
                                     <property name="fill">True</property>
-                                    <property name="position">10</property>
+                                    <property name="position">9</property>
                                   </packing>
                                 </child>
                               </object>

mercurial