On Fri, 14 Dec 2012 14:49:07 +0900
Carsten Haitzler (The Rasterman) <[email protected]> wrote:

> On Thu, 13 Dec 2012 12:39:22 +0000 Michael Blumenkrantz
> <[email protected]> said:
> 
> i suggest you then revert 80747, 80774, and 80890 which are all feature adds.
> 80654 is a code cleanup and no fix - so under the same strict rules - revert.

big difference between adding an actual feature like this one and copy/pasting 
a couple lines of code around to add an event which is not used internally and 
thus cannot cause bugs by default. if you were going to make this argument, you 
should have tested it adequately such that the themes in SVN at least look 
decent with it.

> 
> also - no improvements to randr dialog then from devilhorns. you can stop work
> now as no features are allowed.
> 
> seriously... you violate this yourself repeatedly... and you plan on reverting
> the randr code devilhorns has been hoarding until now and is meant to go into
> svn? it ADDS cloning controls... for example.

you assume that this will be done in time for release I guess. he's only got a 
couple more days to work on it since he'll be on vacation, and, while I have 
high hopes and trust in his ability, his current outlook is not nearly as 
optimistic.

regardless, this would be a bug fix since the current dialog barely works, and 
we've previously discussed that it needed to go in.

> 
> :)

at this point I give up. we may as well go on and throw the edbus2 changes in 
now

> 
> > being awesome is great, but we're 1 week away from final release and have
> > been in a feature freeze since last friday when we hit beta. please revert
> > this until after the release.
> > 
> > On Thu, Dec 13, 2012 at 12:27 PM, Enlightenment SVN <
> > [email protected]> wrote:
> > 
> > > Log:
> > > lets be awesome ... and make the theme preview.. look a lot more like
> > >   a REAL PREVIEW OF A THEME!
> > >
> > >
> > >
> > > Author:       raster
> > > Date:         2012-12-13 04:27:03 -0800 (Thu, 13 Dec 2012)
> > > New Revision: 80834
> > > Trac:         http://trac.enlightenment.org/e/changeset/80834
> > >
> > > Modified:
> > >   trunk/e/src/modules/conf_theme/e_int_config_theme.c
> > ------------------------------------------------------------------------------
> > LogMeIn Rescue: Anywhere, Anytime Remote support for IT. Free Trial
> > Remotely access PCs and mobile devices and provide instant support
> > Improve your efficiency, and focus on delivering more value-add services
> > Discover what IT Professionals Know. Rescue delivers
> > http://p.sf.net/sfu/logmein_12329d2d
> > _______________________________________________
> > enlightenment-devel mailing list
> > [email protected]
> > https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
> > 
> 
> 

------------------------------------------------------------------------------
LogMeIn Rescue: Anywhere, Anytime Remote support for IT. Free Trial
Remotely access PCs and mobile devices and provide instant support
Improve your efficiency, and focus on delivering more value-add services
Discover what IT Professionals Know. Rescue delivers
http://p.sf.net/sfu/logmein_12329d2d
_______________________________________________
enlightenment-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel

Reply via email to