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

master-stopPropagation-nby #474

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

Goaman
Copy link
Contributor

@Goaman Goaman commented Nov 16, 2020

No description provided.

Those stopPropagation were not usefull and created problems in Odoo.
@Goaman Goaman self-assigned this Nov 16, 2020
@Goaman Goaman requested a review from dmo-odoo November 16, 2020 10:10
@Gorash
Copy link
Contributor

Gorash commented Nov 19, 2020

le mouse down, attention au changement de range pour que ça garde la selection dans l'arch.
(exemple: ouvrir une dialog de notre arch pour faire un choix couleur, le click dans l'input ne doit pas enlever la selection du dom)

Ca doit changer le range du dom réel (pour le focus et tout) mais pas celui de l'arch.

Est ce que c'est bien vérifier ?
(je pense qu'il n'y hélas a pas de test)

@Gorash Gorash added the question Further information is requested label Nov 19, 2020
@Goaman
Copy link
Contributor Author

Goaman commented Nov 19, 2020

J'ai fait un test manuel et l'input "FontSize" de la toolbar ne trigger pas de setSelection.
Meme si l'input générait une sélection je ne pense pas qu'on devrait l'en empêcher avec un stop[Immediate]Propagation.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
awaiting review question Further information is requested
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants