[MonoDevelop] SourceEditorDisplayBinding.patch

[Lluis Sanchez]

Hi!

I tried your patch and I found some issues:

      * It crashes when opening a file which defines a delegate.
      * It also crashes if you add a delegate to a file.
      * I got an exception trying to open an AssemblyInfo.cs file (maybe
        because there isn't any class?).
      * If a member is very long (e.g. a method with lots of
        parameters), the combo grows beyond the screen.
      * The combos are not updated when moving from method to method in
        the editor.

Lluis.

El ds 03 de 03 del 2007 a les 17:07 +0000, en/na James Fitzsimons va
escriure:
> Hi Lluis and list,
> 
> I've been using Monodevelop quite a lot recently and decided to revive
> this patch I never quite finished last year.
> 
> Lluis, I think I have addressed all the issues you mentioned last time
> now. I've been using it a little this afternoon and it seems to be
> working pretty well.
> 
> Hopefully this time we'll get it committed!
> 
> Cheers,
> James
> 
> On Mon, 2006-08-14 at 13:19 +0200, Lluis Sanchez wrote:
> > 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.
> > 
> > The patch looks great!
> > I had some crashes though, when trying to open a file not belonging to a
> > project, or when opening a nemerle file.
> > 
> > > 
> > > > 
> > > >       * 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.
> > 
> > Maybe a better place is the General section. Behavior is more for the
> > text editing behavior.
> > 
> > > 
> > > >       * 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!
> > 
> > It's easy. Each method has a body region, so you only need to loop
> > through all methods and find the one which contains the cursor position.
> > 
> > Lluis.
> > 
> > 
> _______________________________________________
> Monodevelop-list mailing list
> Monodevelop-list at lists.ximian.com
> http://lists.ximian.com/mailman/listinfo/monodevelop-list