odoo/o-spreadsheet#5575

Created by BI, Adrien Minne (adrm)

label

odoo:master-composer-hover-result-adrm

head

ae6f1f09eb8b2386f1f582b46dc5b2a969f25d70

merged

9 months ago by BI, Lucas Lefèvre (lul)

	odoo/o-spreadsheet
master	#5575

Master composer hover result adrm

Description:

[IMP] composer: preview formula expressions result

This commit adds a preview of the formula expression result in the
composer when hovering over a formula token. We can hover any part of
the formula (a range, a primitive, a sub-formula, etc.) and see the
result of the expression in a speech bubble. This allows for easy
debugging of complex formulas.

[MOV] test: move some composer tests in correct files

Some tests in the composer_component.test.ts were mounting a
spreadsheet after a ComposerWrapper was already mounted in the
beforeEach.

It caused issue with the following commit that adds a Portal target in
the ComposerWrapper. This led to two Portal targets in the document and
a test failing.

This commit moves these tests to the composer_integration_component.test.ts
file where they belong.

Task: 4471274

review checklist

• [ ] feature is organized in plugin, or UI components
• [ ] support of duplicate sheet (deep copy)
• [ ] in model/core: ranges are Range object, and can be adapted (adaptRanges)
• [ ] in model/UI: ranges are strings (to show the user)
• [ ] undo-able commands (uses this.history.update)
• [ ] multiuser-able commands (has inverse commands and transformations where needed)
• [ ] new/updated/removed commands are documented
• [ ] exportable in excel
• [ ] translations (_t("qmsdf %s", abc))
• [ ] unit tested
• [ ] clean commented code
• [ ] track breaking changes
• [ ] doc is rebuild (npm run doc)
• [ ] status is correct in Odoo