Surely elm_object_resize would be more appropriate - after all elm should all be scaled, so adding that to the name now could be confusing?
A good point about the idea of general object resize though :) Andy On Wed, 27 Apr 2016 at 17:07, Tom Hacohen <t...@osg.samsung.com> wrote: > On 27/04/16 16:59, Andrew Williams wrote: > > Hi, > > > > It seems that this is a frequent issue, and a common use case is to set > the > > window size. > > As it's not clear to everyone that evas is not pre-multiplied (as it > were) > > with elm's scale value I wonder if it would be better to provide an > > elm_win_resize that will simply multiiply the value and call the > > evas_object_resize. > > > > I suggested this a while back but can't recall why people didn't like it. > > Anyone against it now? > > elm_object_scaled_resize() sounds better to me. > > > > > Thanks, > > Andrew > > > > On Wed, 27 Apr 2016 at 16:56 Andy Williams <a...@andywilliams.me> wrote: > > > >> ajwillia-ms pushed a commit to branch master. > >> > >> > >> > http://git.enlightenment.org/core/efl.git/commit/?id=c47b5a364c7f8e467bc3976eb0e040e0fd787d3d > >> > >> commit c47b5a364c7f8e467bc3976eb0e040e0fd787d3d > >> Author: Andy Williams <a...@andywilliams.me> > >> Date: Wed Apr 27 16:20:02 2016 +0100 > >> > >> elementary: Set window size based on scale > >> --- > >> src/bin/elementary/config.c | 3 ++- > >> src/bin/elementary/test.c | 2 +- > >> 2 files changed, 3 insertions(+), 2 deletions(-) > >> > >> diff --git a/src/bin/elementary/config.c b/src/bin/elementary/config.c > >> index ed1261a..2eb64b8 100644 > >> --- a/src/bin/elementary/config.c > >> +++ b/src/bin/elementary/config.c > >> @@ -4167,7 +4167,8 @@ win_create(void) > >> _status_config(win, bx0); > >> else _status_config_full(win, bx0); > >> > >> - evas_object_resize(win, 320, 480); > >> + evas_object_resize(win, 320 * elm_config_scale_get(), > >> + 480 * elm_config_scale_get()); > >> evas_object_show(win); > >> } > >> > >> diff --git a/src/bin/elementary/test.c b/src/bin/elementary/test.c > >> index 5f272a0..de8e6d3 100644 > >> --- a/src/bin/elementary/test.c > >> +++ b/src/bin/elementary/test.c > >> @@ -940,7 +940,7 @@ add_tests: > >> } > >> > >> /* set an initial window size */ > >> - evas_object_resize(win, 480, 480); > >> + evas_object_resize(win, 480 * elm_config_scale_get(), 480 * > >> elm_config_scale_get()); > >> evas_object_show(win); > >> } > >> > >> > >> -- > >> > >> > >> > > > ------------------------------------------------------------------------------ > > Find and fix application performance issues faster with Applications > Manager > > Applications Manager provides deep performance insights into multiple > tiers of > > your business applications. It resolves application problems quickly and > > reduces your MTTR. Get your free trial! > > https://ad.doubleclick.net/ddm/clk/302982198;130105516;z > > _______________________________________________ > > enlightenment-devel mailing list > > enlightenment-devel@lists.sourceforge.net > > https://lists.sourceforge.net/lists/listinfo/enlightenment-devel > > > > > > ------------------------------------------------------------------------------ > Find and fix application performance issues faster with Applications > Manager > Applications Manager provides deep performance insights into multiple > tiers of > your business applications. It resolves application problems quickly and > reduces your MTTR. Get your free trial! > https://ad.doubleclick.net/ddm/clk/302982198;130105516;z > _______________________________________________ > enlightenment-devel mailing list > enlightenment-devel@lists.sourceforge.net > https://lists.sourceforge.net/lists/listinfo/enlightenment-devel > ------------------------------------------------------------------------------ Find and fix application performance issues faster with Applications Manager Applications Manager provides deep performance insights into multiple tiers of your business applications. It resolves application problems quickly and reduces your MTTR. Get your free trial! https://ad.doubleclick.net/ddm/clk/302982198;130105516;z _______________________________________________ enlightenment-devel mailing list enlightenment-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
