[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-bugs-dist
Subject:    [kmail2] [Bug 314221] New: composer attaches directory without confirmation if also a file is select
From:       Daniel Franke <franke.daniel () gmail ! com>
Date:       2013-01-31 23:26:40
Message-ID: bug-314221-17878 () http ! bugs ! kde ! org/
[Download RAW message or body]

https://bugs.kde.org/show_bug.cgi?id=314221

            Bug ID: 314221
           Summary: composer attaches directory without confirmation if
                    also a file is selected
    Classification: Unclassified
           Product: kmail2
           Version: unspecified
          Platform: Ubuntu Packages
                OS: Linux
            Status: UNCONFIRMED
          Severity: normal
          Priority: NOR
         Component: composer
          Assignee: kdepim-bugs@kde.org
          Reporter: franke.daniel@gmail.com

Ubuntu 12.04, Kontact/KMail 4.8.5

In the "Attach File" dialog: I accidentally selected a folder (gigabytes of
data, thousands of files) plus one extra file via [CTRL] and pressed "Attach". 

If only the folder is selected, a confirmation dialog pops up: "do you reallly
want to attach this directory: ..., yes/no". Fair enough.

With an additional file besides the directory, or with two directories, this
dialog does not appear and the directory/directories are attached without
further confirmation. In above case the kmail GUI froze while trying to pack
~5GB into a mail attachment. The only way to get it back was to kill kontact
and start over.

It might be good to (1) always show the confirmation dialog for every
ddirectory that should be attached and (2) have a second confirmation dialog
once a (configurable?) attachment limit is reached, say 10MB by default,
confirm that really more files should be attached. And (3) maybe do the
collecting/packing in a background worker thread instead of a GUI thread?


Reproducible: Always

-- 
You are receiving this mail because:
You are watching all bug changes.
[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic