Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

✨ Improve Design Implementation for Find in File #3

Open
austincondiff opened this issue Sep 29, 2022 · 6 comments
Open

✨ Improve Design Implementation for Find in File #3

austincondiff opened this issue Sep 29, 2022 · 6 comments
Assignees
Labels
enhancement New feature or request

Comments

@austincondiff
Copy link
Contributor

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

image

Top: Xcode | Bottom: CodeEdit
@austincondiff austincondiff changed the title Improve Design for Find in File ✨ Improve Design for Find in File Oct 6, 2022
@austincondiff austincondiff changed the title ✨ Improve Design for Find in File ✨ Improve Design Implementation for Find in File Oct 6, 2022
@lukepistrol lukepistrol added the enhancement New feature or request label Nov 22, 2022
@austincondiff
Copy link
Contributor Author

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

@austincondiff austincondiff self-assigned this Feb 16, 2023
@thecoolwinter
Copy link
Contributor

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

@luah5
Copy link
Member

luah5 commented Apr 11, 2023

@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).

@thecoolwinter
Copy link
Contributor

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

@luah5
Copy link
Member

luah5 commented Apr 11, 2023

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?

@thecoolwinter
Copy link
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.

@austincondiff austincondiff transferred this issue from CodeEditApp/CodeEditSourceEditor Dec 20, 2023
EstebanBorai pushed a commit that referenced this issue Feb 12, 2024
[feat]: Added java language support
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: 📋 Todo
Development

No branches or pull requests

4 participants