Skip to content

Conversation

@Annonnymmousss
Copy link
Contributor

@Annonnymmousss Annonnymmousss commented Jan 15, 2026

closes #3637

Screen.Recording.2026-01-15.at.10.16.16.AM.mov

@Keavon is it okay?

DragStart,
DragStop,
EditSelected,
Enter,
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please call this BeginEditing.

Copy link
Contributor Author

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?

Copy link
Member

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.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

renamed Enter to BeginEditing

@Keavon Keavon changed the title feat : With the Text tool active and a text layer selected, Enter should begin editing it Make the Text tool begin editing a selected layer by pressing Enter Jan 15, 2026
@Keavon
Copy link
Member

Keavon commented Jan 15, 2026

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.

@Annonnymmousss
Copy link
Contributor Author

Annonnymmousss commented Jan 15, 2026

that is because i have again and again sync the fork to graphite master i guess.
Letme fix it .

@Keavon
Copy link
Member

Keavon commented Jan 15, 2026

You shouldn't never need to close the PR.

@Annonnymmousss
Copy link
Contributor Author

i mistakenly close it while rebasing
sorry for that..
ill make another one

@Keavon
Copy link
Member

Keavon commented Jan 15, 2026

No, you never need to open a separate PR, you can just reopen this one.

@Annonnymmousss
Copy link
Contributor Author

okayy

@Annonnymmousss
Copy link
Contributor Author

@Keavon git history is fixed.

@Keavon
Copy link
Member

Keavon commented Jan 15, 2026

!build

@github-actions
Copy link

📦 Build Complete for 9e70bae
https://f6ccbab0.graphite.pages.dev

@Keavon
Copy link
Member

Keavon commented Jan 15, 2026

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?

@Keavon Keavon marked this pull request as draft January 15, 2026 06:24
@Annonnymmousss
Copy link
Contributor Author

sure looking into it

@Annonnymmousss
Copy link
Contributor Author

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

With the Text tool active and a text layer selected, Enter should begin editing it

2 participants