Skip to content

Conversation

cobussmit74
Copy link

Hi Matt

I'm loving DoddleReport! It solves a problem I've been struggling with for ages... Thanks!

I'm hoping that the small change I made will be useful to others.

Cheers
Cobus

Cobus Smit and others added 2 commits February 21, 2017 11:14
…ng,gif) to be displayed in the Title section via ReportTitleStyle class. User supplies details using TitleStyle render hint
@matthidinger
Copy link
Owner

Hi @cobussmit74, many thanks for the contribution! When I first created the project I set a goal to keep the report writers as functionally identical as possible. Are you familiar enough with the other writers to add this title image to them as well by chance?

  1. The HtmlReportWriter and ExcelReportWriter (from DoddleReport project) are nearly identical (ExceptReportWriter inherits from Html and adds some XML tags to the top, so hopefully those will be straight forward.)
  2. OpenXml.ExcelReportWriter I haven't touched in a while, so not sure how difficult that one would be.
  3. As for AbcPdfReportWriter I assume people have started using it less ever since I made the iTextSharp one, so if you aren't familiar with that I think that would be fine and I can try and get to it.

@cobussmit74
Copy link
Author

Hi @matthidinger, that makes sense. I'll see what I can do.

Cobus Smit added 2 commits February 22, 2017 08:25
… to HtmlReportWriter. User can also specify logo image when passing in a ReportTitleStyle object to TitleStyle.
…led this feature for now when using the Excel writer.
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