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

Confirmation by pressing Space on confirm screens #784

Merged
merged 4 commits into from
Oct 5, 2020

Conversation

utchoang
Copy link

@utchoang utchoang commented Oct 2, 2020

@utchoang
Copy link
Author

utchoang commented Oct 2, 2020

@blueorangutan package

@blueorangutan
Copy link

@utchoang a Jenkins job has been kicked to build primate packages. I'll keep you posted as I make progress.

@blueorangutan
Copy link

Packaging result: ✔️centos ✔️debian ✔️archive.
QA: http://primate-qa.cloudstack.cloud:8080/client/pr/784 (JID-3555)

@rohityadavcloud
Copy link
Member

Good to have, could this cause any regressions?

@utchoang
Copy link
Author

utchoang commented Oct 5, 2020

@rhtyd I think it won't cause a regression. Thanks.

@davidjumani
Copy link
Contributor

@utchoang I think what @borisstoyanov meant was that hitting space toggles the value of the first button / checkbox / switch. An autofocus on the first form item would be an appropriate fix

@utchoang
Copy link
Author

utchoang commented Oct 5, 2020

@blueorangutan package

@blueorangutan
Copy link

@utchoang a Jenkins job has been kicked to build primate packages. I'll keep you posted as I make progress.

@blueorangutan
Copy link

Packaging result: ✔️centos ✔️debian ✔️archive.
QA: http://primate-qa.cloudstack.cloud:8080/client/pr/784 (JID-3560)

@rohityadavcloud rohityadavcloud added this to the 1.0-GA milestone Oct 5, 2020
Copy link
Contributor

@davidjumani davidjumani left a comment

Choose a reason for hiding this comment

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

LGTM. Need to check whether changes might be needed in other custom forms
Thanks @utchoang

@rohityadavcloud rohityadavcloud merged commit cfa0c45 into apache:master Oct 5, 2020
@utchoang utchoang deleted the feature/fix-issues-781 branch October 27, 2020 07:31
weizhouapache pushed a commit that referenced this pull request Jan 19, 2021
* submit button when pressing Space on confirm modal

* removed console

* add auto-focus into switch & remove keyup function

* revert message on storagejs
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BUG] Confirmation by pressing Space on confirm screens
4 participants