Skip to content

[FIX] autofill: smaller revisions #6056

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Closed
wants to merge 1 commit into from

Conversation

LucasLefevre
Copy link
Collaborator

When auto-filling cell, the UPDATE_CELL commands always contains all 3 properties: content, style and format.

They could be useless though. Typically, when you autofill on blank cells, it useless to have style: null and format: "" in the command.

Description:

description of this task, what is implemented and why it is implemented that way.

Task: TASK_ID

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
Copy link
Collaborator

robodoo commented Apr 2, 2025

Pull request status dashboard

When auto-filling cell, the UPDATE_CELL commands always contains all
3 properties: content, style and format.

They could be useless though. Typically, when you autofill on blank
cells, it useless to have `style: null` and `format: ""` in the command.

Auto-fill A1 to Z1, then the row A1:Z1 to A100:Z100

before: 275kB
after:  210kB (~25% smaller)
@LucasLefevre LucasLefevre force-pushed the 18.0-smaller-autofill-revisions-lul branch from f7f3617 to adaf899 Compare April 17, 2025 12:33
@LucasLefevre
Copy link
Collaborator Author

nope: #6128

@LucasLefevre LucasLefevre deleted the 18.0-smaller-autofill-revisions-lul branch April 17, 2025 12:58
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants