Skip to content
New issue

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

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

Already on GitHub? # to your account

Update SheetDataWriter.cs #1072

Merged
merged 1 commit into from
Apr 26, 2023
Merged

Update SheetDataWriter.cs #1072

merged 1 commit into from
Apr 26, 2023

Conversation

SamuelPlentz
Copy link
Contributor

@SamuelPlentz SamuelPlentz commented Apr 26, 2023

LibreOffice 7.3.4.2 has a problem with xml like
<row r="1" customHeight="true" ht="15">, however this
<row r="1" customHeight="true" ht="15"> works well.

So I propose to remove this second space.

LibreOffice 7.3.4.2 has a problem with xml like
```<row r="1" customHeight="true"  ht="15">```, however this
```<row r="1" customHeight="true" ht="15">``` works well.

So I propose to remove this space
@tonyqus
Copy link
Member

tonyqus commented Apr 26, 2023

LGTM

@tonyqus tonyqus added this to the NPOI 2.6.1 milestone Apr 26, 2023
@tonyqus tonyqus merged commit f05ed94 into nissl-lab:master Apr 26, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants