Shortcut for Save

Bug #667105 reported by Tim White
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
OpenShot Video Editor
Fix Released
High
Jonathan Thomas

Bug Description

Currently Ctrl + S isn't a shortcut to save. (It'll split(razor) a clip as that is what s does).
A previous bug was supposed to fix it https://bugs.launchpad.net/openshot/+bug/510056 but appears to have only worked on the second half. I'm opening a new bug as the other half of that bug is fixed.

Even things like Firefox use Ctrl + S to save. The current behaviour is very confusing for users as they goto save and instead split the current clip they are working on.

Also, we should probably have tooltips/menus show what the shortcuts are. However that will probably be a separate bug unless whom ever takes on this one does it at the same time.

Ubuntu 10.10
Installed Deb from standard Maverick repo
OpenShot 1.2.2

Tags: patch
Revision history for this message
Andy Finch (fincha) wrote :

This used to work in earlier versions, it looks like it got broken in v1.2

Changed in openshot:
status: New → Confirmed
milestone: none → 1.3.0
moimael (moimael)
Changed in openshot:
importance: Undecided → Low
Revision history for this message
moimael (moimael) wrote :

I made a patch who fix that and make the shortcuts visible in the menu ;)

Revision history for this message
Jonathan Thomas (jonoomph) wrote :

I have removed the "s" shortcut from the "Cut all clips" function, and verified that CTRL + S calls the save function. I also found a few issues with the "Cut all clips" function, so I fixed those as well. Hopefully, this will be less confusing now. =)

Changed in openshot:
assignee: nobody → Jonathan Thomas (jonoomph)
importance: Low → Medium
status: Confirmed → Fix Committed
Revision history for this message
moimael (moimael) wrote :

How about the visibility in the menu ? My patch above display the shortcut associated with each menu item, it can be useful when we don't know every shortcuts ;)

Revision history for this message
moimael (moimael) wrote :

Rebased version of the shortcuts patch.

Changed in openshot:
importance: Medium → High
moimael (moimael)
Changed in openshot:
status: Fix Committed → Fix Released
tags: added: patch
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.