Ticket #3646 (new enhancement)

Opened 3 months ago

Last modified 2 months ago

[with patch, needs work] Implement paren matching

Reported by: boothby Assigned to: boothby
Priority: minor Milestone: sage-3.2
Component: notebook Keywords:
Cc: was, tclemans, yiqiang

Description

People have been complaining about a lack of paren matching in the notebook. Do something.

Attachments

3646-paren-matching-all.patch (6.7 kB) - added by boothby on 07/29/2008 10:20:54 AM.

Change History

07/11/2008 03:53:01 PM changed by boothby

  • cc set to was, tclemans, yiqiang.
  • summary changed from Implement paren matching to [with patch, needs review] Implement paren matching.

07/11/2008 08:29:05 PM changed by boothby

  • milestone set to sage-3.0.6.

07/18/2008 01:41:11 PM changed by jhpalmieri

On my intel mac: works for me in Safari, and works in Firefox but noisily: it beeps whenever I hit ctrl-0 (whether it matches a parenthesis or not).

07/29/2008 10:20:54 AM changed by boothby

  • attachment 3646-paren-matching-all.patch added.

07/29/2008 10:22:55 AM changed by boothby

I just discovered quilt! 4 patches all rolled into one.

07/29/2008 10:23:55 AM changed by boothby

Oh, and this patch addresses jhpalmieri's issue.

07/29/2008 10:33:23 AM changed by was

  • summary changed from [with patch, needs review] Implement paren matching to [with patch, needs work] Implement paren matching.

REVIEW:

  • Looks very good but does *not* address jhpalmieri's issue successfully. I can replicate the bing, which is OK, but pressing ctrl-] does not get recognized at all. I'm using a macbookpro.