Skip to content

SF-3426 Auto focus note dialog input when it opens - #4059

Merged
Nateowami merged 1 commit into
masterfrom
fix/SF-3426-note-dialog-auto-focus
Aug 19, 2026
Merged

SF-3426 Auto focus note dialog input when it opens#4059
Nateowami merged 1 commit into
masterfrom
fix/SF-3426-note-dialog-auto-focus

Conversation

@Nateowami

@Nateowami Nateowami commented Aug 19, 2026

Copy link
Copy Markdown
Collaborator

This additionally steals focus from the editor, and I'm hoping that fixes the issues where the user can accidentally edit behind the dialog (but this change can also stand without that).


This change is Reviewable

@Nateowami
Nateowami marked this pull request as draft August 19, 2026 02:47
@codecov

codecov Bot commented Aug 19, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 81.10%. Comparing base (be216cd) to head (417ccf7).
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #4059   +/-   ##
=======================================
  Coverage   81.10%   81.10%           
=======================================
  Files         661      661           
  Lines       43060    43060           
  Branches     7033     7059   +26     
=======================================
  Hits        34925    34925           
  Misses       6977     6977           
  Partials     1158     1158           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

@Nateowami
Nateowami deployed to screenshot_diff August 19, 2026 03:08 — with GitHub Actions Active
@Nateowami Nateowami added testing complete Testing of PR is complete and should no longer hold up merging of the PR and removed ready to test labels Aug 19, 2026
@Nateowami
Nateowami marked this pull request as ready for review August 19, 2026 14:14

@RaymondLuong3 RaymondLuong3 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

@RaymondLuong3 reviewed 2 files and all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on Nateowami).

@RaymondLuong3
RaymondLuong3 force-pushed the fix/SF-3426-note-dialog-auto-focus branch from a52d5ab to 3065db7 Compare August 19, 2026 15:54
@Nateowami
Nateowami enabled auto-merge (squash) August 19, 2026 15:55
@Nateowami
Nateowami force-pushed the fix/SF-3426-note-dialog-auto-focus branch from 3065db7 to c64bcab Compare August 19, 2026 17:41
This additionally steals focus from the editor
@Nateowami
Nateowami force-pushed the fix/SF-3426-note-dialog-auto-focus branch from c64bcab to 417ccf7 Compare August 19, 2026 17:43
@Nateowami
Nateowami deployed to screenshot_diff August 19, 2026 17:50 — with GitHub Actions Active
@Nateowami
Nateowami merged commit f7468c8 into master Aug 19, 2026
28 checks passed
@Nateowami
Nateowami deleted the fix/SF-3426-note-dialog-auto-focus branch August 19, 2026 17:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

testing complete Testing of PR is complete and should no longer hold up merging of the PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants