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

[Firefox] Unable to navigate with arrow keys inside table #3263

Closed
f1ames opened this issue Feb 20, 2019 · 4 comments
Closed

[Firefox] Unable to navigate with arrow keys inside table #3263

f1ames opened this issue Feb 20, 2019 · 4 comments
Labels
browser:firefox package:table pending:feedback This issue is blocked by necessary feedback. resolution:duplicate This issue is a duplicate of another issue and was merged into it. type:bug This issue reports a buggy (incorrect) behavior.

Comments

@f1ames
Copy link
Contributor

f1ames commented Feb 20, 2019

Is this a bug report or feature request? (choose one)

🐞 Bug report

💻 Version of CKEditor

Latest master and v11.1.1

📋 Steps to reproduce

  1. Go to https://ckeditor.com/docs/ckeditor5/latest/examples/builds/document-editor.html.
  2. Focus table inside editor.
  3. Try to navigate with arrow keys between table cells.

✅ Expected result

It is possible to navigate with arrow keys between cells.

❎ Actual result

Nothing happens.

📃 Other details that might be useful

Tested with Firefox 65.0.1 (64-bit) on macOS 10.13.6 (17G65).

It works fine on Chrome. From what I have found in the https://github.com/ckeditor/ckeditor5-table/issues/108#issuecomment-419114747 this behaviour is left for the browser to handle so I'm not sure if it worked in Firefox earlier (doesn't work with v11.1.1). It works fine when table is placed inside contenteditable (tested without CKEditor).

@jodator
Copy link
Contributor

jodator commented Feb 20, 2019

@f1ames That worked on Firefox some time ago - at least when the plugin was created. There was even mismatch in that behavior between browsers. In Firefox you could use arrow up/down to go to previous/next row. On Chrome you'd go to previous/next cell hence the #3227 issue was created.

ps.: I liked the FF implementation more.

@jodator
Copy link
Contributor

jodator commented Feb 20, 2019

@f1ames
Copy link
Contributor Author

f1ames commented Feb 20, 2019

I have checked with Firefox 59, 62, 63 (on Browserstack) with 11.1.1 and 11.0.1 and it doesn't work there either.

@Reinmar
Copy link
Member

Reinmar commented Feb 20, 2019

I don't think this ever worked. There were differences between browsers from the very beginning and I think it was one of them.

In any case, let's keep all those navigation issues in one place. Merging to https://github.com/ckeditor/ckeditor5-table/issues/108

@Reinmar Reinmar closed this as completed Feb 20, 2019
@mlewand mlewand transferred this issue from ckeditor/ckeditor5-table Oct 9, 2019
@mlewand mlewand added browser:firefox pending:feedback This issue is blocked by necessary feedback. resolution:duplicate This issue is a duplicate of another issue and was merged into it. type:bug This issue reports a buggy (incorrect) behavior. package:table labels Oct 9, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
browser:firefox package:table pending:feedback This issue is blocked by necessary feedback. resolution:duplicate This issue is a duplicate of another issue and was merged into it. type:bug This issue reports a buggy (incorrect) behavior.
Projects
None yet
Development

No branches or pull requests

4 participants