Skip to content

Add height and width options, Cell Comments#42

Open
frenchsomething wants to merge 3 commits into
RamblingCookieMonster:masterfrom
frenchsomething:patch-1
Open

Add height and width options, Cell Comments#42
frenchsomething wants to merge 3 commits into
RamblingCookieMonster:masterfrom
frenchsomething:patch-1

Conversation

@frenchsomething
Copy link
Copy Markdown

@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
Copy Markdown

mi-hol commented Jun 6, 2018

@RamblingCookieMonster are you considering to merge this PR?

@clijsters
Copy link
Copy Markdown
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
Copy Markdown
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