matteo
/
malcontent
1
0
Fork 0
Code Issues Pull Requests 2 Packages Projects Releases Wiki Activity

user-controls: Relabel ‘allow web browsers’ as ‘restrict web browsers’ 

This is part of a move to make all the controls restrictive, rather than
permissive.

Signed-off-by: Philip Withnall <withnall@endlessm.com>

Helps: #11
This commit is contained in:
Philip Withnall 2020-02-24 16:34:37 +00:00
parent 8027351f58
commit 8e4fa643d3
2 changed files with 37 additions and 34 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

48
libmalcontent-ui/user-controls.c
View File

@ -77,7 +77,7 @@ struct _MctUserControls
GMenu *age_menu; GMenu *age_menu;
GtkSwitch *allow_system_installation_switch; GtkSwitch *allow_system_installation_switch;
GtkSwitch *allow_user_installation_switch; GtkSwitch *allow_user_installation_switch;
GtkSwitch *allow_web_browsers_switch; GtkSwitch *restrict_web_browsers_switch;
GtkButton *restriction_button; GtkButton *restriction_button;
GtkPopover *restriction_popover; GtkPopover *restriction_popover;
MctRestrictApplicationsDialog *restrict_applications_dialog; MctRestrictApplicationsDialog *restrict_applications_dialog;
@ -111,9 +111,9 @@ static void on_allow_installation_switch_active_changed_cb (GtkSwitch *s,
GParamSpec *pspec, GParamSpec *pspec,
MctUserControls *self); MctUserControls *self);
static void on_allow_web_browsers_switch_active_changed_cb (GtkSwitch *s, static void on_restrict_web_browsers_switch_active_changed_cb (GtkSwitch *s,
GParamSpec *pspec, GParamSpec *pspec,
MctUserControls *self); MctUserControls *self);
static void on_restrict_applications_button_clicked_cb (GtkButton *button, static void on_restrict_applications_button_clicked_cb (GtkButton *button,
gpointer user_data); gpointer user_data);
@ -455,23 +455,23 @@ update_allow_app_installation (MctUserControls *self)
} }
static void static void
update_allow_web_browsers (MctUserControls *self) update_restrict_web_browsers (MctUserControls *self)
{ {
gboolean allow_web_browsers; gboolean restrict_web_browsers;
allow_web_browsers = mct_app_filter_is_content_type_allowed (self->filter, restrict_web_browsers = !mct_app_filter_is_content_type_allowed (self->filter,
WEB_BROWSERS_CONTENT_TYPE); WEB_BROWSERS_CONTENT_TYPE);
g_signal_handlers_block_by_func (self->allow_web_browsers_switch, g_signal_handlers_block_by_func (self->restrict_web_browsers_switch,
on_allow_web_browsers_switch_active_changed_cb, on_restrict_web_browsers_switch_active_changed_cb,
self); self);
gtk_switch_set_active (self->allow_web_browsers_switch, allow_web_browsers); gtk_switch_set_active (self->restrict_web_browsers_switch, restrict_web_browsers);
g_debug ("Allow web browsers: %s", allow_web_browsers ? "yes" : "no"); g_debug ("Restrict web browsers: %s", restrict_web_browsers ? "yes" : "no");
g_signal_handlers_unblock_by_func (self->allow_web_browsers_switch, g_signal_handlers_unblock_by_func (self->restrict_web_browsers_switch,
on_allow_web_browsers_switch_active_changed_cb, on_restrict_web_browsers_switch_active_changed_cb,
self); self);
} }
@ -497,7 +497,7 @@ setup_parental_control_settings (MctUserControls *self)
update_oars_level (self); update_oars_level (self);
update_categories_from_language (self); update_categories_from_language (self);
update_allow_app_installation (self); update_allow_app_installation (self);
update_allow_web_browsers (self); update_restrict_web_browsers (self);
} }
/* Callbacks */ /* Callbacks */
@ -562,9 +562,9 @@ on_allow_installation_switch_active_changed_cb (GtkSwitch *s,
} }
static void static void
on_allow_web_browsers_switch_active_changed_cb (GtkSwitch *s, on_restrict_web_browsers_switch_active_changed_cb (GtkSwitch *s,
GParamSpec *pspec, GParamSpec *pspec,
MctUserControls *self) MctUserControls *self)
{ {
/* Save the changes. */ /* Save the changes. */
schedule_update_blacklisted_apps (self); schedule_update_blacklisted_apps (self);
@ -930,7 +930,7 @@ mct_user_controls_class_init (MctUserControlsClass *klass)
gtk_widget_class_bind_template_child (widget_class, MctUserControls, age_menu); gtk_widget_class_bind_template_child (widget_class, MctUserControls, age_menu);
gtk_widget_class_bind_template_child (widget_class, MctUserControls, allow_system_installation_switch); gtk_widget_class_bind_template_child (widget_class, MctUserControls, allow_system_installation_switch);
gtk_widget_class_bind_template_child (widget_class, MctUserControls, allow_user_installation_switch); gtk_widget_class_bind_template_child (widget_class, MctUserControls, allow_user_installation_switch);
gtk_widget_class_bind_template_child (widget_class, MctUserControls, allow_web_browsers_switch); gtk_widget_class_bind_template_child (widget_class, MctUserControls, restrict_web_browsers_switch);
gtk_widget_class_bind_template_child (widget_class, MctUserControls, restriction_button); gtk_widget_class_bind_template_child (widget_class, MctUserControls, restriction_button);
gtk_widget_class_bind_template_child (widget_class, MctUserControls, restriction_popover); gtk_widget_class_bind_template_child (widget_class, MctUserControls, restriction_popover);
gtk_widget_class_bind_template_child (widget_class, MctUserControls, restrict_applications_dialog); gtk_widget_class_bind_template_child (widget_class, MctUserControls, restrict_applications_dialog);
@ -938,7 +938,7 @@ mct_user_controls_class_init (MctUserControlsClass *klass)
gtk_widget_class_bind_template_child (widget_class, MctUserControls, software_installation_permissions_listbox); gtk_widget_class_bind_template_child (widget_class, MctUserControls, software_installation_permissions_listbox);
gtk_widget_class_bind_template_callback (widget_class, on_allow_installation_switch_active_changed_cb); gtk_widget_class_bind_template_callback (widget_class, on_allow_installation_switch_active_changed_cb);
gtk_widget_class_bind_template_callback (widget_class, on_allow_web_browsers_switch_active_changed_cb); gtk_widget_class_bind_template_callback (widget_class, on_restrict_web_browsers_switch_active_changed_cb);
gtk_widget_class_bind_template_callback (widget_class, on_restrict_applications_button_clicked_cb); gtk_widget_class_bind_template_callback (widget_class, on_restrict_applications_button_clicked_cb);
gtk_widget_class_bind_template_callback (widget_class, on_restrict_applications_dialog_delete_event_cb); gtk_widget_class_bind_template_callback (widget_class, on_restrict_applications_dialog_delete_event_cb);
gtk_widget_class_bind_template_callback (widget_class, on_restrict_applications_dialog_response_cb); gtk_widget_class_bind_template_callback (widget_class, on_restrict_applications_dialog_response_cb);
@ -1354,7 +1354,7 @@ void
mct_user_controls_build_app_filter (MctUserControls *self, mct_user_controls_build_app_filter (MctUserControls *self,
MctAppFilterBuilder *builder) MctAppFilterBuilder *builder)
{ {
gboolean allow_web_browsers; gboolean restrict_web_browsers;
gsize i; gsize i;
g_return_if_fail (MCT_IS_USER_CONTROLS (self)); g_return_if_fail (MCT_IS_USER_CONTROLS (self));
@ -1389,11 +1389,11 @@ mct_user_controls_build_app_filter (MctUserControls *self,
} }
/* Web browsers */ /* Web browsers */
allow_web_browsers = gtk_switch_get_active (self->allow_web_browsers_switch); restrict_web_browsers = gtk_switch_get_active (self->restrict_web_browsers_switch);
g_debug ("\t → %s web browsers", allow_web_browsers ? "Enabling" : "Disabling"); g_debug ("\t → %s web browsers", restrict_web_browsers ? "Restricting" : "Allowing");
if (!allow_web_browsers) if (restrict_web_browsers)
mct_app_filter_builder_blacklist_content_type (builder, WEB_BROWSERS_CONTENT_TYPE); mct_app_filter_builder_blacklist_content_type (builder, WEB_BROWSERS_CONTENT_TYPE);
/* App installation */ /* App installation */

23
libmalcontent-ui/user-controls.ui
View File

@ -57,18 +57,18 @@
<property name="row-spacing">4</property> <property name="row-spacing">4</property>
<property name="column-spacing">4</property> <property name="column-spacing">4</property>
<child> <child>
<object class="GtkLabel" id="allow_web_browsers_label"> <object class="GtkLabel" id="restrict_web_browsers_label">
<property name="visible">True</property> <property name="visible">True</property>
<property name="can_focus">False</property> <property name="can_focus">False</property>
<property name="halign">start</property> <property name="halign">start</property>
<property name="hexpand">True</property> <property name="hexpand">True</property>
<property name="ellipsize">end</property> <property name="ellipsize">end</property>
<property name="xalign">0</property> <property name="xalign">0</property>
<property name="label" translatable="yes">Allow _Web Browsers</property> <property name="label" translatable="yes">Restrict _Web Browsers</property>
<property name="use_underline">True</property> <property name="use_underline">True</property>
<property name="mnemonic_widget">allow_web_browsers_switch</property> <property name="mnemonic_widget">restrict_web_browsers_switch</property>
<accessibility> <accessibility>
<relation target="allow_web_browsers_switch" type="label-for"/> <relation target="restrict_web_browsers_switch" type="label-for"/>
</accessibility> </accessibility>
</object> </object>
<packing> <packing>
@ -77,7 +77,7 @@
</packing> </packing>
</child> </child>
<child> <child>
<object class="GtkLabel" id="allow_web_browsers_description"> <object class="GtkLabel" id="restrict_web_browsers_description">
<property name="visible">True</property> <property name="visible">True</property>
<property name="can_focus">False</property> <property name="can_focus">False</property>
<property name="halign">start</property> <property name="halign">start</property>
@ -92,7 +92,7 @@
<class name="dim-label" /> <class name="dim-label" />
</style> </style>
<accessibility> <accessibility>
<relation target="allow_web_browsers_switch" type="description-for"/> <relation target="restrict_web_browsers_switch" type="description-for"/>
</accessibility> </accessibility>
</object> </object>
<packing> <packing>
@ -101,12 +101,15 @@
</packing> </packing>
</child> </child>
<child> <child>
<object class="GtkSwitch" id="allow_web_browsers_switch"> <object class="GtkSwitch" id="restrict_web_browsers_switch">
<property name="visible">True</property> <property name="visible">True</property>
<property name="can_focus">True</property> <property name="can_focus">True</property>
<property name="halign">end</property> <property name="halign">end</property>
<property name="valign">center</property> <property name="valign">center</property>
<signal name="notify::active" handler="on_allow_web_browsers_switch_active_changed_cb" object="MctUserControls" swapped="no" /> <signal name="notify::active" handler="on_restrict_web_browsers_switch_active_changed_cb" object="MctUserControls" swapped="no" />
<style>
<class name="restricts" />
</style>
</object> </object>
<packing> <packing>
<property name="left-attach">1</property> <property name="left-attach">1</property>
@ -514,8 +517,8 @@
<object class="GtkSizeGroup"> <object class="GtkSizeGroup">
<property name="mode">horizontal</property> <property name="mode">horizontal</property>
<widgets> <widgets>
<widget name="allow_web_browsers_label" /> <widget name="restrict_web_browsers_label" />
<widget name="allow_web_browsers_description" /> <widget name="restrict_web_browsers_description" />
<widget name="restrict_applications_label" /> <widget name="restrict_applications_label" />
<widget name="restrict_applications_description" /> <widget name="restrict_applications_description" />
<widget name="restriction_label" /> <widget name="restriction_label" />