Review Request: Add a "Save As" option to the SaveBar in GwenView

"Harsh J" <[email protected]> Wed, 28 Apr 2010 18:53:43 -0000
Newsgroups gmane.comp.kde.gwenview
Message-ID <20100428185343.27920.41048@localhost>
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/3828/
-----------------------------------------------------------

Review request for Gwenview.


Summary
-------

Patch aims to add a 'Save As' option to the SaveBar that appears when any kind of change is made to an image.

Simple addition (thanks to existing actionCollection) that activates the "file_save_as" action as it already exists and is visible by the same condition that 'current Url save' action uses.

I find this useful myself particularly -- I never wish my original image to be changed (rotate, crop or etc.), and would like an easier access to the Save As action that I otherwise have to access via the toolbar (if configured) or the File menu. The SaveBar makes it easier to spot, feels at home there since I like using the Toolbar for simple navigation (at which its brilliant!).

Not sure if it actually fixes the associated bug, but seemed relevant (has a fullscreen request but I don't see a SaveBar inside that mode..)

OK to commit?

P.s. I noticed a FIXME regarding use of Q/KAction instead, but I didn't get what to fix -- Should all QToolButtons be changed to use Q/KActions? Point me on the task and I shall gladly try to get it fixed if also required :)


This addresses bug 146095.
    https://bugs.kde.org/show_bug.cgi?id=146095


Diffs
-----

  /trunk/KDE/kdegraphics/gwenview/app/savebar.cpp 1119657 

Diff: http://reviewboard.kde.org/r/3828/diff


Testing
-------

With single and multiple files changed - works fine, only the requested image is saved after supplying a new name/location via the prompt.


Screenshots
-----------

Save As Toolbutton on the SaveBar
  http://reviewboard.kde.org/r/3828/s/377/


Thanks,

Harsh


------------------------------------------------------------------------------