diff options
| author | Danilo M. <danix@danix.xyz> | 2026-09-14 19:31:00 +0200 |
|---|---|---|
| committer | Danilo M. <danix@danix.xyz> | 2026-09-14 19:31:00 +0200 |
| commit | 7304e3dd96f0bae4d63ccf3d4fa4f176b760a81f (patch) | |
| tree | e4094588b4122f7443a0c1a8f3724619fee994a9 /AGENTS.md | |
| parent | 32f88e68dfa9489ef09408b3dd34180b76357750 (diff) | |
| download | quickshell-7304e3dd96f0bae4d63ccf3d4fa4f176b760a81f.tar.gz quickshell-7304e3dd96f0bae4d63ccf3d4fa4f176b760a81f.zip | |
fix(desktop): name the kdeconnect injected property kc, not mod
The tile, page and row each took a required property named mod while the
module object's id is also mod. In a Component scope the id resolves to
the property's own unset self, so every page.mod read was undefined and
the device list never rendered. Only a runtime TypeError showed it; the
smoke check was clean because the drawer was closed.
kc matches the net and bt convention and cannot collide with the id.
Diffstat (limited to 'AGENTS.md')
| -rw-r--r-- | AGENTS.md | 7 |
1 files changed, 7 insertions, 0 deletions
@@ -174,6 +174,13 @@ changing that component. The ones that generalise: - **`qdbus` resolves a bare `org.kde.kdeconnect.device.<name>` as either a property or a method**, so `name`, `isReachable` and the `verificationKey()` method all use the same call shape. +- **A component property named the same as the enclosing object's `id` binds to + itself.** `KdeConnectModule` is `id: mod` and used to inject `KdeConnectTile { + mod: mod }`; in a `Component` scope that resolves to the tile's own unset + property, so `mod` arrived undefined and the page rendered no devices, with + only a runtime `TypeError` in the log and a clean smoke check (the drawer was + closed, so nothing was instantiated). Inject an id under a different name: + `net`, `bt`, `kc`. ## Theme |
