Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Add height and width options, Cell Comments #42

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

frenchsomething
Copy link

@frenchsomething frenchsomething commented Nov 15, 2016

Adding parameters for row height and column width
Will iterate through all selected/specified rows and/or columns as applicable, and set height and width as defined

Adding cell comments
The PSExcel comment methods only allow adding a comment to one cell at a time, so I had to iterate over all cells in an arbitrary cell range to avoid errors (and failure to apply comment to more than the first cell) if more than one cell is specified.

Adding parameters for height and width
Will iterate through all selected rows and/or columns as applicable, and set height and width as defined
The PSExcel comment methods only allow adding a comment to one cell at a time, so I had to iterate over all cells in an arbitrary cell range to avoid errors (and failure to apply comment to all cells) if more than one cell is specified.
@frenchsomething frenchsomething changed the title Update Format-Cell.ps1 to add height and width options Add height and width options, Cell Comments Nov 15, 2016
@mi-hol
Copy link

mi-hol commented Jun 6, 2018

@RamblingCookieMonster are you considering to merge this PR?

@Clijsters
Copy link
Contributor

Clijsters commented Jun 12, 2018

If I would be an active contributor I would blame you for your whitespace mess. But I'm not.

@RamblingCookieMonster
Copy link
Owner

Yikes, somehow missed this!

I'd be happy to merge, will see if I have time to s/tab/spaces at some point and merge this in.

Cheers!

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.

4 participants