镜像站点 · 本页由第三方 GitHub 只读镜像提供,非 GitHub 官方站点,不接受任何登录或凭据输入。前往 github.com
Skip to content

✨ Improve Design Implementation for Find in File #3

Description

@austincondiff

We need the Find and Replace bar to look closer to Xcode's.

image

Top: Xcode | Bottom: CodeEdit

Activity

  1. changed the title [-]Improve Design for Find in File[/-] [+]✨ Improve Design for Find in File[/+] on Oct 6, 2022
  2. changed the title [-]✨ Improve Design for Find in File[/-] [+]✨ Improve Design Implementation for Find in File[/+] on Oct 6, 2022
  3. austincondiff commented on Feb 16, 2023

    @austincondiff
    ContributorAuthor

    Are there any standard input styles we can use for this or will we need to create this custom?

  4. thecoolwinter commented on Feb 17, 2023

    @thecoolwinter
    Contributor

    The one in CE right now is the default NSTextFinder, we'll need to create a custom one.

  5. luah5 commented on Apr 11, 2023

    @luah5
    Member

    @thecoolwinter Is it in CE, CETV or STTextView? It seems like STTextView (prove me wrong) has this file for find/replace, meaning that we need to override this in base CE (again, i might be wrong).

  6. thecoolwinter commented on Apr 11, 2023

    @thecoolwinter
    Contributor

    The file you sent is extending STTextView to support NSTextFinder, not implementing a custom finder.

  7. luah5 commented on Apr 11, 2023

    @luah5
    Member

    Ok i get it, so you're suggesting that we create a custom Find/Replace in CETV. But, considering that the find/replace is in this file in CE shouldn't the custom view be created in CE (or am i just making a incorrect point), @thecoolwinter?

  8. thecoolwinter commented on Apr 11, 2023

    @thecoolwinter
    Contributor

    That command sends a showFindPanel message down to the first active view that can respond to it. If you go into CE rn and press command F in the editor you'll get a find menu.

    This issue is saying the UI part for entering text to find should be custom. Because that finder view needs to be installed on the STTextView, it has to be implemented in CETV where that view can be accessed.

  9. added a commit that references this issue on Feb 12, 2024
  10. added a commit that references this issue on Apr 8, 2025
  11. moved this from 📋 Todo to 🏁 Complete in CodeEdit Projecton Apr 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

enhancementNew feature or request

Type

No type

Projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions