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

Added Keypress event to catch Enter key on Login page #1198

Closed
wants to merge 20 commits into from
Closed

Added Keypress event to catch Enter key on Login page #1198

wants to merge 20 commits into from

Conversation

leigh-pointer
Copy link
Contributor

Currently when the [Enter] key is press no action is carried out. This pull request sets the focus to the User Name field and when the [Enter] key is pressed the Login procedure is executed.

@hishamco
Copy link
Contributor

Looks good but I'm not sure why there's 10 commits, can you rebase on dev

@hishamco
Copy link
Contributor

I think the No. of commits is too much according to what it needed, also I see some formatting has been added, the PR should be focus on what it should to make it easy to review

Thanks

@sbwalker
Copy link
Member

sbwalker commented Apr 1, 2021

@leigh-pointer could you resubmit this PR in a cleaner format so that it does not contain a lot of historical commits?

sbwalker added a commit that referenced this pull request Apr 2, 2021
Added Keypress event to catch Enter key on Login page #1198
@sbwalker
Copy link
Member

sbwalker commented Apr 2, 2021

Closing this PR as it was replaced by #1216

@sbwalker sbwalker closed this Apr 2, 2021
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.

5 participants