Menu

#1350 AutoComplete::Select() and ignore case

Feature_Request
closed-fixed
Scintilla (833)
5
2012-04-20
2012-04-10
No

This is a "revival" of Bug 3455479 which was closed a while ago. I hereby turn in a patch to implement the feature that was discussed in the aforementioned bug.

Besides the patch, I also attached a screenshot in order to visualize the need for this feature request:

Currently, even if you tell autocompletion that case does not matter, it will respect it when selecting an item in the autocompletion list. This is very unfortunate as you can see in the attached screenshot: in our case insensitive language, the keyword "End" closes a block. For simplicity, the user enters the keyword without paying attention to the case. Autocompletion is configured to show up automatically upon entering characters.

You actually expect autocompletion to select "End" from the list; though, just in case you defined a variable that starts in lowercase exactly the same way, "End" is not the first hit. In my example, the local variable "ende" (German for "end") is selected -- even though "End" would be the perfect case-insensitive match!

The provided patch is actually pretty simple to understand. Feel free to rename identifiers just as you think they fit best into Scintilla nomenclature (plus I'm no native English speaker).

We patched Scintilla.h ... you might want to incorporate the changes into Scintilla.iface ... at short term, we didn't know how to do that correctly. The patch as-is works great for our needs. We would love to see it making into the official Scintilla version.

Discussion

  • Markus Nißl

    Markus Nißl - 2012-04-10

    Autocompletion does not select the perfect ci-match "End"

     
  • Markus Nißl

    Markus Nißl - 2012-04-10
     
  • Neil Hodgson

    Neil Hodgson - 2012-04-12
    • assigned_to: nobody --> nyamatongwe
     
  • Neil Hodgson

    Neil Hodgson - 2012-04-12

    The feature needs to be documented.

     
  • Markus Nißl

    Markus Nißl - 2012-04-12

    Documentation snippet for this feature

     
  • Markus Nißl

    Markus Nißl - 2012-04-12

    Documentation snippet uploaded.

     
  • Neil Hodgson

    Neil Hodgson - 2012-04-15

    Committed with name changes.

     
  • Marko Njezic

    Marko Njezic - 2012-04-16

    You forgot to change property names in documentation part. Also speaking about property names - "INSENSITITIVE" is not a word.

     
  • Neil Hodgson

    Neil Hodgson - 2012-04-16

    Updated property value names.

     
  • Neil Hodgson

    Neil Hodgson - 2012-04-20
    • status: open --> closed-fixed
     

Log in to post a comment.

MongoDB Logo MongoDB