Created attachment 124380 [details] Annotated screenshot The Document Recovery dialog used to feature a bitmap banner (remember the Windows 95 days? Remember the “Wizard97” design?) with the text “%PRODUCTNAME Document Recovery” on it, in boldface. The bitmap was eventually removed, and for a long time it remained as an empty white strip and the title. Nowadays, not even the white background is rendered — just the text appears. Since this is a redundant widget [1], let’s just get rid of this old widget. Code pointers: https://cgit.freedesktop.org/libreoffice/core/commit/?id=1413d6bc9902a536c0c67f42d2cb16ea45174fe4 http://opengrok.libreoffice.org/xref/core/svx/uiconfig/ui/docrecoveryrecoverdialog.ui#244 https://cgit.freedesktop.org/libreoffice/core/tree/svx/source/dialog/docrecovery.cxx#n634 [1]: The title bar of the dialog is where this text should be; that’s the point of having a title bar in windows!
Oh, http://opengrok.libreoffice.org/xref/core/svx/uiconfig/ui/docrecoverysavedialog.ui is also relevant.
Can we make it a friendly dialog similar to what I suggested in tdf#99341? I mean a big icon/image left (e.g. smiling bug), informative static text at top-left with the scroll area of recoverable documents below. https://bug-attachments.documentfoundation.org/attachment.cgi?id=122077
Heiko, thanks for pointing that out! I think that is something best handled in a separate bug report, though (so that we can iron out the details there) =)
Created attachment 124400 [details] default recovery dialog Totally agree that it should be removed. Also think we should expand this bug report to remove some additional unnecessary features found in the default recovery dialog that isnt present in the crash recover dialog, like the "The 'Status' column shows whether the document can be recoverd", and the large area taken up by the "Recovery document:" label. Below are screenshots of the various stages of the non-crash recovery dialog. http://www.picpaste.com/Recover_01-309DbSEA.png (same as attached file) http://www.picpaste.com/Recover_02-LSG5TwNo.png http://www.picpaste.com/Recover_03-JwRZ1yRc.png
(In reply to Yousuf (Jay) Philips from comment #4) > Created attachment 124400 [details] > default recovery dialog > > Totally agree that it should be removed. Also think we should expand this > bug report to remove some additional unnecessary features found in the > default recovery dialog that isnt present in the crash recover dialog, like > the "The 'Status' column shows whether the document can be recoverd", and > the large area taken up by the "Recovery document:" label. > > Below are screenshots of the various stages of the non-crash recovery dialog. > > http://www.picpaste.com/Recover_01-309DbSEA.png (same as attached file) > > http://www.picpaste.com/Recover_02-LSG5TwNo.png > > http://www.picpaste.com/Recover_03-JwRZ1yRc.png Image links are not working but I have made some changes based on the description. Still I have no idea what the area taken up by "Recovery document:" label is for. It was always empty during recovery. Or does it seem so because the document was recovered quickly? Please see https://gerrit.libreoffice.org/26607
(In reply to Muhammet Kara from comment #5) > Still I have no idea what the area taken up by "Recovery document:" label is > for. It was always empty during recovery. Or does it seem so because the > document was recovered quickly? > > Please see https://gerrit.libreoffice.org/26607 Much better, though I have no idea how Delete/Delete All works (maybe that's an outcome of missing object type). "Start Recovery >" indicates a wizard with the greater than char, and "Discard Recovery Data" sounds complicated. The need for explanation above is a sign of bad usability. How about just "Start" and "Discard"? Recovering Document is just the label for the progress bar below, or do you mean something else? So how about having a list of recoverable documents with checkboxes to selectively _start_ recovering. And the progress bar goes below that list. I wonder if there is need for the progress. Does it really take more than 5 seconds, which is a threshold where feedback becomes necessary. If not the success could be shown by green checkmark icons instead of the checkboxes in the list.
Created attachment 125877 [details] Document recovery in progress (In reply to Heiko Tietze from comment #6) > (In reply to Muhammet Kara from comment #5) > > ... > > Much better, though I have no idea how Delete/Delete All works (maybe that's > an outcome of missing object type). "Start Recovery >" indicates a wizard > with the greater than char, and "Discard Recovery Data" sounds complicated. > The need for explanation above is a sign of bad usability. How about just > "Start" and "Discard"? As the purpose of the buttons are already explained above, "Start" and "Discard" looks fine to me. > Recovering Document is just the label for the > progress bar below, or do you mean something else? That's exactly what I mean, but I asked the question because I didn't see any progress bar near that label. Just a large empty space. I have tried with several documents, taking maybe about 5 seconds. But yet no progress bar. Maybe we should remove that label completely since the progress of each document is already shown on the list box below. (Please see the attached screenshot.) > So how about having a list of recoverable documents with checkboxes to > selectively _start_ recovering. And the progress bar goes below that list. I > wonder if there is need for the progress. Does it really take more than 5 > seconds, which is a threshold where feedback becomes necessary. If not the > success could be shown by green checkmark icons instead of the checkboxes in > the list. I don't know if the selective recovery is still an easyhack, but it may at least belong to a separate bug report. Please see the attached screenshot, and tell me what you think.
Looks good, definitely an improvement. When removing the label you should also kill the progress bar. Just to avoid that it pos up somehow. But first ask one a dev (irc: #libreoffice-dev) what's going on with this object.
Muhammet Kara committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=232dcd258dc03c318e197fe4bfb0080353423760 Simplify Document Recovery dialog tdf#99341 It will be available in 5.3.0. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Seems solved
Not completely solved as the large area taken up by the "Recovery document:" label is still present. http://picpaste.com/document_recovery_01-rpgf4jVA.png Might be also useful to remove the instructions labels explaining what to do when you press the 'Start', 'Discard', and 'Finish' buttons, as they are quite self explanatory. @Adolfo, @Heiko: What you guys think?
(In reply to Yousuf (Jay) Philips from comment #11) >@Adolfo, @Heiko: What you guys think? The label looks like the caption from the progress bar. About the static text: Yes, but a short explanation would be nice. Something like 'We are deeply sorry but last time LibO crashed. We will try to restore the files. Please start this procedure per click on recover or cancel per discard'. Without any text the user is being confronted with a dialog out of the blue.
Assign
Created attachment 126396 [details] Document Recovery Dialog [New]
Could you please see the attached screenshot (Document Recovery Dialog [New]), and the patch on gerrit,[0] and tell me what you think? [0] https://gerrit.libreoffice.org/27511
"Please start by clicking on recover..." but the button is labeled Start. And I would expect the default at the right position (while this depends on the OS/DE). 'Not yet recovered' or in whatever state the item is, has an indentation (guess there was an icon). But there is no hierarchy so it should rather be left-aligned. The dialog itself is quite large for two simple columns, better make it a little bit smaller.
(In reply to Muhammet Kara from comment #14) > Created attachment 126396 [details] > Document Recovery Dialog [New] Dialog looks smaller which is great, but the wording needs improvement. So the dialog says: We are deeply sorry but last time LibreOffice crashed. We will try to restore the files. Please start this procedure per click on 'recover', or cancel per 'discard'. And likely it should be changed to We are deeply sorry but LibreOffice crashed the last time it ran. We will attempt to recover the files that were being modified before the crash. To begin the recovery process, press the 'Start' button or press the 'Discard' button to cancel the recovery. @Adolfo: your English magic is appreciated. :D
Could you please check the new screenshots[0][1][2] and the updated patch?[3] - Removed the large width-request code causing the dialog to be too wide. - Updated the explanation text. - Didn't touch the space (or 'indentation') to the left of the status strings, because it is placeholder for the success/fail images (see the screenshots). [0] http://picpaste.com/beforeRecovery.png [1] http://picpaste.com/duringRecovery.png [2] http://picpaste.com/afterRecovery.png [3] https://gerrit.libreoffice.org/#/c/27511/
Reviewed the Gerrit patch (great work!), I’d make the message even more concise given that the user of a Breakpad-enabled build will see a dialog prior to this one, so in many cases this one won’t be there out of the blue; also, I don’t like too much apologizing in UI texts. With that I consider this bug fixed 👌
(In reply to Muhammet Kara from comment #18) > [0] http://picpaste.com/beforeRecovery.png Slightly modified version of what Adolfo mentioned in gerrit: %PRODUCTNAME will attempt to recover the state of the documents you were working on before it crashed. Click “Start” to begin the process. > [1] http://picpaste.com/duringRecovery.png I'd suggest changing this to: %PRODUCTNAME has begun recovering your documents. Depending on the size of the documents, this process can take some time. > [2] http://picpaste.com/afterRecovery.png Text is fine, but can we remove the line break between these two short sentences.
Muhammet Kara committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=3a7aa4cb86a62249ff2193238b94cfa3a60dffff Remove unnecessary label tdf#99341 It will be available in 5.3.0. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Closing issue