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
> 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!
> Monodevelop-list mailing list
> Monodevelop-list at lists.ximian.com
More information about the Monodevelop-list