On Mon, 4 Jan 2016 14:11:26 +0100 Davide Andreoli <d...@gurumeditation.it> said:
> 2015-12-20 18:55 GMT+01:00 Davide Andreoli <d...@gurumeditation.it>: > > > > > > > 2015-12-08 1:39 GMT+01:00 taehyub <taehyub....@samsung.com>: > > > >> cedric pushed a commit to branch master. > >> > >> > >> http://git.enlightenment.org/core/elementary.git/commit/?id=61648ba5a38bc8ab1e7a4721f17683abc701ebd4 > >> > >> commit 61648ba5a38bc8ab1e7a4721f17683abc701ebd4 > >> Author: taehyub <taehyub....@samsung.com> > >> Date: Mon Dec 7 15:53:47 2015 -0800 > >> > >> popup: add popup default align config for other profiles. > >> > >> Summary: > >> The alignment of popup can be different in each profiles. > >> So I added the align configuration of popup. > >> @feature > >> > > > > After this commit (I think) all the popup on my system appear top-left > > aligned. > > > > Reading the commit seems quite clear that you need a fresh elm config > > to have the popups correcly aligned... :/ > > > > So? no one replayed to this :( cedric? > > I think we still have broken popup align everywhere (unless you wipe your > elm config) a bug to fix :) > >> Reviewers: cedric, woohyun > >> > >> Differential Revision: https://phab.enlightenment.org/D3357 > >> > >> Signed-off-by: Cedric BAIL <ced...@osg.samsung.com> > >> --- > >> config/default/base.src.in | 2 ++ > >> config/mobile/base.src.in | 2 ++ > >> config/standard/base.src.in | 2 ++ > >> src/lib/elc_popup.c | 5 ++++- > >> src/lib/elm_config.c | 9 +++++++++ > >> src/lib/elm_priv.h | 2 ++ > >> 6 files changed, 21 insertions(+), 1 deletion(-) > >> > >> diff --git a/config/default/base.src.in b/config/default/base.src.in > >> index 5caff3d..188f41f 100644 > >> --- a/config/default/base.src.in > >> +++ b/config/default/base.src.in > >> @@ -95,6 +95,8 @@ group "Elm_Config" struct { > >> value "win_auto_focus_animate" uchar: 1; > >> value "transition_duration_factor" double: 1.0; > >> value "naviframe_prev_btn_auto_pushed" uchar: 1; > >> + value "popup_horizontal_align" double: 0.5; > >> + value "popup_vertical_align" double: 0.5; > >> group "color_palette" list { > >> group "Elm_Custom_Palette" struct { > >> value "palette_name" string: "default"; > >> diff --git a/config/mobile/base.src.in b/config/mobile/base.src.in > >> index 6621fb8..aea7ae2 100644 > >> --- a/config/mobile/base.src.in > >> +++ b/config/mobile/base.src.in > >> @@ -99,6 +99,8 @@ group "Elm_Config" struct { > >> value "win_auto_focus_animate" uchar: 1; > >> value "transition_duration_factor" double: 1.0; > >> value "naviframe_prev_btn_auto_pushed" uchar: 1; > >> + value "popup_horizontal_align" double: 0.5; > >> + value "popup_vertical_align" double: 0.5; > >> group "color_palette" list { > >> group "Elm_Custom_Palette" struct { > >> value "palette_name" string: "default"; > >> diff --git a/config/standard/base.src.in b/config/standard/base.src.in > >> index a89beba..2d87884 100644 > >> --- a/config/standard/base.src.in > >> +++ b/config/standard/base.src.in > >> @@ -96,6 +96,8 @@ group "Elm_Config" struct { > >> value "win_auto_focus_animate" uchar: 1; > >> value "transition_duration_factor" double: 1.0; > >> value "naviframe_prev_btn_auto_pushed" uchar: 1; > >> + value "popup_horizontal_align" double: 0.5; > >> + value "popup_vertical_align" double: 0.5; > >> group "color_palette" list { > >> group "Elm_Custom_Palette" struct { > >> value "palette_name" string: "default"; > >> diff --git a/src/lib/elc_popup.c b/src/lib/elc_popup.c > >> index 9e8d041..7c926f9 100644 > >> --- a/src/lib/elc_popup.c > >> +++ b/src/lib/elc_popup.c > >> @@ -1475,7 +1475,10 @@ _elm_popup_evas_object_smart_add(Eo *obj, > >> Elm_Popup_Data *priv) > >> > >> priv->notify = elm_notify_add(obj); > >> elm_object_style_set(priv->notify, style); > >> - elm_notify_align_set(priv->notify, 0.5, 0.5); > >> + > >> + elm_notify_align_set(priv->notify, > >> + _elm_config->popup_horizontal_align, > >> + _elm_config->popup_vertical_align); > >> elm_notify_allow_events_set(priv->notify, EINA_FALSE); > >> evas_object_size_hint_weight_set > >> (priv->notify, EVAS_HINT_EXPAND, EVAS_HINT_EXPAND); > >> diff --git a/src/lib/elm_config.c b/src/lib/elm_config.c > >> index 22b4909..8e5320c 100644 > >> --- a/src/lib/elm_config.c > >> +++ b/src/lib/elm_config.c > >> @@ -439,6 +439,8 @@ _desc_init(void) > >> ELM_CONFIG_VAL(D, T, win_auto_focus_animate, T_UCHAR); > >> ELM_CONFIG_VAL(D, T, transition_duration_factor, T_DOUBLE); > >> ELM_CONFIG_VAL(D, T, naviframe_prev_btn_auto_pushed, T_UCHAR); > >> + ELM_CONFIG_VAL(D, T, popup_horizontal_align, T_DOUBLE); > >> + ELM_CONFIG_VAL(D, T, popup_vertical_align, T_DOUBLE); > >> #undef T > >> #undef D > >> #undef T_INT > >> @@ -1468,6 +1470,8 @@ _config_load(void) > >> _elm_config->gl_stencil = 0; > >> _elm_config->transition_duration_factor = 1.0; > >> _elm_config->naviframe_prev_btn_auto_pushed = EINA_TRUE; > >> + _elm_config->popup_horizontal_align = 0.5; > >> + _elm_config->popup_vertical_align = 0.5; > >> } > >> > >> static void > >> @@ -2146,6 +2150,11 @@ _env_get(void) > >> > >> s = getenv("ELM_TRANSITION_DURATION_FACTOR"); > >> if (s) _elm_config->transition_duration_factor = atof(s); > >> + > >> + s = getenv("ELM_POPUP_HORIZONTAL_ALIGN"); > >> + if (s) _elm_config->popup_horizontal_align = _elm_atof(s); > >> + s = getenv("ELM_POPUP_VERTICAL_ALIGN"); > >> + if (s) _elm_config->popup_vertical_align = _elm_atof(s); > >> } > >> > >> static void > >> diff --git a/src/lib/elm_priv.h b/src/lib/elm_priv.h > >> index 61c06a8..6c26a03 100644 > >> --- a/src/lib/elm_priv.h > >> +++ b/src/lib/elm_priv.h > >> @@ -246,6 +246,8 @@ struct _Elm_Config > >> unsigned char first_item_focus_on_first_focus_in; /**< This sets the > >> first item focus on first focus in feature*/ > >> Elm_Focus_Autoscroll_Mode focus_autoscroll_mode; /**< This shows the > >> focus auto scroll mode. By default, @c ELM_FOCUS_AUTOSCROLL_MODE_SHOW is > >> set. */ > >> Elm_Slider_Indicator_Visible_Mode slider_indicator_visible_mode; > >> /**< this sets the slider indicator visible mode */ > >> + double popup_horizontal_align; > >> + double popup_vertical_align; > >> int toolbar_shrink_mode; > >> unsigned char fileselector_expand_enable; > >> unsigned char fileselector_double_tap_navigation_enable; > >> > >> -- > >> > >> > >> > > > ------------------------------------------------------------------------------ > _______________________________________________ > 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" -------------- The Rasterman (Carsten Haitzler) ras...@rasterman.com ------------------------------------------------------------------------------ _______________________________________________ enlightenment-devel mailing list enlightenment-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
