Bug 256760

Summary:	Floppy disk icon closes attachment tab
Product:	[Unmaintained] KMail Mobile	Reporter:	Sabine Faure <sabine>
Component:	general	Assignee:	kdepim bugs <pim-bugs-null>
Status:	CLOSED FIXED
Severity:	normal	CC:	ludwig.reiter
Priority:	NOR
Version First Reported In:	unspecified
Target Milestone:	---
Platform:	Compiled Sources
OS:	Linux
Latest Commit:		Version Fixed/Implemented In:
Sentry Crash Report:
Attachments:	Floppy disk icon in attachment tab

Description Sabine Faure 2010-11-13 01:52:26 UTC

Version:           unspecified (using Devel) 
OS:                Linux

Clicking on the Floppy disk icon from the attachment tab is closing the tab atm which is confusing for the user as normally such icons are used for saving.

Reproducible: Always

Steps to Reproduce:
Prerequisite: Send an email with a small image attached from a desktop computer
to your Imap account on the N900 (ex: Email 2)

- Launch Kmail-mobile
- Click on your Imap inbox
- Click on Email 2
- Open the 'Attachment' tab
- The tab opens with the name of the file over the grey backgroup of the tab
- Click on the file name in the attachment tab
- The file is partly displayed and in transparency in the attachment tab and over the email text
- Click on the Floppy disk icon at the bottom right hand side of the tab (see snapshot enclosed)


Actual Results:  
The attachment tab closes which is confusing as this is not the behavior the user expects.

Expected Results:  
The attachment tab should not close and I guess a browser to select a location to save the attachment should appear.

Note: I could be a good idea to have a closing icon within the Attachment tab though since it is not always easy to close it, especially when the image is not displayed properly (see fourth screenshot in bug #256758 for more details).

N900, 4:4.6~20101112.1195992-1maemo1.1196001

Comment 1 Sabine Faure 2010-11-13 01:54:53 UTC

Created attachment 53379 [details]
Floppy disk icon in attachment tab

Clicking on this Floppy disk icon closes the attachment tab whereas it should not.

N900, 4:4.6~20101112.1195992-1maemo1.1196001

Comment 2 Volker Krause 2010-11-26 14:37:27 UTC

SVN commit 1200997 by vkrause:

Actually attempt to save image attachments.

BUG: 256760
BUG: 257790


 M  +3 -0      AttachmentList.qml  


WebSVN link: http://websvn.kde.org/?view=rev&revision=1200997

Comment 3 Sabine Faure 2010-11-29 01:40:43 UTC

I tried retesting this bug today but could not because of the blocker bug #257812 which prevents me front attaching files to emails.

N900, 4:4.6~20101128.1201814-1maemo1.1200957

Comment 4 Ludwig Reiter 2010-12-02 13:16:01 UTC

version: 20101202-1202744-1maemo1

can save a pb attachment.