Re: minor rclick issue

2008-06-16 Thread Edward K. Ream
On Jun 16, 8:51 am, Edward K. Ream [EMAIL PROTECTED] wrote: You were wise not to attempt a specific fix. Actually, the fix involved creating new callbacks in several plugins where there are calls of the form: b.configure(command=command) In other words, a common pattern is: b =

Re: minor rclick issue

2008-06-14 Thread bobjack
On Jun 13, 12:22 pm, Edward K. Ream [EMAIL PROTECTED] wrote: On Fri, Jun 13, 2008 at 3:58 AM, bobjack [EMAIL PROTECTED] wrote: The same thing applies to menu.add_radiobutton and menu.add_checkbutton. Another potential source of problems is in button commands, especially as some

Re: minor rclick issue

2008-06-12 Thread Edward K. Ream
On Thu, Jun 12, 2008 at 8:14 AM, bobjack [EMAIL PROTECTED] wrote: On Jun 11, 3:51 pm, Edward K. Ream [EMAIL PROTECTED] wrote: On Jun 11, 8:44 am, Edward K. Ream [EMAIL PROTECTED] wrote: The little mystery of the missing redraw has been solved: cleo uses aMenu.add_command to invoke

Re: minor rclick issue

2008-06-12 Thread Edward K. Ream
On Jun 12, 10:07 am, Edward K. Ream [EMAIL PROTECTED] wrote: rClick is broken by this. Sorry about that. I could have sworn I checked rClick specifically. I'll fix this today. No need to revert anything. The fix is on the trunk. All unit tests pass. Please let me know how it works

Re: minor rclick issue

2008-06-11 Thread bobjack
On Jun 10, 5:26 pm, Terry Brown [EMAIL PROTECTED] wrote: With rclick enabled I don't get redraws when I change cleo priority icons, there is an redraw with rclick disabled. This isn't so! The problem occurs regardless of whether rclick is enabled or not. The problem is in Edwards new redraw

Re: minor rclick issue

2008-06-11 Thread bobjack
Yipee! I can now commit to bzr again! Doh! Now I have to go to work :-( --~--~-~--~~~---~--~~ You received this message because you are subscribed to the Google Groups leo-editor group. To post to this group, send email to leo-editor@googlegroups.com To

Re: minor rclick issue

2008-06-11 Thread bobjack
No. On Jun 11, 11:26 am, Edward K. Ream [EMAIL PROTECTED] wrote: Some of yesterday's changes (augmented with today's fix of c.tag_bind) may solve the problem. Those changes were intended to guarantee that c.outerUpdate gets called after all events get handled. As a special case, I suppose,

Re: minor rclick issue

2008-06-11 Thread Edward K. Ream
On Jun 11, 8:22 am, Edward K. Ream [EMAIL PROTECTED] wrote: On second thought, c.redraw_now indeed probably should call c.outerUpdate. The little mystery of the missing redraw has been solved: cleo uses aMenu.add_command to invoke several commands. This is another way into code.

Re: minor rclick issue

2008-06-11 Thread Edward K. Ream
On Jun 11, 8:44 am, Edward K. Ream [EMAIL PROTECTED] wrote: The little mystery of the missing redraw has been solved: cleo uses aMenu.add_command to invoke several commands. This is another way into code. add_command is used by several plugins, so I suppose changing m.add_command(...)

Re: minor rclick issue

2008-06-11 Thread Edward K. Ream
On Wed, Jun 11, 2008 at 12:07 PM, Terry Brown [EMAIL PROTECTED] wrote: I just committed cleo.py with a couple of c = self.c needed for c.add_command to work. Thanks. Edward --~--~-~--~~~---~--~~ You received this message because you are subscribed to the

minor rclick issue

2008-06-10 Thread Terry Brown
With rclick enabled I don't get redraws when I change cleo priority icons, there is an redraw with rclick disabled. Cheers -Terry --~--~-~--~~~---~--~~ You received this message because you are subscribed to the Google Groups leo-editor group. To post to this