-
Notifications
You must be signed in to change notification settings - Fork 54
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
Move cursor with new characters in Scribe commands #286
Comments
Hey @andrewtavis I would like to work on this issue 😊 |
Thanks for taking it up @smalik2811 Feel free to ask any queries you might have here :) |
Thanks for your vested interest in the project, @smalik2811! :) |
Hi @andrewtavis , Thanks for your kind words! I'm really enjoying contributing to this project. It's been a fantastic learning experience, and I'm grateful for the support and guidance I've received from the community. I'm excited to continue contributing and see where this project goes! Best regards, |
Feel free to start this one at your convenience, @smalik2811! Looking forward to getting this up and running so the command UX can start taking the next level :) Here's the function that we use in iOS, btw: func insertPriorToCursor(char: String) -> String {
return substring(toIdx: count - 1) + char + commandCursor
} Let us know if there are any questions! |
Thanks for sharing the iOS code snippet. It's helpful to see how that's being handled. I'll use this as a starting point and let you know, if I run into any questions. |
Hi @andrewtavis I have been working on the bug and the desired functionality has been achieved. Draft.Cursor.webmHowever I am facing 2 issues related to the UI.
Could you please guide about on how to deal with these issues? |
Would it be helpful to adjust the padding of the text dynamically according to whether the user types content or not ? If I am thinking right the space is coming from the text padding that is being continuously extended as we type more. |
Would be great if @angrezichatterbox's suggestion can work! Let us know and we can go from there. Great work, @smalik2811! |
Hi @angrezichatterbox, Thankyou for the help. However I tried typing the command character in the notepad it turns out, it is the character "\u2502" that contains the space as part of it. I explored some other characters to replace it and found the new character Line 1 is the old character. Here is the latest preview: Fix.Presentation.mp4 |
Terms
Description
In the current state of Scribe-Android the cursor for the command bar appears after the prompt for the command and before the placeholder text. When the user starts typing the placeholder text disappears as it should, but the cursor is behind the new text. Ideally the command bar cursor should function like a normal cursor and should be the right most character within the command bar text input. See the video below for how it works in Scribe-iOS:
Simulator.Screen.Recording.-.iPhone.16.Pro.-.2025-01-02.at.00.20.41.mp4
Contribution
Happy to assist with work on this and review it once it's finished! 😊
The text was updated successfully, but these errors were encountered: