Skip to content

Conversation

fpaupier
Copy link
Contributor

@fpaupier fpaupier commented Mar 30, 2021

Summary

Current doc for cell filling could lead to misunderstanding on how to apply a simple fill color
to a cell.

Related to #1658 (#272)

Test plan

No test required for a readme update.

Current doc for cell filling could lead to misunderstanding on how to apply a simple fill color
to a cell.

Related to exceljs#272
@fpaupier fpaupier mentioned this pull request Mar 30, 2021
Copy link
Member

@Siemienik Siemienik left a comment

Choose a reason for hiding this comment

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

LGTM, Thank you! 📖 👍

@Siemienik Siemienik merged commit ae2d2e5 into exceljs:master Apr 11, 2021
@exceljs exceljs locked as resolved and limited conversation to collaborators Apr 11, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants