Skip to content
This repository has been archived by the owner on Jan 24, 2023. It is now read-only.

Improve focus & tabbing #3288

Merged
merged 7 commits into from
Dec 20, 2018
Merged

Improve focus & tabbing #3288

merged 7 commits into from
Dec 20, 2018

Conversation

richard-cox
Copy link
Contributor

@richard-cox richard-cox commented Dec 17, 2018

  • Add focus to first input field when opening a stepper
  • Ideally this should apply when navigating forward to a fresh step, however it's a real hack to implement
  • Ensured steppers with form have correct width
  • Removed app-event-detail-dialog (not used)
  • Fixed typo in newNonnAngularInterval
  • Other misc fixes (imports, private params, etc)
  • fixes Forms should auto-focus field and allow enter to submit #2449

- Fixed appFocus directive
- Set focus of confirm dialog to cancel instead of confirm
- Add/Edit App name - Add more space between async name taken check spinner and input field
- Tidied up imports
- Removed unused `AppEventDetailDialogComponentComponent`. Looks like a bad merge remnent
- Fixed `newNonnAngularInterval` typo
@cfdreddbot
Copy link

✅ Hey richard-cox! The commit authors and yourself have already signed the CLA.

@codecov
Copy link

codecov bot commented Dec 17, 2018

Codecov Report

Merging #3288 into v2-master will decrease coverage by 0.01%.
The diff coverage is 86.2%.

@@              Coverage Diff              @@
##           v2-master    #3288      +/-   ##
=============================================
- Coverage       70.9%   70.89%   -0.02%     
=============================================
  Files            642      641       -1     
  Lines          28204    28174      -30     
  Branches        6412     6405       -7     
=============================================
- Hits           19999    19973      -26     
+ Misses          8205     8201       -4

@KlapTrap KlapTrap merged commit c4122e8 into v2-master Dec 20, 2018
@KlapTrap KlapTrap deleted the form-autofocus-return branch December 20, 2018 15:29
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Forms should auto-focus field and allow enter to submit
3 participants