Skip to content

fix(reader): skip docx annotation auto-save so close prompts save-as … #378

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.

Sign up for GitHub

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

Merged
deepin-bot merged 1 commit into from
Sep 10, 2026
Merged

fix(reader): skip docx annotation auto-save so close prompts save-as … #378

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
9 changes: 7 additions & 2 deletions reader/uiframe/DocSheet.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2166,9 +2166,14 @@ void DocSheet::onAutoSave()
qCDebug(appLog) << "Auto-save triggered for:" << m_filePath;

// 先保存文档注释（高亮、文本标注等），防止异常退出时注释丢失；
// 保存会改变文件内容，之后的指纹/哈希计算必须基于最新内容
// 保存会改变文件内容，之后的指纹/哈希计算必须基于最新内容。
// docx 为转换打开格式，注释只能写入临时目录中的转换产物 temp.pdf，
// 自动保存无法真正持久化；若此处照常保存并清掉脏标志，关闭时会因
// fileChanged() 为假而跳过保存确认框，注释随临时目录删除而静默丢失
// （BUG-376501）。故 docx 保持未保存状态，由关闭时的确认框引导
// 用户另存为 PDF 完成持久化。
bool contentSaved = false;
if (m_documentChanged) {
if (m_documentChanged && Dr::DOCX != fileType()) {
if (m_renderer && m_renderer->save()) {
m_documentChanged = false;
m_sidebar->changeResetModelData();
Expand Down
Loading