Save buttons not working

Bug #83733 reported by Dan Munckton
10
This bug affects 1 person
Affects Status Importance Assigned to Milestone
meld
Fix Released
Low
meld (Ubuntu)
Fix Released
Low
Unassigned

Bug Description

Binary package hint: meld

Cannot save changes until closing compared files tab. It appears the Save buttons aren't working.

Steps to reproduce:

1. Create a file, make a copy, modify the copy slightly
2. Open meld to compare the two files
3. Merge the change from the second file to the first
4. Note the '*' in the title of the modified file.
5. Click the save toolbar button - note changes not saved
6. Click File > Save from the meny - changes not saved
7. Type Ctrl+s - changes not saved.
8. Click the X on the comparison tab, you'll get a prompt about unsaved changes and you can now save the modifications

Versions:
 - Noticed in 1.1.2-2 (Dapper)
 - Not checked in Edgy
 - May be able to check for this in Feisty Herd 3 if I get time today.

Revision history for this message
Dan Munckton (munckfish) wrote :

Ok I've just noticed that the reason the buttons are working is because the mouse/cursor focus isn't on the file which has changed. It's probably on neither, so when the save button is clicked it doesn't know which file should be changed and so does nothing.

By clicking the in the file you want to save it gains focus and the save buttons work.

So the bug is probably more about UI design and UI feedback. Before I moved to Linux I used to use WinMerge on Windows and I don't remember a problem, I can take a look and see how it behaves for this scenario.

Revision history for this message
Timmie (timmie) wrote :

+1

please fix this: users must be able to click on the save button to save the changes!

Dan Munckton (munckfish)
Changed in meld:
assignee: nobody → munckfish
status: Unconfirmed → Needs Info
Áron Sisak (asisak)
Changed in meld:
importance: Undecided → Low
status: Needs Info → Confirmed
Changed in meld:
status: Unknown → Unconfirmed
Dan Munckton (munckfish)
Changed in meld:
assignee: munckfish → nobody
Revision history for this message
Christoph Langner (chrissss) wrote :

I can confirm the problem for hardy

$ dpkg -l meld
ii meld 1.1.5.1-2ubuntu1 graphical tool to diff and merge files

Changed in meld:
status: New → Confirmed
Revision history for this message
Steven Harms (sharms) wrote :

I can also confirm this on hardy

Revision history for this message
Leslie Viljoen (leslieviljoen) wrote :

This problem is reported upstream here:
http://bugzilla.gnome.org/show_bug.cgi?id=449087

It has not been solved yet, even in the new release that has been packaged for Jaunty.

Revision history for this message
VinceLe (legoll) wrote :

A patch has been attached to the upstream bug report, please test if it deos what you want

Changed in meld:
status: Confirmed → In Progress
Changed in meld:
importance: Unknown → Low
Changed in meld:
status: In Progress → Fix Released
Revision history for this message
Paul White (paulw2u) wrote :

Upstream issue showing fixed in meld 3.11 on 2014-03-01
So fixed in all recent Ubuntu versions

Changed in meld (Ubuntu):
status: Confirmed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.