[Kde-pim] Review Request: Add config file only option to Akregator to enable/disable of showing flash content in the article viewer

[Michael Seiwert]

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

(Updated 2010-01-04 07:16:48.647848)


Review request for KDE PIM.


Summary
-------

This patch adds a config file only option to akregator to configure enabling/disabling showing flash content in the article viewer. As you have to manually set the config entry "Plugins Enabled" to "true" the default behavior stays unchanged (flash plugins disabled). I'd like to add a visual config option to the browser section in akgregator configuration dialog later as well.      


Diffs
-----

  /trunk/KDE/kdepim/akregator/src/articleviewer.h 1068826 
  /trunk/KDE/kdepim/akregator/src/articleviewer.cpp 1068826 

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


Testing (updated)
-------

Enabling and disabling seems to work if manually configured via "Plugins Enabled" 


Thanks,

Michael

_______________________________________________
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/