After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 783659 - <Shift+Ctrl+R> (View->Reload) shortcut is missed in modern versions of Meld
<Shift+Ctrl+R> (View->Reload) shortcut is missed in modern versions of Meld
Status: RESOLVED OBSOLETE
Product: meld
Classification: Other
Component: general
unspecified
Other Linux
: Normal normal
: ---
Assigned To: meld-maint
meld-maint
Depends on:
Blocks:
 
 
Reported: 2017-06-11 13:46 UTC by Nrbrtx
Modified: 2017-12-13 19:26 UTC
See Also:
GNOME target: ---
GNOME version: ---



Description Nrbrtx 2017-06-11 13:46:32 UTC
Steps to reproduce:
1. Install Ubuntu 16.04 LTS with modern Meld (3.14.2).
2. Open two files in Meld, change one of them with text editor (kate, pluma, geany, etc.)
3. Try to use <Shift+Ctrl+R> to Refresh left and right objects.

Expected results:
* user is able to use <Shift+Ctrl+R> to refresh left and right objects.

Actual results:
* user is unable to use <Shift+Ctrl+R> to refresh left and right objects. User must to use mouse and click "File->Revert" with it.

Notes:
1. For backward compatibility Meld show have <Shift+Ctrl+R> shortcut. It should Refresh both sides. Current "Revert" label is not correct. 
In text editors revert means "reject changes and reload original version of file from disk".
Previous version (1.5.3 on Ubuntu 12.04 LTS) does this correctly.
2. Sometimes Meld does not detect file changes, so <Shift+Ctrl+R> shortcut will be very useful.
Comment 1 Kai Willadsen 2017-06-12 08:43:03 UTC
I'm sorry, but I feel that there should be very little demand for this action, which is why there's no shortcut for it. The other bug you've filed about not showing the refresh prompt is, as far as I'm concerned, the real problem here.

The term "Revert" was chosen because it's what gedit used for this action (prior to the new-style UI, where there's no label) and because that's what it does. It reverts the state of the files in the editor to the state on disk. It's worth mentioning that you keep saying "Refresh", but that's definitely not what refresh did. In older versions of Meld this was called "Reload".
Comment 2 Kai Willadsen 2017-06-12 08:48:52 UTC
I should mention that it's relatively easy to hack this shortcut in yourself. If you edit the installed `filediff.ui` file, find the `Revert` GtkAction and add a line like:

      <accelerator key="R" modifiers="GDK_SHIFT_MASK | GDK_CONTROL_MASK"/>

then that should work.
Comment 3 Nrbrtx 2017-06-12 09:36:58 UTC
Dear Kai!

I'm sorry I'made a mistake. In previous versions it really was called Reload (with <Shift+Ctrl+R> shortcut). "Refresh" works different.
Comment 4 Nrbrtx 2017-06-12 09:42:09 UTC
About accelerator key I want to say this:
you (as developers) removed shortcut for Reload/Revert option.
It would be great if you bring accelerator key back. I'm user, not developer. 
I expect stable user experience with Meld.
Comment 5 GNOME Infrastructure Team 2017-12-13 19:26:18 UTC
-- GitLab Migration Automatic Message --

This bug has been migrated to GNOME's GitLab instance and has been closed from further activity.

You can subscribe and participate further through the new bug through this link to our GitLab instance: https://gitlab.gnome.org/GNOME/meld/issues/135.