-
-
Notifications
You must be signed in to change notification settings - Fork 1k
Make the Text tool begin editing a selected layer by pressing Enter #3640
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
base: master
Are you sure you want to change the base?
Make the Text tool begin editing a selected layer by pressing Enter #3640
Conversation
| DragStart, | ||
| DragStop, | ||
| EditSelected, | ||
| Enter, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please call this BeginEditing.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You mean the enter message variant right?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That's the line I am commenting about, yes.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
renamed Enter to BeginEditing
|
You've done something to really mess up the Git history, I'm not exactly sure what, but please rebase onto the current upstream master so you only have one commit per actual change and remove all the merge commits. Then force push to your fork's branch. |
|
that is because i have again and again sync the fork to graphite master i guess. |
49d01ff to
39849c9
Compare
|
You shouldn't never need to close the PR. |
|
i mistakenly close it while rebasing |
|
No, you never need to open a separate PR, you can just reopen this one. |
|
okayy |
|
@Keavon git history is fixed. |
|
!build |
|
|
After pressing Enter to edit the text, as soon as you type something, you will find that a newline has been added to the text, which is of course not desired. Can you look into a fix for that bug? |
|
sure looking into it |
|
can you a send a video demo of this bug(the newline) as I am unable to reproduce this. Screen.Recording.2026-01-15.at.3.20.40.PM.mov |
closes #3637
Screen.Recording.2026-01-15.at.10.16.16.AM.mov
@Keavon is it okay?