Skip to content
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

STCOM-1288: Focus the last modified query field when opening <AdvancedSearch>. #2278

Merged
merged 2 commits into from May 7, 2024

Conversation

Dmytro-Melnyshyn
Copy link
Contributor

@Dmytro-Melnyshyn Dmytro-Melnyshyn commented May 2, 2024

Description

Before this PR, after opening the advanced search modal window, the focus was on the first query field.

After this PR is merged, the focus will be on the last modified query field made in the advanced search modal, or, if the previously modified field wasn't a query, the focus will be shifted to the last non-empty query field.

The isCursorAtEnd property will place the cursor at the end of the field value.

Tests will pass after this PR #2277 is merged.

Related PRs

#2277

Issues

STCOM-1288

Screencast

2024-05-02_22h29_58.mp4

Copy link

github-actions bot commented May 2, 2024

Jest Unit Test Statistics

0 tests   0 ✔️  0s ⏱️
0 suites  0 💤
0 files    0

Results for commit e6cccc8.

♻️ This comment has been updated with latest results.

@Dmytro-Melnyshyn Dmytro-Melnyshyn changed the title STCOM-1288: Put focus on the last modified non-empty field after closing and opening <AdvancedSearch>, otherwise on the last query field. STCOM-1288: Focus the last modified field when opening <AdvancedSearch>. May 3, 2024
@Dmytro-Melnyshyn Dmytro-Melnyshyn changed the title STCOM-1288: Focus the last modified field when opening <AdvancedSearch>. STCOM-1288: Focus the last modified query field when opening <AdvancedSearch>. May 3, 2024
Copy link

github-actions bot commented May 3, 2024

BigTest Unit Test Statistics

       1 files         1 suites   13s ⏱️
1 450 tests 1 444 ✔️ 6 💤 0
1 452 runs  1 446 ✔️ 6 💤 0

Results for commit e6cccc8.

♻️ This comment has been updated with latest results.

@Dmytro-Melnyshyn Dmytro-Melnyshyn marked this pull request as ready for review May 3, 2024 06:32
Copy link

sonarcloud bot commented May 7, 2024

@Dmytro-Melnyshyn Dmytro-Melnyshyn merged commit c075e40 into master May 7, 2024
6 checks passed
@Dmytro-Melnyshyn Dmytro-Melnyshyn deleted the STCOM-1288 branch May 7, 2024 15:56
github-actions bot pushed a commit that referenced this pull request May 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants