<table><tr><td style="">davidedmundson added inline comments.
</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/D12235">View Revision</a></tr></table><br /><div><strong>INLINE COMMENTS</strong><div><div style="margin: 6px 0 12px 0;"><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D12235#inline-64380">View Inline</a><span style="color: #4b4d51; font-weight: bold;">romangg</span> wrote in <span style="color: #4b4d51; font-weight: bold;">xdgoutput_interface.cpp:83</span></div>
<div style="margin: 8px 0; padding: 0 12px; color: #74777D;"><p style="padding: 0; margin: 8px;">This variable is only relevant until the compositor has sent for the first time the done event via <tt style="background: #ebebeb; font-size: 13px;">XdgOutputInterface::done()</tt>. Afterwards it is always true. Is it meant this way? If yes, could we use another name making this fact more explicit, like: <tt style="background: #ebebeb; font-size: 13px;">doneOnce</tt>?</p>

<p style="padding: 0; margin: 8px;">On the other side what if a compositor never sends the done event via <tt style="background: #ebebeb; font-size: 13px;">XdgOutputInterface::done()</tt> because the xdg output never changes its logical size or position after the compositor publishes the xdg output to clients and it expects clients to receive the current values on connecting to the resource?</p></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p style="padding: 0; margin: 8px;">If yes, could we use another name making this fact more explicit, like: doneOnce?</p></blockquote>

<p style="padding: 0; margin: 8px;">It is. I can make that change</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 style="padding: 0; margin: 8px;">On the other side what if a compositor never sends the done event via</p></blockquote>

<p style="padding: 0; margin: 8px;">It should be set on creation. If it's not the compositor is doing things wrong.  I can update the docs to make it clear</p></div></div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R127 KWayland</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D12235">https://phabricator.kde.org/D12235</a></div></div><br /><div><strong>To: </strong>davidedmundson, romangg<br /><strong>Cc: </strong>Frameworks, michaelh, bruns<br /></div>