[MonoDevelop] SourceEditorDisplayBinding.patch

Lluis Sanchez lluis at ximian.com
Mon Aug 14 10:07:22 EDT 2006


I found another problem: if the file contains a method with a long name
or many parameters, the method combo grows to the size of the method,
and it can grow beyond the screen. 

El ds 12 de 08 del 2006 a les 20:02 +0100, en/na James Fitzsimons va
escriure:
> Hi all,
> Work commitments and a busy summer has kept me from finishing this but I
> have a new version ready.
> 
> On Mon, 2006-07-10 at 12:33 +0200, Lluis Sanchez wrote:
> > Thanks for the patch! A few comments:
> > 
> >       * You need to unsubscribe the ParseInformationChanged event in the
> >         Dispose method, or you'll keep references to all opened editors
> >         forever.
> >       * The idea of the 1 sec delay is to avoid updating the combo at
> >         every key press, but you are only delaying the updates, not
> >         avoiding them. That is, if you get a ParseInformationChanged
> >         event, and a combo update is already scheduled, you should not
> >         schedule a new update.
> >       * Instead of using a dictionary to store name->IClass relations,
> >         you can add a new data item in the list store of type Object,
> >         and directly store the IClass or IMember reference there.
> >       * We need the corresponding ChangeLog entries.
> I think I have addressed all of these issues now.
> 
> > And a couple of "nice to have":
> > 
> >       * It would be nice to add an option to the editor configuration
> >         panel to enable/disable the class bar.
> This one is done, not sure if it is the best place but you can turn the
> class bar on and off on the behaviour tab of the text editor properties.
> 
> >       * It would be nice to always show in the method combo the method
> >         on which the editor cursor is positioned.
> I spent some time looking into this one. I thought it would be easy but 
> unfortunately I am struggling to find a way to figure out what method the
> cursor is currently in. As I probably won't have much time to work on
> this in the immediate future I've added it to my TODO list and thought
> I'd submit a patch without it. Better to have some features than none at
> all right!
> 
> Cheers,
> James
> _______________________________________________
> Monodevelop-list mailing list
> Monodevelop-list at lists.ximian.com
> http://lists.ximian.com/mailman/listinfo/monodevelop-list



More information about the Monodevelop-list mailing list