On Fri, Jan 18, 2019 at 04:42:29PM -0500, Richard Kimberly Heck wrote: > > The previous code > > forced you to do *something* about the situation when it's most due -- > > until you deal with the emergency file properly, you will be asked. > Not really. You can perfectly well leave the file sitting there. Then, > as you say, you'll be asked about it again the next time, but only if > you do not save the original file (so that it's modtime is later). But
I did not know about this modtime check (I guess my workflow is always to solve the issue first). > if you have another crash, then the emergency file will be over-written. > We could try to rename it or something first, but presumably we need to > do as little as possible when writing the emergency file. > > Now you you just hit Keep and postpone/forget about that, which is way > > more prone to dataloss than previous situation IMHO. > I'm not sure I understand that worry. The file's there, right? The worry was that you forget about the mismatch between file and its emergency, thus losing the changes in the emergency; anyway the modtime feature made my argument wrong. > > It's also change of the behaviour for the experienced users so if we > > keep this commit we should at least notify about the changed workflow. > > Perhaps "Keep and Rename" as the button. > Let's resolve this first, and then I will change the button or message > or something. I guess my point now is that experienced user is not expecting emergency file to be renamed and stored, so some hint should be provided. > > What I would find as a real improvement in the emergency case workflow > > is to offer comparison of the emergency and stored file. > > I found myself doing this manually bunch of times after emergency file > > was created, so new button for that would be handy and not so much work > > I believe. > > Sure, we could certainly do that. > > Personally, I have not found the comparison stuff very useful. It seems > to proceed character by character, which can be lead to bizarre diffs. > I've occasionally thought about converting it (or adding an option) so > that it went word by word. That would be more informative, I think. Agreed, I use comparison just for getting indication which part of document changed, the character based diff makes the comparison feature somewhat useless due to the bizarre diffs you mentioned. We even have bug discussing this issue, unf Vincent never got to fix that and it might not be trivial. OTOH to have sense whether something changed in the emergency file this feauture is enough. Pavel
