Ok, if there is no objection. Just leave it :)

Daniel Juyung Seo (SeoZ)


On Tue, Sep 10, 2013 at 10:21 PM, Kim Shinwoo <kimcinoo....@gmail.com>wrote:

> hello. I'm standing on the 2nd side. :)
>
>
> On Sun, Sep 8, 2013 at 12:58 AM, Daniel Juyung Seo <seojuyu...@gmail.com
> >wrote:
>
> > Well, I just realized "access_text" was added in elementary 1.7 and this
> is
> > a theme api break.
> > So what we can do is:
> > 1. Support both "access_text" and "access.txt" for backport support. But
> > code gets dirty.
> > 2. Break theme API. I guess this access text was introduced right before
> > 1.7 release and would not be used that much. As we allow theme API break
> > for some exceptional cases, it's the last chance to fix wrong thing.
> >
> > Daniel Juyung Seo (SeoZ)
> >
> >
> > On Sat, Sep 7, 2013 at 11:56 AM, Shinwoo Kim - Enlightenment Git <
> > no-re...@enlightenment.org> wrote:
> >
> > > kimcinoo pushed a commit to branch master.
> > >
> > > commit e74b8896c5eb37609d8a829e6c1f978c29012a05
> > > Author: Shinwoo Kim <cinoo....@samsung.com>
> > > Date:   Sat Sep 7 11:56:11 2013 +0900
> > >
> > >     [spinner] change part name to access.text from access_text for
> naming
> > > consistency
> > > ---
> > >  NEWS                            |  1 +
> > >  data/themes/widgets/spinner.edc | 12 ++++++------
> > >  src/lib/elm_spinner.c           |  6 +++---
> > >  3 files changed, 10 insertions(+), 9 deletions(-)
> > >
> > > diff --git a/NEWS b/NEWS
> > > index c6046bd..30d5df0 100644
> > > --- a/NEWS
> > > +++ b/NEWS
> > > @@ -275,6 +275,7 @@ Fixes:
> > >     imf is available, the keyboard mode should be decided by ime, but
> not
> > > entry.
> > >     This prevents the asynchronous states between ime and conformant.
> > >     * Flip : Fix the flip page to not flip on a direction if the
> > direction
> > > is disabled.
> > > +   * Spinner : change part name to access.text from access_text on
> > > default style.
> > >
> > >  Removals:
> > >
> > > diff --git a/data/themes/widgets/spinner.edc
> > > b/data/themes/widgets/spinner.edc
> > > index ac518e1..3edad88 100644
> > > --- a/data/themes/widgets/spinner.edc
> > > +++ b/data/themes/widgets/spinner.edc
> > > @@ -355,7 +355,7 @@ group { name: "elm/spinner/base/default";
> > >              visible: 1;
> > >           }
> > >        }
> > > -      part { name: "access_text";
> > > +      part { name: "access.text";
> > >           type: RECT;
> > >           repeat_events: 1;
> > >           description { state: "default" 0.0;
> > > @@ -518,7 +518,7 @@ group { name: "elm/spinner/base/default";
> > >           target: "right_over1";
> > >           target: "right_over2";
> > >           target: "disabler";
> > > -         target: "access_text";
> > > +         target: "access.text";
> > >           after: "disable_text";
> > >        }
> > >        program { name: "disable_text";
> > > @@ -549,7 +549,7 @@ group { name: "elm/spinner/base/default";
> > >           target: "right_over1";
> > >           target: "right_over2";
> > >           target: "disabler";
> > > -         target: "access_text";
> > > +         target: "access.text";
> > >           after: "enable_text";
> > >        }
> > >        program { name: "enable_text";
> > > @@ -908,7 +908,7 @@ group { name: "elm/spinner/base/vertical";
> > >              visible: 1;
> > >           }
> > >        }
> > > -      part { name: "access_text";
> > > +      part { name: "access.text";
> > >           type: RECT;
> > >           repeat_events: 1;
> > >           description { state: "default" 0.0;
> > > @@ -993,7 +993,7 @@ group { name: "elm/spinner/base/vertical";
> > >           target: "down_bt";
> > >           target: "up_bt";
> > >           target: "disabler";
> > > -         target: "access_text";
> > > +         target: "access.text";
> > >           after: "disable_text";
> > >        }
> > >        program { name: "disable_text";
> > > @@ -1020,7 +1020,7 @@ group { name: "elm/spinner/base/vertical";
> > >           target: "down_bt";
> > >           target: "up_bt";
> > >           target: "disabler";
> > > -         target: "access_text";
> > > +         target: "access.text";
> > >           after: "enable_text";
> > >        }
> > >        program { name: "enable_text";
> > > diff --git a/src/lib/elm_spinner.c b/src/lib/elm_spinner.c
> > > index 5861c04..1afe98b 100644
> > > --- a/src/lib/elm_spinner.c
> > > +++ b/src/lib/elm_spinner.c
> > > @@ -614,7 +614,7 @@ _access_spinner_register(Evas_Object *obj,
> Eina_Bool
> > > is_access)
> > >            (obj, elm_layout_edje_get(obj), decrement_part);
> > >
> > >          _elm_access_edje_object_part_object_unregister
> > > -          (obj, elm_layout_edje_get(obj), "access_text");
> > > +          (obj, elm_layout_edje_get(obj), "access.text");
> > >
> > >          return;
> > >       }
> > > @@ -639,7 +639,7 @@ _access_spinner_register(Evas_Object *obj,
> Eina_Bool
> > > is_access)
> > >
> > >     /* register spinner label */
> > >     ao = _elm_access_edje_object_part_object_register
> > > -          (obj, elm_layout_edje_get(obj), "access_text");
> > > +          (obj, elm_layout_edje_get(obj), "access.text");
> > >
> > >     ai = _elm_access_info_get(ao);
> > >     _elm_access_text_set(ai, ELM_ACCESS_TYPE, E_("spinner"));
> > > @@ -797,7 +797,7 @@ _elm_spinner_smart_focus_next(Eo *obj, void *_pd
> > > EINA_UNUSED, va_list *list)
> > >          decrement_part = "left_bt";
> > >       }
> > >
> > > -   ao = _access_object_get(obj, "access_text");
> > > +   ao = _access_object_get(obj, "access.text");
> > >     items = eina_list_append(items, ao);
> > >
> > >     ao = _access_object_get(obj, decrement_part);
> > >
> > > --
> > >
> > >
> > >
> >
> ------------------------------------------------------------------------------
> > > Learn the latest--Visual Studio 2012, SharePoint 2013, SQL 2012, more!
> > > Discover the easy way to master current and previous Microsoft
> > technologies
> > > and advance your career. Get an incredible 1,500+ hours of step-by-step
> > > tutorial videos with LearnDevNow. Subscribe today and save!
> > >
> >
> http://pubads.g.doubleclick.net/gampad/clk?id=58041391&iu=/4140/ostg.clktrk
> > >
> >
> >
> ------------------------------------------------------------------------------
> > Learn the latest--Visual Studio 2012, SharePoint 2013, SQL 2012, more!
> > Discover the easy way to master current and previous Microsoft
> technologies
> > and advance your career. Get an incredible 1,500+ hours of step-by-step
> > tutorial videos with LearnDevNow. Subscribe today and save!
> >
> http://pubads.g.doubleclick.net/gampad/clk?id=58041391&iu=/4140/ostg.clktrk
> > _______________________________________________
> > enlightenment-devel mailing list
> > enlightenment-devel@lists.sourceforge.net
> > https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
> >
>
> ------------------------------------------------------------------------------
> How ServiceNow helps IT people transform IT departments:
> 1. Consolidate legacy IT systems to a single system of record for IT
> 2. Standardize and globalize service processes across IT
> 3. Implement zero-touch automation to replace manual, redundant tasks
> http://pubads.g.doubleclick.net/gampad/clk?id=51271111&iu=/4140/ostg.clktrk
> _______________________________________________
> enlightenment-devel mailing list
> enlightenment-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
>
------------------------------------------------------------------------------
How ServiceNow helps IT people transform IT departments:
1. Consolidate legacy IT systems to a single system of record for IT
2. Standardize and globalize service processes across IT
3. Implement zero-touch automation to replace manual, redundant tasks
http://pubads.g.doubleclick.net/gampad/clk?id=51271111&iu=/4140/ostg.clktrk
_______________________________________________
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel

Reply via email to