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

Editor: Is Forcing focus changes #4335

Closed
tmcconechy opened this issue Sep 2, 2020 · 1 comment · Fixed by #4493
Closed

Editor: Is Forcing focus changes #4335

tmcconechy opened this issue Sep 2, 2020 · 1 comment · Fixed by #4493
Assignees
Labels
focus: accessibility Main focus is for accessibility team: landmark For Landmark issues type: bug 🐛 [3] Velocity rating (Fibonacci)

Comments

@tmcconechy
Copy link
Member

Describe the bug
A clear and concise description of what the bug is.

To Reproduce
Steps to reproduce the behavior:

  1. Go to https://master-enterprise.demo.design.infor.com/components/editor/example-index.html
  2. Tab to the B button
  3. Hit Enter
  4. See error - the focus is put into the editor

Expected behavior
If possible when keying onto these buttons the focus should not switch into the editor. When contents are selected we have to be careful it still selects. I do have a question on that case if its ok to change state on click/selection?

Version

  • ids-enterprise: [e.g. v4.32]

Platform

  • Chrome / Mac (Any)

Additional context
https://jira.lawson.com/browse/LMCLIENT-31969

@tmcconechy tmcconechy added [3] Velocity rating (Fibonacci) focus: accessibility Main focus is for accessibility team: landmark For Landmark issues type: bug 🐛 labels Sep 2, 2020
@tmcconechy tmcconechy added this to To do in Enterprise 4.34.x (Oct 2020) Sprint via automation Sep 2, 2020
@deep7102 deep7102 self-assigned this Oct 9, 2020
@tmcconechy tmcconechy moved this from To do to In progress in Enterprise 4.34.x (Oct 2020) Sprint Oct 9, 2020
@tmcconechy tmcconechy moved this from In progress to Ready for QA (beta) in Enterprise 4.34.x (Oct 2020) Sprint Oct 14, 2020
@janahintal
Copy link
Contributor

janahintal commented Oct 15, 2020

QA Passed.
Observation: When I tab to https://master-enterprise.demo.design.infor.com/components/editor/example-index.html
the focus is only going thru Normal Text field in the toolbar and not on the remaining buttons/fields. @tmcconechy may I know if this is also expected?

image

@janahintal janahintal moved this from Ready for QA (beta) to Done in Enterprise 4.34.x (Oct 2020) Sprint Oct 15, 2020
@janahintal janahintal moved this from Done to Ready for QA (beta) in Enterprise 4.34.x (Oct 2020) Sprint Oct 15, 2020
@janahintal janahintal moved this from Ready for QA (beta) to Done in Enterprise 4.34.x (Oct 2020) Sprint Oct 19, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
focus: accessibility Main focus is for accessibility team: landmark For Landmark issues type: bug 🐛 [3] Velocity rating (Fibonacci)
Projects
No open projects
Development

Successfully merging a pull request may close this issue.

3 participants