Skip to content

Fix LT-22475: The tab order is wrong for "Create and Edit" button#828

Merged
jtmaxwell3 merged 2 commits intomainfrom
LT-22475
Apr 10, 2026
Merged

Fix LT-22475: The tab order is wrong for "Create and Edit" button#828
jtmaxwell3 merged 2 commits intomainfrom
LT-22475

Conversation

@jtmaxwell3
Copy link
Copy Markdown
Collaborator

@jtmaxwell3 jtmaxwell3 commented Apr 10, 2026

This fixes https://jira.sil.org/browse/LT-22475.


This change is Reviewable

Copy link
Copy Markdown
Contributor

@papeh papeh left a comment

Choose a reason for hiding this comment

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

I think a better place to set the tab order would be in the resx file.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 10, 2026

NUnit Tests

    1 files  ±0      1 suites  ±0   6m 16s ⏱️ -9s
4 103 tests ±0  4 032 ✅ +1  71 💤 ±0  0 ❌  - 1 
4 112 runs  ±0  4 041 ✅ +1  71 💤 ±0  0 ❌  - 1 

Results for commit d720a85. ± Comparison against base commit 9e3106c.

♻️ This comment has been updated with latest results.

@jtmaxwell3
Copy link
Copy Markdown
Collaborator Author

@papeh: We can't do it in the resx file since the Create and Edit button is only included in some contexts. When it is added, the downstream buttons have to have their TabIndex values incremented.

@papeh
Copy link
Copy Markdown
Contributor

papeh commented Apr 10, 2026

Tab orders are allowed to be nonconsecutive, so hiding the button should not cause a problem.

@jtmaxwell3 jtmaxwell3 enabled auto-merge (squash) April 10, 2026 18:29
@jtmaxwell3 jtmaxwell3 merged commit ee2c78a into main Apr 10, 2026
6 checks passed
@jtmaxwell3 jtmaxwell3 deleted the LT-22475 branch April 10, 2026 18:46
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.

2 participants