[Kde-pim] [Differential] [Changed Subscribers] D1991: Show a visual component to switch between text/html view for alternative mails

[dvratil (Daniel Vrátil)]

dvratil added inline comments.

INLINE COMMENTS

> knauss wrote in alternativepart.html:33
> @mlaurent: is there a better solution, how i can wait for jquery to be loaded before triggering this function?

IIRC you should wrap your jQuery into anonymous function like this:

  $(function()) {
   // your code comes here
  });

or

  $(document).ready(function() {
    // your code comes here
  });

to ensure it's invoked after jquery is initialized.

> knauss wrote in csshelperbase.cpp:624
> @colomar: I use here hard color defintions, because i have no glue, what palett color is best here.

Why not use QPalette::Button for background and QPalette::ButtonText for test?

REPOSITORY
  rMESSAGELIB PIM: Message Library

REVISION DETAIL
  https://phabricator.kde.org/D1991

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: knauss, mlaurent, colomar
Cc: dvratil, kde-pim, dvasin, winterz, smartins, vkrause, mlaurent, knauss
_______________________________________________
KDE PIM mailing list kde-pim at kde.org
https://mail.kde.org/mailman/listinfo/kde-pim
KDE PIM home page at http://pim.kde.org/