On Sun, Nov 2, 2008 at 8:41 PM, Edward K. Ream <[EMAIL PROTECTED]> wrote:

> Rev 1441 contains the code that connects headline widgets with Leo's
> core.  This involved some major hacks to the eventFilter, but in the
> end the hacks turned out to be fairly clean, if that makes any
> sense :-)

I don't know if this is the cause, but ctrl+I no longer works
correctly. It inserts 2 "NewHeadline" nodes and starts editing the
last one. I'm not quite sure how necessary connecting the headline
editing to core is in the first place, I was also able to crash leo by
pressing up after ctrl+I. It would be simpler and less bug-prone to
just use the default qt functionality for headline editing, with the
slight drawback that ctrl+a / ctrl+e would not work. That's a small
price to pay for having yet another area that does not depend on code
in leo side at all...

Unrelated issues:

- Clicking on the expand plus sign sometimes scrolls the tree pane
right (try it by making the horizontal area of the tree smaller by
dragging the border).

- The default font should probably be smaller, for better "first impression".

- On a more controversial note, scintilla should probably be the
default editor, again for better first impression. QTextEdit is still
too slow to use comfortably (IMHO).

- The scintilla widget should choose the lexer according to the
programming language (on @nocolor, @killcolor...).

-- 
Ville M. Vainio
http://tinyurl.com/vainio

--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups 
"leo-editor" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to [EMAIL PROTECTED]
For more options, visit this group at 
http://groups.google.com/group/leo-editor?hl=en
-~----------~----~----~----~------~----~------~--~---

Reply via email to