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

List:       kmail-devel
Subject:    [Bug 52813] Wish for prompt for save location of edited attachment
From:       davidblunkett <dav1dblunk3tt () hotmail ! com>
Date:       2006-09-29 8:30:43
Message-ID: 20060929083043.9920.qmail () ktown ! kde ! org
[Download RAW message or body]

------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
         
http://bugs.kde.org/show_bug.cgi?id=52813         




------- Additional Comments From dav1dblunk3tt hotmail com  2006-09-29 \
10:30 ------- This is NOT a bug - the resultant "bug fix" is hugely \
annoying and I patch the "fix" out everytime I get a new kmail.

Marking files RO is hugely irritating - I suffer from a huge volume of word \
docs which I read in ooffice or wahtever.  They are invetitably formated \
incorrectly and I want to change the font size etc but cannot due to the RO \
flag.

The solution is to leave this on by default for the small percentage of \
novice users but include a button to stop it marking things RO for anyone \
with more than 1 weeks computer experience.

The real problem is kmail not checking to see if a temp file has been \
modified since it saved it - if it has been modified then kmail should not \
delete it as kmail NO LONGER OWNS IT!


--- kmreaderwin.cpp.wrong     2006-01-19 16:55:47.000000000 +0000
+++ kmreaderwin.cpp.right        2006-09-29 09:24:24.000000000 +0100
 @ -1692,7 +1692,8  @
   mTempFiles.append( fname );
   // make file read-only so that nobody gets the impression
that he might
   // edit attached files (cf. bug #52813)
-  ::chmod( QFile::encodeName( fname ), S_IRUSR );
+//disable this bloody feature as it is multoannoying
+  //::chmod( QFile::encodeName( fname ), S_IRUSR );

   return fname;
 }
_______________________________________________
KMail developers mailing list
KMail-devel@kde.org
https://mail.kde.org/mailman/listinfo/kmail-devel


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

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