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

FIX/Spec change: tab on table #2970

Merged
merged 1 commit into from Mar 12, 2024
Merged

FIX/Spec change: tab on table #2970

merged 1 commit into from Mar 12, 2024

Conversation

cammarosano
Copy link

@robodoo
Copy link

robodoo commented Mar 12, 2024

This PR targets the un-managed branch odoo-dev/odoo:master-phoenix, it needs to be retargeted before it can be merged.

Copy link

@ged-odoo ged-odoo left a comment

Choose a reason for hiding this comment

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

could you add a test to make sure that the toolbar is not open? in toolbar.test.js

const { el } = await setupEditor(contentBefore);
press("Tab");
expect(getContent(el)).toBe(contentAfter);
await animationFrame();
Copy link
Author

Choose a reason for hiding this comment

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

Not the greatest way to test it, as the toolbar could still not be open after one animation frame...
But at least this will not fail undeterministically.
The test below is a better test for toolbar close.

@ged-odoo ged-odoo merged commit 9e0daba into master-phoenix Mar 12, 2024
@ged-odoo ged-odoo deleted the master-phoenix-rcdl branch March 12, 2024 21:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants