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

List:       kde-pim
Subject:    Re: [Kde-pim] Review Request: KMail: Polishing - icons for context
From:       klebezettel () gmx ! net
Date:       2009-03-08 2:57:42
Message-ID: 20090308025742.23792.26101 () localhost
[Download RAW message or body]


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/253/
-----------------------------------------------------------

(Updated 2009-03-07 18:57:41.908169)


Review request for KDE PIM.


Changes
-------

mention KMail in summary


Summary (updated)
-------

Hiya!

Some simple polishing:

------------------------------
Context-Menu of a message:
    "Copy To", "Move To"
        - these sub menus use the icons of the folderview

Context-Menu of a folder:
    "Copy Folder To"
        - these sub menus use the icons of the folderview
        - above entry "Copy to This Folder"
          (e.g. in "Copy Folder To" -> "Local Folders")
          is now a separator

Context-Menu of a URL in the messsage viewer:
    - Entry "Open URL" now uses the "document-open"
      icon (like klinkstatus)

      Note: "Save Links As..." is left without an icon.
            khtml does not use any icon for this kind of
            entry, too.

Enable sorting of identity list in the preferences
    - Incoming and outgoings accounts are sortable too,
      why not this one? (I'm always confused because
      my identities are not sorted)
------------------------------


Diffs
-----

  trunk/KDE/kdepim/kmail/identitylistview.cpp 936593 
  trunk/KDE/kdepim/kmail/kmreaderwin.cpp 936593 
  trunk/KDE/kdepim/kmail/mainfolderview.cpp 936593 

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


Testing
-------


Thanks,

nik

_______________________________________________
KDE PIM mailing list kde-pim@kde.org
https://mail.kde.org/mailman/listinfo/kde-pim
KDE PIM home page at http://pim.kde.org/
[prev in list] [next in list] [prev in thread] [next in thread] 

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