Skip to content

Fix/pdf merged cells ignores hidden - #2450

Merged
swmal merged 62 commits into
develop9from
fix/pdf_MergedCellsIgnoresHidden
Aug 4, 2026
Merged

Fix/pdf merged cells ignores hidden#2450
swmal merged 62 commits into
develop9from
fix/pdf_MergedCellsIgnoresHidden

Conversation

@AdrianEPPlus

@AdrianEPPlus AdrianEPPlus commented Aug 4, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Before submitting this pull request I have...

  • read EPPlus Softwares Contribution Guidlines
  • ensured that the functionality I have added/changed is covered by new unit tests.
  • merged the target branch into the PR branch and resolved any merge conflicts
  • Run all the unit tests and ensured that they all are green (unless the purpose of the PR is to provide us with failing unit tests).

AdrianEPPlus and others added 30 commits May 27, 2026 15:48
@AdrianEPPlus
Added headings back
9a2d676
@AdrianEPPlus
fixed some issues with headings.
843bc28
@swmal
Perf fixes
58d80a9
@AdrianEPPlus
print titles progress
8bd3760
@swmal
Fix PDF rich text rendering; remove source workbook mutation in export
d9d855f
@swmal
Merged develop9 into testshaping-perf branch
631546b
@AdrianEPPlus
print titles progress
7cc305d
@AdrianEPPlus
Merge branch 'textshaping-perf' of https://github.com/epplussoftware/…
245a66c 
…epplus into feature/pdfheadings

# Conflicts:
#	src/EPPlus.Export.Pdf/ExcelPdf.cs
@AdrianEPPlus
fixed errors
ca64ab0
@AdrianEPPlus
fixed wiriting negative pdf object references in font resource on page.
8cb5310
@AdrianEPPlus
Merge branch 'develop9' into feature/pdfheadings
9ab0fd1
@AdrianEPPlus
changed type
45ecf68
@OssianEPPlus
Fixed duplication bug
f7e2250
@OssianEPPlus
Merge branch 'feature/pdfheadings' of https://github.com/EPPlusSoftwa…
9dc0822 
…re/EPPlus into feature/pdfheadings
@OssianEPPlus
Changed test name
21c4b06
@OssianEPPlus
Reverted tint change for pdf
d2c9e5c
@AdrianEPPlus
Fixed font dictionary setting wrong font name.
e762f66
@swmal
Added logging for trouble shooting
0db239e
@AdrianEPPlus
fixed more stuff with font dictionary adding wrong font name
dca477d
@AdrianEPPlus
print titles progress
f478edc
@swmal
Fixed performance issue with heading rendering
58c0e0f
@AdrianEPPlus
print titles progress
7336c3a
@AdrianEPPlus
Merge branch 'feature/pdfheadings' of https://github.com/epplussoftwa…
98d734c 
…re/epplus into feature/pdfheadings

# Conflicts:
#	src/EPPlus.Export.Pdf/PdfCatalog/PdfLayout.cs
@AdrianEPPlus
fixed errors from merge
4031bf5
@swmal
Unittest for pdf added
0660489
@swmal
Added more unittests with improvment areas
9248326
@AdrianEPPlus
print titles fixes progress
6be95c8
@AdrianEPPlus
Merge branch 'feature/pdfheadings' of https://github.com/epplussoftwa…
841d4cd 
…re/epplus into feature/pdfheadings
@swmal
One more commit
3fcca9a
@AdrianEPPlus
Merge branch 'feature/pdfheadings' of https://github.com/epplussoftwa…
e180d80 
…re/epplus into feature/pdfheadings
AdrianEPPlus and others added 26 commits June 16, 2026 16:12
@AdrianEPPlus
Added conditional formatting for text and borders.
b0d37f6
@swmal
Changed unittest
8497e9d
@swmal
Merge branch 'feature/pdfheadings' of https://github.com/EPPlusSoftwa…
2620988 
…re/EPPlus into feature/pdfheadings
@AdrianEPPlus
Made some changes to conditional formatting so formats get applied to…
59a569a 
… empty cells like excel does.
@swmal
Fixed patternfills and DimensionByVisibility
ec6eeb2
@swmal
Merge branch 'feature/pdfheadings' of https://github.com/EPPlusSoftwa…
fae8130 
…re/EPPlus into feature/pdfheadings
@AdrianEPPlus
Moving and restructing pdf code progress.
05fcfb0
@AdrianEPPlus
Moving pdf code progress.
949a2fd
@AdrianEPPlus
Moving files progress
a6b5508
@AdrianEPPlus
Fixed more errors
b4c1779
@AdrianEPPlus
All errors has been slain.
1ad8bcc
@AdrianEPPlus
Removed some unused code.
b9b1f69
@AdrianEPPlus
test
818499f
@AdrianEPPlus
Updated todo.
f4c6bf5
@AdrianEPPlus
Added entry points for pdf export.
bc40b0a
@AdrianEPPlus
Async methods. Some tests and issues fixed regarding orientation.
33b1bf6
@AdrianEPPlus
Added tests and stream versions of save pdf to worksheet and range
709cc6a
@AdrianEPPlus
removed font export debug
f9d9305
@AdrianEPPlus
Merge branch 'develop9' into feature/pdfentry
cb2d4a1 
# Conflicts:
#	src/EPPlus.DrawingRenderer.Tests/Chart/LineChartToSvgTests.cs
#	src/EPPlus.Export.Pdf.Tests/PdfTests.cs
#	src/EPPlus.Export.Pdf/ExcelPdf.cs
#	src/EPPlus.Export.Pdf/Settings/PdfPageSettings.cs
#	src/EPPlus.Export.Pdf/Settings/PdfPageSizes/PdfPageSize.cs
#	src/EPPlus.Fonts.OpenType/OpenTypeFontEngine.cs
#	src/EPPlus/EPPlus.csproj
#	src/EPPlus/ExcelWorkbook.cs
#	src/EPPlus/Export/PdfExport/PdfCatalog.cs
@AdrianEPPlus
Using font enigne in workbook
da47d3f
@AdrianEPPlus
fixed margins using normal if not set.
fdf104f
@AdrianEPPlus
printer settings now has default values for margins and orientation.
ba65086
@AdrianEPPlus
Changed to use DimensionByValue
96f1b42
@AdrianEPPlus
removed usings.
11314db
@AdrianEPPlus
Merge branch 'develop9' into feature/pdfentry
e674e2d 
# Conflicts:
#	src/EPPlus.Export.Pdf.Tests/PdfTests.cs
@AdrianEPPlus
fixed merged cells ignoring hidden rows and columns. Also fixed rende…
c5b6400 
…ring issue caused when hidding rows/columns with gradients.
@AdrianEPPlus
AdrianEPPlus requested a review from swmal August 4, 2026 08:15
@AdrianEPPlus AdrianEPPlus linked an issue Aug 4, 2026 that may be closed by this pull request
Pdf: Merged cells uses full width/height even when rows/columns are hidden. #2399
Open
@swmal
swmal merged commit 160a3bc into develop9 Aug 4, 2026
2 of 3 checks passed
@swmal
swmal deleted the fix/pdf_MergedCellsIgnoresHidden branch August 4, 2026 13:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@swmal swmal Awaiting requested review from swmal

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Pdf: Merged cells uses full width/height even when rows/columns are hidden.

3 participants

@AdrianEPPlus @swmal @OssianEPPlus