Changeset View
Standalone View
abstract_wayland_output.cpp
| Show First 20 Lines • Show All 237 Lines • ▼ Show 20 Line(s) | |||||
| 238 | { | 238 | { | ||
| 239 | m_waylandOutput->setCurrentMode(size, refreshRate); | 239 | m_waylandOutput->setCurrentMode(size, refreshRate); | ||
| 240 | m_xdgOutput->setLogicalSize(pixelSize() / scale()); | 240 | m_xdgOutput->setLogicalSize(pixelSize() / scale()); | ||
| 241 | m_xdgOutput->done(); | 241 | m_xdgOutput->done(); | ||
| 242 | } | 242 | } | ||
| 243 | 243 | | |||
| 244 | void AbstractWaylandOutput::initInterfaces(const QString &model, const QString &manufacturer, | 244 | void AbstractWaylandOutput::initInterfaces(const QString &model, const QString &manufacturer, | ||
| 245 | const QByteArray &uuid, const QSize &physicalSize, | 245 | const QByteArray &uuid, const QSize &physicalSize, | ||
| 246 | const QVector<DeviceInterface::Mode> &modes) | 246 | const QVector<DeviceInterface::Mode> &modes, | ||
| 247 | const QString &identifier) | ||||
| 247 | { | 248 | { | ||
| 248 | m_waylandOutputDevice->setUuid(uuid); | 249 | m_waylandOutputDevice->setUuid(uuid); | ||
| 249 | 250 | | |||
| 250 | if (!manufacturer.isEmpty()) { | 251 | if (!manufacturer.isEmpty()) { | ||
| 251 | m_waylandOutputDevice->setManufacturer(manufacturer); | 252 | m_waylandOutputDevice->setManufacturer(manufacturer); | ||
| 252 | } else { | 253 | } else { | ||
| 253 | m_waylandOutputDevice->setManufacturer(i18n("unknown")); | 254 | m_waylandOutputDevice->setManufacturer(i18n("unknown")); | ||
| 254 | } | 255 | } | ||
| Show All 17 Lines | 265 | for (auto mode : modes) { | |||
| 272 | if (mode.flags & DeviceInterface::ModeFlag::Preferred) { | 273 | if (mode.flags & DeviceInterface::ModeFlag::Preferred) { | ||
| 273 | flags |= KWayland::Server::OutputInterface::ModeFlag::Preferred; | 274 | flags |= KWayland::Server::OutputInterface::ModeFlag::Preferred; | ||
| 274 | } | 275 | } | ||
| 275 | m_waylandOutput->addMode(mode.size, flags, mode.refreshRate); | 276 | m_waylandOutput->addMode(mode.size, flags, mode.refreshRate); | ||
| 276 | } | 277 | } | ||
| 277 | 278 | | |||
| 278 | // start off enabled | 279 | // start off enabled | ||
| 279 | m_waylandOutput->create(); | 280 | m_waylandOutput->create(); | ||
| 281 | if (!identifier.isEmpty()) { | ||||
| 282 | m_xdgOutput->setName(identifier); | ||||
| 283 | } | ||||
| 284 | if (!name().isEmpty()) { | ||||
zzag: Please rename `AbstractOutput::name()` to `AbstractOutput::description()` to keep… | |||||
I had looked. Without bringing in further behavioural changes it makes things worse. You then need to change screens to be in sync, and that one gets used a lot. Then instead of having one piece of code refer to name when they mean description, we have dozens refer to description when they just want a simple readable name. davidedmundson: I had looked. Without bringing in further behavioural changes it makes things worse.
You then… | |||||
So, I assume you're talking about output mapping for touch devices. If that's the case, we could set the udev 'WL_OUTPUT` property on the input device, but nobody does it. Alternatively, we could follow libinput's documentation [1] and use something else. My main concern about this code is that we start mixing identifiers, names, and descriptions, which makes code more difficult to work with and reason about. zzag: So, I assume you're talking about output mapping for touch devices. If that's the case, we… | |||||
No I was referring to OutputScreens::name(int screen) Following that, it gets used in the UI in context menus. I initially was trying to avoid a behavioural change on that, but it turns out we probably want that. davidedmundson: No I was referring to OutputScreens::name(int screen)
Following that, it gets used in the UI… | |||||
We probably don't even need to bother with libinput_device_get_output_name() as long as our output names match weston's output names. I also believe that libinput_device_get_output_name() returns NULL in many cases since I don't see who sets WL_OUTPUT. Either way, worth checking... zzag: We probably don't even need to bother with libinput_device_get_output_name() as long as our… | |||||
Maybe it's not so bad. It seems on X11 the name is in the form HDMI-0 which is what we call "name" here. So this actually is an opportunity to make things more consistent. davidedmundson: Maybe it's not so bad.
It seems on X11 the name is in the form HDMI-0 which is what we call… | |||||
| 285 | m_xdgOutput->setDescription(name()); | ||||
| 286 | } | ||||
| 280 | m_xdgOutput->setLogicalSize(pixelSize() / scale()); | 287 | m_xdgOutput->setLogicalSize(pixelSize() / scale()); | ||
| 281 | m_xdgOutput->done(); | 288 | m_xdgOutput->done(); | ||
| 282 | } | 289 | } | ||
| 283 | 290 | | |||
| 284 | QSize AbstractWaylandOutput::orientateSize(const QSize &size) const | 291 | QSize AbstractWaylandOutput::orientateSize(const QSize &size) const | ||
| 285 | { | 292 | { | ||
| 286 | using Transform = DeviceInterface::Transform; | 293 | using Transform = DeviceInterface::Transform; | ||
| 287 | const Transform transform = m_waylandOutputDevice->transform(); | 294 | const Transform transform = m_waylandOutputDevice->transform(); | ||
| Show All 23 Lines | |||||
Please rename AbstractOutput::name() to AbstractOutput::description() to keep AbstractOutput::name() and XdgOutputInterface::name() in sync.