<table><tr><td style="">aheinecke planned changes to this revision.<br />aheinecke 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/D10048" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>Mh for me this did not work when just setting the text on a field. Looking at JSField::fieldSetValue I probably missed the refreshPixmaps call after setting the value.</p>

<p>One difference here is that we change the Field through the Event object and not in pure JavaScript. But yes both ways should use the same mechanism. I look into it some more. Maybe some general "field changed through script" mechanism would be better to support stuff like changing visibility or ReadOnly state of fields. I'm not sure if the "refreshPixmaps" would already make this possible.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R223 Okular</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D10048" rel="noreferrer">https://phabricator.kde.org/D10048</a></div></div><br /><div><strong>To: </strong>aheinecke, Okular<br /><strong>Cc: </strong>aacid, michaelweghorn, ngraham<br /></div>