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

Triple click merge with the next cell #129

Open
Goaman opened this issue Mar 17, 2020 · 2 comments
Open

Triple click merge with the next cell #129

Goaman opened this issue Mar 17, 2020 · 2 comments

Comments

@Goaman
Copy link
Contributor

Goaman commented Mar 17, 2020

Is it the proper behavior that we want?
https://youtu.be/b2hx6G9Vf9I
@dmo-odoo
Copy link
Collaborator

I don't think we explicitely bound anything on a "triple click" so I guess it's up to the normalizer in this case. Is the triggered setSelection correct ? Besides, you can upload videos to Odoo's drive if you like.

@dmo-odoo dmo-odoo changed the title triple click merge with the next cellule Triple click merge with the next cellule Mar 17, 2020
@Goaman
Copy link
Contributor Author

Goaman commented Mar 17, 2020

It is indeed the Normalizer and it's doing what the contenteditable is doing but the result seems odd to me. Therefore I think we should think about that case in the future. Thus this issue.

@Zynton Zynton changed the title Triple click merge with the next cellule Triple click merge with the next cell Mar 24, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@Goaman @dmo-odoo