<table><tr><td style="">alukichev added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D26753">View Revision</a></tr></table><br /><div><div><blockquote style="border-left: 3px solid #8C98B8;
          color: #6B748C;
          font-style: italic;
          margin: 4px 0 12px 0;
          padding: 8px 12px;
          background-color: #F8F9FC;">
<div style="font-style: normal;
          padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D26753#597754" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D26753#597754</a>, <a href="https://phabricator.kde.org/p/SGOrava/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@SGOrava</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>So I tested it and it works as intended.</p></div>
</blockquote>

<p>Thanks for testing!</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>What I see as a problem is that this patch does not honour the relationship between PDF viewer and Pepper plugins.<br />
 So if you disable pepper plugins PDF viewer would also be gone and users would complain.</p></blockquote>

<p>Indeed. I'll change the patch to account for that.</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>I would add this option under pepper plugin option if possible add some indent and used some signals&slots to make it active only when pepper plugins are enabled and disabled when pepper plugins are disabled.</p></blockquote>

<p>Notwithstanding the dependency noted above, do you think it is a good idea to put that option directly under "Allow Pepper Plugins (Flash plugin)"? The viewer is not a plugin, in the sense that other Falkon plugins are, or even in relation to qtwebengine, as it is its built-in component within QtEngineCore. What do you think?</p>

<p>I've also noticed the viewer's dependency on v8 in the source code, so even though it appears to work when you uncheck "Allow Javascript" checkbox in the preferences, it may not work entirely as expected. There may be other dependencies we don't know about. All that appears to be internal implementation details prone to change along with "chromium" bundle that Qt takes as a 3rd party component. There seems to be no point in tying all those details on Falkon's top level UI...</p></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D26753">https://phabricator.kde.org/D26753</a></div></div><br /><div><strong>To: </strong>alukichev, Falkon<br /><strong>Cc: </strong>SGOrava, falkon, alukichev, grune, akilgus, siraj_qazi, saishm, anmolgautam, iodelay, spoorun, ptabis, navarromorales, clivej, mparillo, ach<br /></div>