Skip to content

[REF] owl3: transform some stores into owl plugins#8894

Open
hokolomopo wants to merge 2 commits into
masterfrom
master-store-to-owl-plugin
Open

[REF] owl3: transform some stores into owl plugins#8894
hokolomopo wants to merge 2 commits into
masterfrom
master-store-to-owl-plugin

Conversation

@hokolomopo

@hokolomopo hokolomopo commented Jun 11, 2026

Copy link
Copy Markdown
Contributor

Description

Transform the DelayedHoveredCellStore, HoveredIconStore and HoveredTableStore into owl plugins.

Task: 6306326

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

@robodoo

robodoo commented Jun 11, 2026

Copy link
Copy Markdown
Collaborator

Pull request status dashboard

Comment thread demo/main.js
Comment on lines +277 to +279
if (message !== "ResizeObserver loop completed with undelivered notifications.") {
this.notifyError();
}

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I don't really get why this is an issue now ...

Transform the `DelayedHoveredCellStore`, `HoveredIconStore` and
`HoveredTableStore` into owl plugins.

Task: 0
@hokolomopo hokolomopo force-pushed the master-store-to-owl-plugin branch from 1026116 to d0c3df4 Compare June 16, 2026 08:41
Convert the cell popover store to owl plugin.

The conversion is not the greatest yet, as we need to access the
plugin in the helpers that only take the env as argument, which
we cannot do ATM.

Task: 6306326
@hokolomopo hokolomopo force-pushed the master-store-to-owl-plugin branch from d0c3df4 to 8cea570 Compare June 16, 2026 08:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants