Skip to content

Added a new function call to the Color class: ToHtml()#40

Merged
mee-ironsoftware merged 1 commit intomasterfrom
IRON-775-iron-pdf-missing-header-and-footer-divider-line-drawing
Feb 21, 2023
Merged

Added a new function call to the Color class: ToHtml()#40
mee-ironsoftware merged 1 commit intomasterfrom
IRON-775-iron-pdf-missing-header-and-footer-divider-line-drawing

Conversation

@mee-ironsoftware
Copy link
Copy Markdown
Member

This translates the specified Color structure to an HTML string color representation.

Copy link
Copy Markdown
Contributor

@mbkrispy mbkrispy left a comment

Choose a reason for hiding this comment

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

LGTM 👍

Comment thread IronSoftware.Drawing/IronSoftware.Drawing.Common/Color.cs Outdated
@mee-ironsoftware mee-ironsoftware force-pushed the IRON-775-iron-pdf-missing-header-and-footer-divider-line-drawing branch from 79770d0 to aca9136 Compare February 21, 2023 04:19
Modify from Jacob feedback
- Change ToHtml to ToHtmlCssColorCode for autocomplete
- Set Binary to null for GC to manage it
@mee-ironsoftware mee-ironsoftware force-pushed the IRON-775-iron-pdf-missing-header-and-footer-divider-line-drawing branch from aca9136 to 73f74fb Compare February 21, 2023 04:20
@mee-ironsoftware mee-ironsoftware merged commit 22a973d into master Feb 21, 2023
@artem-iron artem-iron deleted the IRON-775-iron-pdf-missing-header-and-footer-divider-line-drawing branch March 28, 2023 02:50
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.

3 participants