This looks quite good to me. I'd like some feedback from Thomas Caswell, who's looked at this part of the code much more than I have lately, but assuming he's ok, you have my go ahead to start the implementation.
Cheers, Mike On 01/08/2014 10:55 AM, Federico Ariza wrote: > Hello everybody. > > I just created a new > MEPhttps://github.com/matplotlib/matplotlib/wiki/Mep22 that is open > for discussion and destruction. > > Background: > > In my way to get a MultiFigureManager PR > https://github.com/matplotlib/matplotlib/pull/2465 we came to the > conclusion that the work has to be splitted in smaller PRs. > > One of the goals of my PR was to "Be able to easily modify the toolbar > adding and removing toolitems." This mostly breakes the existing > infrastructure of Toolbar. > > Working on a smaller PR https://github.com/matplotlib/matplotlib/pull/2557 > to address this point, it became clear that a more radical change was > needed to have a clean interface for "User Interaction tools" > > To make it easier to discuss, I created the MEP22 proposing a > separation of powers (insert political joke here). It is open to all > kind of modifications so don't be shy. > > As reference, there are a couple of previous PRs relating to this > https://github.com/matplotlib/matplotlib/pull/1849 and > https://github.com/matplotlib/matplotlib/pull/2557 > Both of these don't address the problem of reconfiguration > > Thank you > Federico > -- _ |\/|o _|_ _. _ | | \.__ __|__|_|_ _ _ ._ _ | ||(_| |(_|(/_| |_/|(_)(/_|_ |_|_)(_)(_)| | | http://www.droettboom.com ------------------------------------------------------------------------------ CenturyLink Cloud: The Leader in Enterprise Cloud Services. Learn Why More Businesses Are Choosing CenturyLink Cloud For Critical Workloads, Development Environments & Everything In Between. Get a Quote or Start a Free Trial Today. http://pubads.g.doubleclick.net/gampad/clk?id=119420431&iu=/4140/ostg.clktrk _______________________________________________ Matplotlib-devel mailing list Matplotlib-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/matplotlib-devel
