Skip to content

fix(compose): focus body textarea for replies#1786

Open
Keegil wants to merge 1 commit intorunbox:masterfrom
Keegil:fix/issue-910
Open

fix(compose): focus body textarea for replies#1786
Keegil wants to merge 1 commit intorunbox:masterfrom
Keegil:fix/issue-910

Conversation

@Keegil
Copy link
Copy Markdown

@Keegil Keegil commented Mar 25, 2026 •
edited
Loading

Summary

  • move plain-text reply focus into ngAfterViewInit so the body textarea exists before focusing it
  • focus the reply body and place the caret at the start for unsaved reply drafts
  • add a focused component spec covering plain-text reply focus

Testing

  • ./node_modules/.bin/tsc -p src/tsconfig.spec.json --noEmit
  • ./node_modules/.bin/eslint src/app/compose/compose.component.ts src/app/compose/compose.component.spec.ts (existing no-explicit-any warnings in compose.component.ts only)
  • ./node_modules/.bin/ng test --watch=false --browsers=FirefoxHeadless --include src/app/compose/compose.component.spec.ts (bundle build succeeds; browser launch fails because FirefoxHeadless is not installed in this environment)

Closes #910

@Keegil
fix(compose): focus body textarea for replies
63e589c 
Move reply textarea focus into the post-render lifecycle so the body field exists before it is focused.
Add a focused component spec that covers opening a plain text reply.

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

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

When replying to a message the cursor is not placed in the text area of the body of the email

1 participant

@Keegil