Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

对功能的修复完善 #227

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

对功能的修复完善 #227

wants to merge 2 commits into from

Conversation

YuanDad
Copy link

@YuanDad YuanDad commented Aug 25, 2022

尝试修复默认样式的丢失和复制的不完全

新建的Excel文件,每个格子会自动创建一个<alignment horizontal="center" vertical="center"/>(垂直居中),但此时并没有添加applyAlignment属性,这将造成读取时丢失默认的垂直居中样式,并且此问题无法在读取动作后得到有效的解决,所以建议此处不管有没有applyAlignment属性,都应该尝试去读取,而且因为QXmlStreamReader::readNextStartElement函数的特性,此处的尝试读取并不会对原功能造成任何影响
QXlsx增加了对打印页边距的支持,但是却没有在复制sheet的时候将打印页边距属性一并复制,这违背了复制的含义,建议修改
@j2doll
Copy link
Member

j2doll commented Jan 4, 2023

尝试修复默认样式的丢失和复制的不完全

Dear @YuanDad

Can you write the content in English? I don't know Chinese.

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.

None yet

2 participants