o-spreadsheet icon indicating copy to clipboard operation
o-spreadsheet copied to clipboard

Master composer hover result adrm

Open hokolomopo opened this issue 11 months ago • 3 comments

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

hokolomopo avatar Jan 29 '25 10:01 hokolomopo

Pull request status dashboard

robodoo avatar Jan 29 '25 10:01 robodoo

@robodoo rebase-ff

hokolomopo avatar Jan 29 '25 10:01 hokolomopo

Merge method set to rebase and fast-forward.

robodoo avatar Jan 29 '25 10:01 robodoo