Skip to content

Conversation

@IMSupperkaka
Copy link
Contributor

…okStyles

close #xxx

Pull Request Checklist

  • Related tickets or issues have been linked in the PR description (or missing issue).
  • Naming convention is followed (do please check it especially when you created new plugins, commands and resources).
  • Unit tests have been added for the changes (if applicable).
  • Breaking changes have been documented (or no breaking changes introduced in this PR).

@github-actions
Copy link

github-actions bot commented Dec 1, 2025

View Deployment

📑 Demo (React@19) Demo (React@16) 📚 Storybook
🔗 Preview link 🔗 Preview link 🔗 Preview link

@IMSupperkaka IMSupperkaka changed the title feat: Add mutations and commands for addWorkbookStyles & removeWorkbookStyles feat(sheets): Add mutations and commands for addWorkbookStyles & removeWorkbookStyles Dec 2, 2025
@IMSupperkaka IMSupperkaka changed the title feat(sheets): Add mutations and commands for addWorkbookStyles & removeWorkbookStyles feat(sheets): add mutations and commands for addWorkbookStyles & removeWorkbookStyles Dec 2, 2025
@jikkai
Copy link
Member

jikkai commented Dec 3, 2025

Could you describe the issue your PR is intended to solve?

@IMSupperkaka
Copy link
Contributor Author

IMSupperkaka commented Dec 4, 2025

Could you describe the issue your PR is intended to solve?

Because using fworkbook.addStyles and fworkbook.removeStyles does not work collaboratively, in some scenarios(Custom API) we need to manually set the cell values and styles:

cellData = {
    v: 'sdad',
    s: 'Yauoq-Xa'
}

At this time, you need to execute the Yauoq-Xa style for the workbook styles: addStyles or removeStyles manually operate the Yauoq-Xa style

addStyles(styles: Record<string, IStyleData>): void {
this._workbook.addStyles(styles);
}

removeStyles(styleKeys: string[]): void {
this._workbook.removeStyles(styleKeys);
}

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