[Geany-devel] [PATCH 05/19] Don't be annoying when not necessary

[Colomban Wendling]

Hi,

Le 16/08/2010 02:51, Erik de Castro Lopo a écrit :
> Jiří Techet wrote:
> 
>>
>> When reloading a file with ctrl+R don't display the warning dialog
>> that the unsaved changes might be lost when the file has not been
>> modified.
> 
> I've been running with this patch for about a week. Looks good to me.
This one seems to have been applied as r5190, and I have a little
complain about it, so perhaps it would need a little more thought.

The problem is that reloading a document not only looses unsaved
changes, but also undo/redo stack. This is particularly annoying if you
hit the shortcut by accident (which I unfortunately did :( ).
So I propose to two possible solutions:
1) simply check whether there's possible undo/redo, and if yes, ask
before reloading (little patch joined);
2) the solution 1 fixes my problem, but probably not (?) for somebody
that was annoyed by this dialog like Jiří was. So, perhaps a... setting
(!) would be useful?

What do you think?

Regards,
Colomban
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-ask-before-reload-if-there-is-undo-redo-stack.patch
Type: text/x-patch
Size: 540 bytes
Desc: not available
URL: <http://lists.geany.org/pipermail/devel/attachments/20100915/fb47a3b9/attachment.bin>