Mantis Bugtracker
Main | My View | View Issues | Change Log | Docs
  

Viewing Issue Simple Details [ Jump to Notes ] [ View Advanced ] [ Issue History ] [ Print ]
ID Category Severity Reproducibility Date Submitted Last Update
0001618 [Squeak] Kernel minor always 08-05-05 18:16 08-05-05 18:25
Reporter KenCausey View Status public  
Assigned To
Priority normal Resolution open  
Status new   Product Version
Summary 0001618: [ENH] ParagraphEditor-iSearch-rkris
Description rkris@kriyative.net:

"Change Set: ParagraphEditor-iSearch-rkris
Date: 28 July 2004
Author: Ram Krishnan

Adds incremental search (similar to emacs but forward search only) to
ParagraphEditor. This changeset remaps the Ctl-S key, which used to
invoke the #search: selector, to the #incrementalSearch: selector. While
in the incrementalSearch mode, Ctl-W can be used to extend the search
term and Backspace to shorten it. This changeset has only been tested
with Squeak3.7. It will fail in Squeak3.6 as there seem to be some
conflicting changes in ParagraphEditor between the two revisions."
Additional Information
Attached Files  ParagraphEditor-iSearch-rkris.cs.gz [^] (4,652 bytes) 08-05-05 18:16
 ParagraphEditor-iSearch-rkris.2.cs.gz [^] (5,152 bytes) 08-05-05 18:24

- Relationships

- Notes
(0002160 - 443 - 551 - 588 - 588 - 588 - 588)
KenCausey
08-05-05 18:20

 Samuel Tardieu <sam@rfc1149.net>:

"The search works fine. However, I find that a non-match is misleading.
If for example you search for "abd" in a text containing "xxxxabcxxx",
"a" will be first highlighted, then "ab", then nothing (which is
not distinct from the default non-search mode). "ab" should be
highlighted in another color (red?) if possible, and no character should
be added to the search string when there are no matches."
 
(0002161 - 1311 - 1574 - 1574 - 1574 - 1574 - 1574)
KenCausey
08-05-05 18:22

 rkris@kriyative.net:

"Thanks for the feedback. The non-match behavior is definitely
misleading. I've pushed another changeset to the squeak-dev list which
should address that issue. I also briefly looked at how to visually
indicate whether or not a ParagraphEditor instance is in iSearch mode.
That will require some changes to the class NewParagraph."

"Change Set: ParagraphEditor-iSearch-rkris
Date: 14 August 2004
Author: Ram Krishnan

Adds incremental search (similar to emacs but forward search only) to
ParagraphEditor. This changeset remaps the Ctl-S key, which used to
invoke the #search: selector, to the #incrementalSearch: selector. While
in the incrementalSearch mode, Ctl-W can be used to extend the search
term and Backspace to shorten it. This changeset has only been tested
with Squeak3.7. It will fail in Squeak3.6 as there seem to be some
conflicting changes in ParagraphEditor between the two revisions.

This changeset improves the behavior to be more compatible with the
Emacs incremental-search functionality, such as:

    * The current `point' is saved when entering incremental search and
restored when exiting via Ctl-G
    
    * A mismatch leaves the point and mark at the last matched region of
text"

(attaching ParagraphEditor-iSearch-rkris.2.cs.gz)
 
(0002162 - 93 - 93 - 93 - 93 - 93 - 93)
KenCausey
08-05-05 18:25

 I seperately loaded each changeset successfully into 3.8-6665-basic but did not test further.
 

- Issue History
Date Modified Username Field Change
08-05-05 18:16 KenCausey New Issue
08-05-05 18:16 KenCausey File Added: ParagraphEditor-iSearch-rkris.cs.gz
08-05-05 18:20 KenCausey Note Added: 0002160
08-05-05 18:22 KenCausey Note Added: 0002161
08-05-05 18:24 KenCausey File Added: ParagraphEditor-iSearch-rkris.2.cs.gz
08-05-05 18:25 KenCausey Note Added: 0002162

Mantis 1.0.8[^]
Copyright © 2000 - 2007 Mantis Group
43 total queries executed.
30 unique queries executed.
Powered by Mantis Bugtracker