SourceEditorDisplayBinding.patch

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-listlists.ximian.com
> http://lists.ximian.com/mailman/listinfo/monodevelop-li
st

_______________________________________________
Monodevelop-list mailing list
Monodevelop-listlists.ximian.com
http://lists.ximian.com/mailman/listinfo/monodevelop-li
st