On Fri, 9 Oct 2020 16:43:02 +0200 Davide Andreoli <d...@gurumeditation.it> said:
> Il giorno ven 9 ott 2020 alle ore 13:40 Carsten Haitzler < > ras...@rasterman.com> ha scritto: > > > raster pushed a commit to branch master. > > > > > > http://git.enlightenment.org/core/enlightenment.git/commit/?id=048a40414f5de2b72117c5860c4e7e47379b53cc > > > > commit 048a40414f5de2b72117c5860c4e7e47379b53cc > > Author: Carsten Haitzler (Rasterman) <ras...@rasterman.com> > > Date: Mon Oct 5 09:20:05 2020 +0100 > > > > remove offline+presentation mode - this has confused people > > > > i've seen various people get confused as to why their screen won't > > blank and never heard of anyone using offline mode. i think this can > > retire and just disabkling blanking manually or as presentation/media > > tools do these days... they ask to suspend screensaver anyway. yes it > > also affects notifications - but that is a good question - if you want > > these or not? may as well simplify. > > > > I use presentation mode quite often :P > when recoding video files using HandBrake, the process took a lots of hours, > and without presentation mode the PC goes standby and handbrake stop the > process. > > I will find another way :) you can just disable blanking... :) it is just a checkbox. :) presentation mode was ANOTHER checkbox in addition to this so blanking would fail in presentation mode was on and actually it mostly seems like e is buggy... until u remember presentation mode. simplifying to "just turn blanking off in blanking" is probably the simpler way to do this. :) > > --- > > src/bin/e_actions.c | 25 ----- > > src/bin/e_client.c | 6 +- > > src/bin/e_comp.c | 4 +- > > src/bin/e_config.c | 6 -- > > src/bin/e_config.h | 6 -- > > src/bin/e_dpms.c | 4 +- > > src/bin/e_screensaver.c | 126 > > ++--------------------- > > src/modules/conf/e_mod_main.c | 56 ---------- > > src/modules/conf_display/e_int_config_desklock.c | 24 +---- > > src/modules/notification/e_mod_main.c | 121 > > ---------------------- > > src/modules/notification/e_mod_main.h | 9 -- > > src/modules/notification/e_mod_popup.c | 2 - > > 12 files changed, 10 insertions(+), 379 deletions(-) > > > > diff --git a/src/bin/e_actions.c b/src/bin/e_actions.c > > index 0955b3e89..9aaa59c55 100644 > > --- a/src/bin/e_actions.c > > +++ b/src/bin/e_actions.c > > @@ -2152,22 +2152,6 @@ ACT_FN_GO(halt_now, EINA_UNUSED) > > e_sys_action_do(E_SYS_HALT_NOW, NULL); > > } > > > > > > -/***************************************************************************/ > > -ACT_FN_GO(mode_presentation_toggle, EINA_UNUSED) > > -{ > > - e_config->mode.presentation = !e_config->mode.presentation; > > - e_config_mode_changed(); > > - e_config_save_queue(); > > -} > > - > > > > -/***************************************************************************/ > > -ACT_FN_GO(mode_offline_toggle, EINA_UNUSED) > > -{ > > - e_config->mode.offline = !e_config->mode.offline; > > - e_config_mode_changed(); > > - e_config_save_queue(); > > -} > > - > > > > > > /***************************************************************************/ > > static E_Dialog *logout_dialog = NULL; > > > > @@ -3745,15 +3729,6 @@ e_actions_init(void) > > e_action_predef_name_set(N_("Enlightenment"), N_("Exit Now"), > > "exit_now", NULL, NULL, 0); > > > > - ACT_GO(mode_presentation_toggle); > > - e_action_predef_name_set(N_("Enlightenment : Mode"), > > - N_("Presentation Mode Toggle"), > > - "mode_presentation_toggle", NULL, NULL, 0); > > - > > - ACT_GO(mode_offline_toggle); > > - e_action_predef_name_set(N_("Enlightenment : Mode"), > > - N_("Offline Mode Toggle"), > > - "mode_offline_toggle", NULL, NULL, 0); > > /* modules */ > > ACT_GO(module_enable); > > e_action_predef_name_set(N_("Enlightenment : Module"), > > diff --git a/src/bin/e_client.c b/src/bin/e_client.c > > index c38054812..b467fe3c7 100644 > > --- a/src/bin/e_client.c > > +++ b/src/bin/e_client.c > > @@ -102,9 +102,7 @@ _e_client_cb_config_mode(void *data EINA_UNUSED, int > > type EINA_UNUSED, void *ev > > > > /* move fullscreen borders above everything */ > > > > - if (e_config->mode.presentation) > > - layer = E_LAYER_CLIENT_TOP; > > - else if (!e_config->allow_above_fullscreen) > > + if (!e_config->allow_above_fullscreen) > > layer = E_LAYER_CLIENT_FULLSCREEN; > > else > > return ECORE_CALLBACK_RENEW; > > @@ -4513,8 +4511,6 @@ e_client_fullscreen(E_Client *ec, E_Fullscreen > > policy) > > ec->saved.set = 1; > > if (!e_config->allow_above_fullscreen) > > evas_object_layer_set(ec->frame, E_LAYER_CLIENT_FULLSCREEN); > > - else if (e_config->mode.presentation) > > - evas_object_layer_set(ec->frame, E_LAYER_CLIENT_TOP); > > > > ec->fullscreen = 1; > > ec->unfullscreen_forced = 0; > > diff --git a/src/bin/e_comp.c b/src/bin/e_comp.c > > index 877f40fdb..9b7f1842a 100644 > > --- a/src/bin/e_comp.c > > +++ b/src/bin/e_comp.c > > @@ -370,8 +370,6 @@ _e_comp_nocomp_end(void) > > { > > if (!e_config->allow_above_fullscreen) > > layer = E_LAYER_CLIENT_FULLSCREEN; > > - else if (e_config->mode.presentation) > > - layer = E_LAYER_CLIENT_TOP; > > } > > evas_object_layer_set(e_comp->nocomp_ec->frame, layer); > > e_comp->nocomp_ec->fullscreen = fs; > > @@ -693,7 +691,7 @@ nocomp: > > if (!ec->visible) continue; > > if (evas_object_data_get(ec->frame, "comp_skip")) > > continue; > > if (e_object_is_del(E_OBJECT(ec)) || > > (!e_client_util_desk_visible(ec, e_desk_current_get(ec->zone)))) continue; > > - if (ec->override || > > (e_config->allow_above_fullscreen && (!e_config->mode.presentation))) > > + if (ec->override || > > (e_config->allow_above_fullscreen)) > > { > > _e_comp_nocomp_end(); > > break; > > diff --git a/src/bin/e_config.c b/src/bin/e_config.c > > index dd9b75379..cb3fa3922 100644 > > --- a/src/bin/e_config.c > > +++ b/src/bin/e_config.c > > @@ -919,9 +919,6 @@ _e_config_edd_init(Eina_Bool old) > > E_CONFIG_VAL(D, T, syscon.do_input, UCHAR); > > E_CONFIG_LIST(D, T, syscon.actions, _e_config_syscon_action_edd); > > > > - E_CONFIG_VAL(D, T, mode.presentation, UCHAR); > > - E_CONFIG_VAL(D, T, mode.offline, UCHAR); > > - > > E_CONFIG_VAL(D, T, exec.expire_timeout, DOUBLE); > > E_CONFIG_VAL(D, T, exec.show_run_dialog, UCHAR); > > E_CONFIG_VAL(D, T, exec.show_exit_dialog, UCHAR); > > @@ -1932,9 +1929,6 @@ e_config_load(void) > > > > E_CONFIG_LIMIT(e_config->ping_clients_interval, 16, 1024); > > > > - E_CONFIG_LIMIT(e_config->mode.presentation, 0, 1); > > - E_CONFIG_LIMIT(e_config->mode.offline, 0, 1); > > - > > E_CONFIG_LIMIT(e_config->exec.expire_timeout, 0.1, 1000); > > E_CONFIG_LIMIT(e_config->exec.show_run_dialog, 0, 1); > > E_CONFIG_LIMIT(e_config->exec.show_exit_dialog, 0, 1); > > diff --git a/src/bin/e_config.h b/src/bin/e_config.h > > index f95ccd8c8..3578da6fd 100644 > > --- a/src/bin/e_config.h > > +++ b/src/bin/e_config.h > > @@ -345,12 +345,6 @@ struct _E_Config > > Eina_List *actions; > > } syscon; > > > > - struct > > - { > > - unsigned char presentation; // INTERNAL > > - unsigned char offline; // INTERNAL > > - } mode; > > - > > struct > > { > > double expire_timeout; > > diff --git a/src/bin/e_dpms.c b/src/bin/e_dpms.c > > index 026e3cc62..cea42679c 100644 > > --- a/src/bin/e_dpms.c > > +++ b/src/bin/e_dpms.c > > @@ -33,7 +33,6 @@ e_dpms_update(void) > > Eina_Bool changed = EINA_FALSE; > > > > enabled = ((e_config->screensaver_enable) && > > - (!e_config->mode.presentation) && > > (!((e_util_fullscreen_current_any()) && > > (e_config->no_dpms_on_fullscreen)))); > > if (_e_dpms_enabled != enabled) > > @@ -87,8 +86,7 @@ e_dpms_force_update(void) > > unsigned int standby = 0, suspend = 0, off = 0; > > int enabled; > > > > - enabled = ((e_config->screensaver_enable) && > > - (!e_config->mode.presentation)); > > + enabled = (e_config->screensaver_enable); > > #ifndef HAVE_WAYLAND_ONLY > > if (e_comp->comp_type == E_PIXMAP_TYPE_X) > > { > > diff --git a/src/bin/e_screensaver.c b/src/bin/e_screensaver.c > > index d83335850..f12f2eed0 100644 > > --- a/src/bin/e_screensaver.c > > +++ b/src/bin/e_screensaver.c > > @@ -11,8 +11,6 @@ static Ecore_Event_Handler > > *_e_screensaver_handler_border_uniconify = NULL; > > static Ecore_Event_Handler *_e_screensaver_handler_border_desk_set = NULL; > > static Ecore_Event_Handler *_e_screensaver_handler_desk_show = NULL; > > static Ecore_Event_Handler *_e_screensaver_handler_powersave = NULL; > > -static E_Dialog *_e_screensaver_ask_presentation_dia = NULL; > > -static int _e_screensaver_ask_presentation_count = 0; > > > > static int _e_screensaver_timeout = 0; > > static int _e_screensaver_blanking = 0; > > @@ -34,11 +32,11 @@ E_API int E_EVENT_SCREENSAVER_OFF_PRE = -1; > > E_API int > > e_screensaver_timeout_get(Eina_Bool use_idle) > > { > > - int timeout = 0, count = (1 + _e_screensaver_ask_presentation_count); > > + int timeout = 0; > > Eina_Bool use_special_instead_of_dim = EINA_FALSE; > > > > if (_screensaver_now) return 1; > > - if ((e_config->screensaver_enable) && (!e_config->mode.presentation)) > > + if (e_config->screensaver_enable) > > { > > if ((e_desklock_state_get()) && > > (e_config->screensaver_desklock_timeout > 0)) > > @@ -47,11 +45,10 @@ e_screensaver_timeout_get(Eina_Bool use_idle) > > use_special_instead_of_dim = EINA_TRUE; > > } > > else > > - timeout = e_config->screensaver_timeout * count; > > + timeout = e_config->screensaver_timeout; > > } > > > > - if ((use_idle) && (!e_config->mode.presentation) && > > - (!use_special_instead_of_dim)) > > + if ((use_idle) && (!use_special_instead_of_dim)) > > { > > if (e_config->backlight.idle_dim) > > { > > @@ -96,7 +93,6 @@ e_screensaver_update(void) > > > > timeout = e_screensaver_timeout_get(EINA_TRUE); > > if (!((e_config->screensaver_enable) && > > - (!e_config->mode.presentation) && > > (!((e_util_fullscreen_current_any()) && > > (e_config->no_dpms_on_fullscreen))))) > > timeout = 0; > > @@ -137,12 +133,8 @@ e_screensaver_update(void) > > // necessary on some hardware. > > if (!e_config->screensaver_dpms_off) > > { > > - int enabled; > > - > > - enabled = ((e_config->screensaver_enable) && > > - (!e_config->mode.presentation)); > > - ecore_x_dpms_enabled_set(!enabled); > > - ecore_x_dpms_enabled_set(enabled); > > + ecore_x_dpms_enabled_set(!e_config->screensaver_enable); > > + ecore_x_dpms_enabled_set(e_config->screensaver_enable); > > } > > ecore_x_screensaver_set(timeout, interval, blanking, expose); > > } > > @@ -157,97 +149,6 @@ _e_screensaver_handler_config_mode_cb(void *data > > EINA_UNUSED, int type EINA_UNUS > > return ECORE_CALLBACK_PASS_ON; > > } > > > > -static void > > -_e_screensaver_ask_presentation_del(void *data) > > -{ > > - if (_e_screensaver_ask_presentation_dia == data) > > - _e_screensaver_ask_presentation_dia = NULL; > > -} > > - > > -static void > > -_e_screensaver_ask_presentation_yes(void *data EINA_UNUSED, E_Dialog *dia) > > -{ > > - e_config->mode.presentation = 1; > > - e_config_mode_changed(); > > - e_config_save_queue(); > > - e_object_del(E_OBJECT(dia)); > > - _e_screensaver_ask_presentation_count = 0; > > -} > > - > > -static void > > -_e_screensaver_ask_presentation_no(void *data EINA_UNUSED, E_Dialog *dia) > > -{ > > - e_object_del(E_OBJECT(dia)); > > - _e_screensaver_ask_presentation_count = 0; > > -} > > - > > -static void > > -_e_screensaver_ask_presentation_no_increase(void *data EINA_UNUSED, > > E_Dialog *dia) > > -{ > > - _e_screensaver_ask_presentation_count++; > > - e_screensaver_update(); > > - e_object_del(E_OBJECT(dia)); > > -} > > - > > -static void > > -_e_screensaver_ask_presentation_no_forever(void *data EINA_UNUSED, > > E_Dialog *dia) > > -{ > > - e_config->screensaver_ask_presentation = 0; > > - e_config_save_queue(); > > - e_object_del(E_OBJECT(dia)); > > - _e_screensaver_ask_presentation_count = 0; > > -} > > - > > -static void > > -_e_screensaver_ask_presentation_key_down(void *data, Evas *e EINA_UNUSED, > > Evas_Object *o EINA_UNUSED, void *event) > > -{ > > - Evas_Event_Key_Down *ev = event; > > - E_Dialog *dia = data; > > - > > - if (strcmp(ev->key, "Return") == 0) > > - _e_screensaver_ask_presentation_yes(NULL, dia); > > - else if (strcmp(ev->key, "Escape") == 0) > > - _e_screensaver_ask_presentation_no(NULL, dia); > > -} > > - > > -static void > > -_e_screensaver_ask_presentation_mode(void) > > -{ > > - E_Dialog *dia; > > - > > - if (_e_screensaver_ask_presentation_dia) return; > > - > > - if (!(dia = e_dialog_new(NULL, "E", "_screensaver_ask_presentation"))) > > return; > > - > > - e_dialog_title_set(dia, _("Activate Presentation Mode?")); > > - e_dialog_icon_set(dia, "dialog-ask", 64); > > - e_dialog_text_set(dia, > > - _("You disabled the screensaver too fast.<ps/><ps/>" > > - "Would you like to enable <b>presentation</b> mode > > and " > > - "temporarily disable screen saver, lock and power > > saving?")); > > - > > - e_object_del_attach_func_set(E_OBJECT(dia), > > - _e_screensaver_ask_presentation_del); > > - e_dialog_button_add(dia, _("Yes"), NULL, > > - _e_screensaver_ask_presentation_yes, NULL); > > - e_dialog_button_add(dia, _("No"), NULL, > > - _e_screensaver_ask_presentation_no, NULL); > > - e_dialog_button_add(dia, _("No, but increase timeout"), NULL, > > - _e_screensaver_ask_presentation_no_increase, NULL); > > - e_dialog_button_add(dia, _("No, and stop asking"), NULL, > > - _e_screensaver_ask_presentation_no_forever, NULL); > > - > > - e_dialog_button_focus_num(dia, 0); > > - e_widget_list_homogeneous_set(dia->box_object, 0); > > - elm_win_center(dia->win, 1, 1); > > - e_dialog_show(dia); > > - > > - evas_object_event_callback_add(dia->bg_object, EVAS_CALLBACK_KEY_DOWN, > > - > > _e_screensaver_ask_presentation_key_down, dia); > > - > > - _e_screensaver_ask_presentation_dia = dia; > > -} > > - > > static Eina_Bool > > _e_screensaver_suspend_cb(void *data EINA_UNUSED) > > { > > @@ -282,8 +183,6 @@ _e_screensaver_handler_powersave_cb(void *data > > EINA_UNUSED, int type EINA_UNUSED > > return ECORE_CALLBACK_PASS_ON; > > } > > > > -static double last_start = 0.0; > > - > > static Eina_Bool > > _e_screensaver_handler_screensaver_on_cb(void *data EINA_UNUSED, int type > > EINA_UNUSED, void *event EINA_UNUSED) > > { > > @@ -297,8 +196,6 @@ _e_screensaver_handler_screensaver_on_cb(void *data > > EINA_UNUSED, int type EINA_U > > _e_screensaver_suspend_timer = > > ecore_timer_loop_add(e_config->screensaver_suspend_delay, > > _e_screensaver_suspend_cb, NULL); > > - last_start = ecore_loop_time_get(); > > - _e_screensaver_ask_presentation_count = 0; > > return ECORE_CALLBACK_PASS_ON; > > } > > > > @@ -313,17 +210,6 @@ _e_screensaver_handler_screensaver_off_cb(void *data > > EINA_UNUSED, int type EINA_ > > ecore_timer_del(_e_screensaver_suspend_timer); > > _e_screensaver_suspend_timer = NULL; > > } > > - if ((last_start > 0.0) && (e_config->screensaver_ask_presentation)) > > - { > > - double current = ecore_loop_time_get(); > > - > > - if ((last_start + e_config->screensaver_ask_presentation_timeout) > > - >= current) > > - _e_screensaver_ask_presentation_mode(); > > - last_start = 0.0; > > - } > > - else if (_e_screensaver_ask_presentation_count) > > - _e_screensaver_ask_presentation_count = 0; > > return ECORE_CALLBACK_PASS_ON; > > } > > > > diff --git a/src/modules/conf/e_mod_main.c b/src/modules/conf/e_mod_main.c > > index 903512607..7f3f272af 100644 > > --- a/src/modules/conf/e_mod_main.c > > +++ b/src/modules/conf/e_mod_main.c > > @@ -385,56 +385,6 @@ _e_mod_conf_cb(void *data EINA_UNUSED, E_Menu *m > > EINA_UNUSED, E_Menu_Item *mi EI > > e_configure_show(NULL); > > } > > > > -static void > > -_e_mod_mode_presentation_toggle(void *data EINA_UNUSED, E_Menu *m > > EINA_UNUSED, E_Menu_Item *mi) > > -{ > > - e_config->mode.presentation = !e_config->mode.presentation; > > - e_menu_item_toggle_set(mi, e_config->mode.presentation); > > - e_config_mode_changed(); > > - e_config_save_queue(); > > -} > > - > > -static void > > -_e_mod_mode_offline_toggle(void *data EINA_UNUSED, E_Menu *m EINA_UNUSED, > > E_Menu_Item *mi) > > -{ > > - e_config->mode.offline = !e_config->mode.offline; > > - e_menu_item_toggle_set(mi, e_config->mode.offline); > > - e_config_mode_changed(); > > - e_config_save_queue(); > > -} > > - > > -static void > > -_e_mod_submenu_modes_fill(void *data EINA_UNUSED, E_Menu *m) > > -{ > > - E_Menu_Item *mi; > > - > > - mi = e_menu_item_new(m); > > - e_menu_item_check_set(mi, 1); > > - e_menu_item_toggle_set(mi, e_config->mode.presentation); > > - e_menu_item_label_set(mi, _("Presentation")); > > - e_util_menu_item_theme_icon_set(mi, "preferences-modes-presentation"); > > - e_menu_item_callback_set(mi, _e_mod_mode_presentation_toggle, NULL); > > - > > - mi = e_menu_item_new(m); > > - e_menu_item_check_set(mi, 1); > > - e_menu_item_toggle_set(mi, e_config->mode.offline); > > - e_menu_item_label_set(mi, _("Offline")); > > - e_util_menu_item_theme_icon_set(mi, "preferences-modes-offline"); > > - e_menu_item_callback_set(mi, _e_mod_mode_offline_toggle, NULL); > > - > > - e_menu_pre_activate_callback_set(m, NULL, NULL); > > -} > > - > > -static E_Menu * > > -_e_mod_submenu_modes_get(void) > > -{ > > - E_Menu *m; > > - > > - if (!(m = e_menu_new())) return NULL; > > - e_menu_pre_activate_callback_set(m, _e_mod_submenu_modes_fill, NULL); > > - return m; > > -} > > - > > /* menu item add hook */ > > static void > > _e_mod_menu_add(void *data EINA_UNUSED, E_Menu *m) > > @@ -445,12 +395,6 @@ _e_mod_menu_add(void *data EINA_UNUSED, E_Menu *m) > > e_menu_item_label_set(mi, _("Settings Panel")); > > e_util_menu_item_theme_icon_set(mi, "preferences-system"); > > e_menu_item_callback_set(mi, _e_mod_conf_cb, NULL); > > - > > - mi = e_menu_item_new(m); > > - e_menu_item_label_set(mi, _("Modes")); > > - e_util_menu_item_theme_icon_set(mi, "preferences-modes"); > > - e_menu_item_submenu_set(mi, _e_mod_submenu_modes_get()); > > - e_object_unref(E_OBJECT(mi->submenu)); > > } > > > > static void > > diff --git a/src/modules/conf_display/e_int_config_desklock.c > > b/src/modules/conf_display/e_int_config_desklock.c > > index c600b573e..603757b14 100644 > > --- a/src/modules/conf_display/e_int_config_desklock.c > > +++ b/src/modules/conf_display/e_int_config_desklock.c > > @@ -45,8 +45,6 @@ struct _E_Config_Dialog_Data > > E_Desklock_Background_Method bg_method; > > int bg_method_prev; > > Eina_List *bgs; > > - int ask_presentation; > > - double ask_presentation_timeout; > > > > struct > > { > > @@ -174,10 +172,6 @@ _fill_data(E_Config_Dialog_Data *cfdata) > > cfdata->login_zone = e_config->desklock_login_box_zone; > > cfdata->zone = 0; > > } > > - > > - cfdata->ask_presentation = e_config->desklock_ask_presentation; > > - cfdata->ask_presentation_timeout = > > - e_config->desklock_ask_presentation_timeout; > > } > > > > static void * > > @@ -354,18 +348,6 @@ _basic_create(E_Config_Dialog *cfd, Evas *evas, > > E_Config_Dialog_Data *cfdata) > > e_widget_toolbook_page_append(otb, NULL, _("Timers"), ol, > > 1, 1, 1, 0, 0.0, 0.0); > > > > - /* Presentation */ > > - ol = e_widget_list_add(evas, 0, 0); > > - oc = e_widget_check_add(evas, _("Suggest if deactivated before"), > > - &(cfdata->ask_presentation)); > > - e_widget_list_object_append(ol, oc, 1, 1, 0.5); > > - ow = e_widget_slider_add(evas, 1, 0, _("%1.0f seconds"), 1.0, 300.0, > > 10.0, 0, > > - &(cfdata->ask_presentation_timeout), NULL, > > 100); > > - e_widget_check_widget_disable_on_unchecked_add(oc, ow); > > - e_widget_list_object_append(ol, ow, 1, 1, 0.5); > > - e_widget_toolbook_page_append(otb, NULL, _("Presentation Mode"), ol, > > - 1, 1, 1, 0, 0.0, 0.0); > > - > > /* Wallpapers */ > > ol = e_widget_list_add(evas, 0, 0); > > of = e_widget_table_add(e_win_evas_win_get(evas), 1); > > @@ -449,8 +431,6 @@ _basic_apply(E_Config_Dialog *cfd EINA_UNUSED, > > E_Config_Dialog_Data *cfdata) > > e_config->desklock_on_suspend = cfdata->lock_on_suspend; > > e_config->desklock_post_screensaver_time = > > cfdata->post_screensaver_time; > > e_config->desklock_autolock_screensaver = cfdata->screensaver_lock; > > - e_config->desklock_ask_presentation = cfdata->ask_presentation; > > - e_config->desklock_ask_presentation_timeout = > > cfdata->ask_presentation_timeout; > > if (e_config->xkb.desklock_layout != cfdata->desklock_layout) > > { > > e_config->xkb.desklock_layout = > > eina_stringshare_ref(cfdata->desklock_layout); > > @@ -568,9 +548,7 @@ _basic_check_changed(E_Config_Dialog *cfd EINA_UNUSED, > > E_Config_Dialog_Data *cfd > > } > > else if (e_config->desklock_custom_desklock_cmd != > > cfdata->custom_lock_cmd) > > return 1; > > - > > - return (e_config->desklock_ask_presentation != > > cfdata->ask_presentation) || > > - (!EINA_DBL_EQ(e_config->desklock_ask_presentation_timeout, > > cfdata->ask_presentation_timeout)); > > + return 0; > > } > > > > static void > > diff --git a/src/modules/notification/e_mod_main.c > > b/src/modules/notification/e_mod_main.c > > index e71eceea0..e8afe13b4 100644 > > --- a/src/modules/notification/e_mod_main.c > > +++ b/src/modules/notification/e_mod_main.c > > @@ -10,9 +10,6 @@ Config *notification_cfg = NULL; > > > > static E_Config_DD *conf_edd = NULL; > > > > -static unsigned int offline_id; > > -static unsigned int pres_id; > > - > > static unsigned int > > _notification_notify(E_Notification_Notify *n) > > { > > @@ -27,108 +24,6 @@ _notification_notify(E_Notification_Notify *n) > > return new_id; > > } > > > > -static void > > -_notification_id_update(void *d, unsigned int id) > > -{ > > - uintptr_t *update_id = d; > > - > > - *update_id = id; > > -} > > - > > -static void > > -_notification_show_common(const char *summary, > > - const char *body, > > - unsigned int *update_id) > > -{ > > - E_Notification_Notify n; > > - memset(&n, 0, sizeof(E_Notification_Notify)); > > - n.app_name = "enlightenment"; > > - n.replaces_id = *update_id; > > - n.icon.icon = "enlightenment"; > > - n.summary = summary; > > - n.body = body; > > - n.urgency = E_NOTIFICATION_NOTIFY_URGENCY_CRITICAL; > > - e_notification_client_send(&n, _notification_id_update, update_id); > > -} > > - > > -static void > > -_notification_show_presentation(Eina_Bool enabled) > > -{ > > - const char *summary, *body; > > - > > - if (enabled) > > - { > > - summary = _("Entered Presentation Mode"); > > - body = _("Enlightenment has now entered <b>presentation</b> mode." > > - "<ps/>During presentation mode, screen saver, lock and " > > - "power saving will be disabled so you are not > > interrupted."); > > - } > > - else > > - { > > - summary = _("Exited Presentation Mode"); > > - body = _("Presentation mode has been exited." > > - "<ps/>Now screen saver, lock and " > > - "power saving settings will be restored."); > > - } > > - > > - _notification_show_common(summary, body, &pres_id); > > -} > > - > > -static void > > -_notification_show_offline(Eina_Bool enabled) > > -{ > > - const char *summary, *body; > > - > > - if (enabled) > > - { > > - summary = _("Enter Offline Mode"); > > - body = _("Enlightenment is in <b>offline</b> mode.<ps/>" > > - "During offline mode, modules that use network will stop > > " > > - "polling remote services."); > > - } > > - else > > - { > > - summary = _("Exited Offline Mode"); > > - body = _("Now in <b>online</b> mode.<ps/>" > > - "Now modules that use network will " > > - "resume regular tasks."); > > - } > > - > > - _notification_show_common(summary, body, &offline_id); > > -} > > - > > -static Eina_Bool > > -_notification_cb_config_mode_changed(Config *m_cfg, > > - int type EINA_UNUSED, > > - void *event EINA_UNUSED) > > -{ > > - if (m_cfg->last_config_mode.presentation != > > e_config->mode.presentation) > > - { > > - m_cfg->last_config_mode.presentation = > > e_config->mode.presentation; > > - _notification_show_presentation(e_config->mode.presentation); > > - } > > - > > - if (m_cfg->last_config_mode.offline != e_config->mode.offline) > > - { > > - m_cfg->last_config_mode.offline = e_config->mode.offline; > > - _notification_show_offline(e_config->mode.offline); > > - } > > - > > - return EINA_TRUE; > > -} > > - > > -static Eina_Bool > > -_notification_cb_initial_mode_timer(Config *m_cfg) > > -{ > > - if (e_config->mode.presentation) > > - _notification_show_presentation(1); > > - if (e_config->mode.offline) > > - _notification_show_offline(1); > > - > > - m_cfg->initial_mode_timer = NULL; > > - return EINA_FALSE; > > -} > > - > > /* Module Api Functions */ > > E_API E_Module_Api e_modapi = {E_MODULE_API_VERSION, "Notification"}; > > > > @@ -210,14 +105,6 @@ e_modapi_init(E_Module *m) > > return NULL; > > } > > > > - notification_cfg->last_config_mode.presentation = > > e_config->mode.presentation; > > - notification_cfg->last_config_mode.offline = e_config->mode.offline; > > - notification_cfg->handler = ecore_event_handler_add > > - (E_EVENT_CONFIG_MODE_CHANGED, > > (Ecore_Event_Handler_Cb)_notification_cb_config_mode_changed, > > - notification_cfg); > > - notification_cfg->initial_mode_timer = ecore_timer_loop_add > > - (0.1, (Ecore_Task_Cb)_notification_cb_initial_mode_timer, > > notification_cfg); > > - > > notification_mod = m; > > > > return m; > > @@ -226,12 +113,6 @@ e_modapi_init(E_Module *m) > > E_API int > > e_modapi_shutdown(E_Module *m EINA_UNUSED) > > { > > - if (notification_cfg->initial_mode_timer) > > - ecore_timer_del(notification_cfg->initial_mode_timer); > > - > > - if (notification_cfg->handler) > > - ecore_event_handler_del(notification_cfg->handler); > > - > > if (notification_cfg->cfd) > > e_object_del(E_OBJECT(notification_cfg->cfd)); > > e_configure_registry_item_del("extensions/notification"); > > e_configure_registry_category_del("extensions"); > > @@ -239,11 +120,9 @@ e_modapi_shutdown(E_Module *m EINA_UNUSED) > > notification_popup_shutdown(); > > e_notification_server_unregister(); > > > > - > > _notification_cfg_free(notification_cfg); > > E_CONFIG_DD_FREE(conf_edd); > > notification_mod = NULL; > > - > > return 1; > > } > > > > diff --git a/src/modules/notification/e_mod_main.h > > b/src/modules/notification/e_mod_main.h > > index b5bb9fc7a..714448166 100644 > > --- a/src/modules/notification/e_mod_main.h > > +++ b/src/modules/notification/e_mod_main.h > > @@ -43,17 +43,8 @@ struct _Config > > float timeout; > > Popup_Corner corner; > > > > - struct > > - { > > - Eina_Bool presentation; > > - Eina_Bool offline; > > - } last_config_mode; > > - > > - Ecore_Event_Handler *handler; > > Eina_List *popups; > > unsigned int next_id; > > - > > - Ecore_Timer *initial_mode_timer; > > }; > > > > struct _Popup_Data > > diff --git a/src/modules/notification/e_mod_popup.c > > b/src/modules/notification/e_mod_popup.c > > index 9c2c7b077..4ca3456d7 100644 > > --- a/src/modules/notification/e_mod_popup.c > > +++ b/src/modules/notification/e_mod_popup.c > > @@ -110,11 +110,9 @@ notification_popup_notify(E_Notification_Notify *n, > > { > > case E_NOTIFICATION_NOTIFY_URGENCY_LOW: > > if (!notification_cfg->show_low) return; > > - if (e_config->mode.presentation) return; > > break; > > case E_NOTIFICATION_NOTIFY_URGENCY_NORMAL: > > if (!notification_cfg->show_normal) return; > > - if (e_config->mode.presentation) return; > > break; > > case E_NOTIFICATION_NOTIFY_URGENCY_CRITICAL: > > if (!notification_cfg->show_critical) return; > > > > -- > > > > > > > > _______________________________________________ > enlightenment-devel mailing list > enlightenment-devel@lists.sourceforge.net > https://lists.sourceforge.net/lists/listinfo/enlightenment-devel > -- ------------- Codito, ergo sum - "I code, therefore I am" -------------- Carsten Haitzler - ras...@rasterman.com _______________________________________________ enlightenment-devel mailing list enlightenment-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/enlightenment-devel