Bug 295295

Summary:	Save Link… context menu saves the target without asking where first (regression)
Product:	[Applications] rekonq	Reporter:	Adrián Chaves (Gallaecio) <adrian>
Component:	general	Assignee:	Andrea Diamantini <adjam7>
Status:	RESOLVED NOT A BUG
Severity:	normal	CC:	johannesobermayr
Priority:	NOR
Version:	0.9.0
Target Milestone:	---
Platform:	Chakra
OS:	Linux
Latest Commit:		Version Fixed In:

Description Adrián Chaves (Gallaecio) 2012-03-04 09:42:41 UTC

User-Agent:       Mozilla/5.0 (X11; Linux i686) AppleWebKit/534.34 (KHTML, like Gecko) rekonq Safari/534.34
Build Identifier: 0.9.0

When I right-click a link and then click on Save Link…, it saves the target diectly to my home folder instead of asking me where to save it.

Reproducible: Always

Steps to Reproduce:
1. Open http://chakra.sourceforge.net/ccr/packages.php?O=0&K=lives&do_Search=Go in Rekonq.
2. Right click “Tarball”, and select “Save Link…”.
Actual Results:  
The linked tarball is saved to the home folder.

Expected Results:  
Rekonq should (and used to) open a dialog for you to choose the destination path.

If I directly click on the link, it works as expected.

Comment 1 Johannes Obermayr 2012-03-04 10:29:29 UTC

Configure rekonq... -> General -> Always ask me where to save files

Comment 2 Adrián Chaves (Gallaecio) 2012-03-05 06:23:58 UTC

Ups, that setting must have been changed during the update from 0.8.

Anyway, I will take the opportunity to strongly suggest that “Save Link…” is changed to “Save Link”, since the ellipsis are meant to lead to another dialog, while it is possible that nothing else happens (visually on Rekonq) after you click that command.

Comment 3 Andrea Diamantini 2012-03-05 21:54:20 UTC

I agree we should anyway better notify started download. You can just see a note with downloaded file name of the bottom left.

The problem is I'd like to not create a "duplicate" of plasma notification system ;)
Different solutions will be investigated during next development cycles. I'll fix the "Save Link..." text as suggested.
Thanks.

Comment 4 Andrea Diamantini 2012-03-05 22:04:57 UTC

Git commit e30f9f58d5c2df9d02ae83f8ae2f5faee6f04814 by Andrea Diamantini.
Committed on 05/03/2012 at 23:09.
Pushed by adjam into branch 'master'.

Don't show dots after "Save Link" text if we are not going to ask for
download path

M  +9    -1    src/webview.cpp

http://commits.kde.org/rekonq/e30f9f58d5c2df9d02ae83f8ae2f5faee6f04814