<table><tr><td style="">alexeymin 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/D1231" rel="noreferrer">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/D1231#39191" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;" rel="noreferrer">D1231#39191</a>, <a href="https://phabricator.kde.org/p/Kanedias/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;" rel="noreferrer">@Kanedias</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>...<br />
 Besides, I didn't find any mentions of multi-screen capabilities in Krfb at all. It currently works like this:</p>

<div class="remarkup-code-block" style="margin: 12px 0;" data-code-lang="text" data-sigil="remarkup-code-block"><pre class="remarkup-code" style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; padding: 12px; margin: 0; background: rgba(71, 87, 120, 0.08);">d->framebufferImage = XGetImage(QX11Info::display(),
                                id,
                                0,
                                0,
                                QApplication::desktop()->width(),
                                QApplication::desktop()->height(),
                                AllPlanes,
                                ZPixmap);</pre></div>

<p>If that's the requirement, there will be huge amount of work to implement it from ground up.<br />
 Patchset for KRfb is already enormous and rewrites half of the input system into plugins instead of built-in libraries (to integrate it with fake-input). I doubt it will endure another set of additions, the review will take forever.<br />
 I think we should implement screen indexing in protocol but start with passing screen №1 only for now.</p></div>
</blockquote>

<p>Speaking about krfb, after <a href="https://phabricator.kde.org/D5211" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: line-through;" rel="noreferrer">D5211</a> X11 plugin (and <tt style="background: #ebebeb; font-size: 13px;">XGetImage</tt> code) does not exist anymore. And krfb is aware of having multiple screens, but it shares only primary screen area:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">xcb plugin: <a href="https://cgit.kde.org/krfb.git/tree/framebuffers/xcb/xcb_framebuffer.cpp#n171" class="remarkup-link" target="_blank" rel="noreferrer">https://cgit.kde.org/krfb.git/tree/framebuffers/xcb/xcb_framebuffer.cpp#n171</a></li>
<li class="remarkup-list-item">qt plugin: <a href="https://cgit.kde.org/krfb.git/tree/framebuffers/qt/qtframebuffer.cpp#n81" class="remarkup-link" target="_blank" rel="noreferrer">https://cgit.kde.org/krfb.git/tree/framebuffers/qt/qtframebuffer.cpp#n81</a></li>
</ul>

<p>I cannot imagine how VNC server application can properly serve multiple monitors at once, especially if they have different resolutions (merge them into one big image covering all monitors at once, with black border around the smaller one?). So I think if krfb will ever support multiple monitors explicitly, there will be a combo box to select which screen to share.</p>

<p>Do you think other screen recording applications will need to capture several monitors at once?</p></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/D1231" rel="noreferrer">https://phabricator.kde.org/D1231</a></div></div><br /><div><strong>To: </strong>Kanedias, graesslin, davidedmundson<br /><strong>Cc: </strong>alexeymin, Frameworks, davidedmundson, plasma-devel, leezu, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, eliasp, sebas, apol, mart, hein, lukas<br /></div>