screencopy: add force 8 bit to fix 10b screensharing (#11623)

ref https://github.com/hyprwm/xdg-desktop-portal-hyprland/issues/270
This commit is contained in:
Vaxry 2025-09-08 11:07:04 +02:00 committed by GitHub
parent bce43f74eb
commit 02bb350bb3
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
6 changed files with 22 additions and 3 deletions

View file

@ -1321,6 +1321,12 @@ inline static const std::vector<SConfigOptionDescription> CONFIG_OPTIONS = {
.type = CONFIG_OPTION_INT,
.data = SConfigOptionDescription::SRangeData{1, 1, 10},
},
SConfigOptionDescription{
.value = "misc:screencopy_force_8b",
.description = "forces 8 bit screencopy (fixes apps that don't understand 10bit)",
.type = CONFIG_OPTION_BOOL,
.data = SConfigOptionDescription::SBoolData{true},
},
/*
* binds:

View file

@ -521,6 +521,7 @@ CConfigManager::CConfigManager() {
registerConfigVar("misc:lockdead_screen_delay", Hyprlang::INT{1000});
registerConfigVar("misc:enable_anr_dialog", Hyprlang::INT{1});
registerConfigVar("misc:anr_missed_pings", Hyprlang::INT{1});
registerConfigVar("misc:screencopy_force_8b", Hyprlang::INT{1});
registerConfigVar("group:insert_after_current", Hyprlang::INT{1});
registerConfigVar("group:focus_removed_window", Hyprlang::INT{1});

View file

@ -60,7 +60,7 @@ CScreencopyFrame::CScreencopyFrame(SP<CZwlrScreencopyFrameV1> resource_, int32_t
return;
}
m_dmabufFormat = m_monitor->m_output->state->state().drmFormat;
m_dmabufFormat = g_pHyprOpenGL->getPreferredReadFormat(m_monitor.lock());
if (box_.width == 0 && box_.height == 0)
m_box = {0, 0, sc<int>(m_monitor->m_size.x), sc<int>(m_monitor->m_size.y)};

View file

@ -108,6 +108,8 @@ class CScreencopyProtocol : public IWaylandProtocol {
bool copyFrameDmabuf(CScreencopyFrame* frame);
bool copyFrameShm(CScreencopyFrame* frame, const Time::steady_tp& now);
uint32_t drmFormatForMonitor(PHLMONITOR pMonitor);
friend class CScreencopyFrame;
friend class CScreencopyClient;
};

View file

@ -114,7 +114,7 @@ CToplevelExportFrame::CToplevelExportFrame(SP<CHyprlandToplevelExportFrameV1> re
return;
}
m_dmabufFormat = PMONITOR->m_output->state->state().drmFormat;
m_dmabufFormat = g_pHyprOpenGL->getPreferredReadFormat(PMONITOR);
m_box = {0, 0, sc<int>(m_window->m_realSize->value().x * PMONITOR->m_scale), sc<int>(m_window->m_realSize->value().y * PMONITOR->m_scale)};

View file

@ -3113,7 +3113,17 @@ void CHyprOpenGLImpl::setCapStatus(int cap, bool status) {
}
uint32_t CHyprOpenGLImpl::getPreferredReadFormat(PHLMONITOR pMonitor) {
return pMonitor->m_output->state->state().drmFormat;
static const auto PFORCE8BIT = CConfigValue<Hyprlang::INT>("misc:screencopy_force_8b");
if (!*PFORCE8BIT)
return pMonitor->m_output->state->state().drmFormat;
auto fmt = pMonitor->m_output->state->state().drmFormat;
if (fmt == DRM_FORMAT_BGRA1010102 || fmt == DRM_FORMAT_ARGB2101010 || fmt == DRM_FORMAT_XRGB2101010 || fmt == DRM_FORMAT_BGRX1010102)
return DRM_FORMAT_XRGB8888;
return fmt;
}
bool CHyprOpenGLImpl::explicitSyncSupported() {