| Summary: | workspace.displayHeight is wrong | ||
|---|---|---|---|
| Product: | [Plasma] kwin | Reporter: | Yichao Yu <yyc1992> |
| Component: | scripting | Assignee: | KWin default assignee <kwin-bugs-null> |
| Status: | RESOLVED FIXED | ||
| Severity: | normal | ||
| Priority: | NOR | ||
| Version: | 4.9.0 | ||
| Target Milestone: | 4.9.2 | ||
| Platform: | Arch Linux | ||
| OS: | Linux | ||
| Latest Commit: | http://commits.kde.org/kde-workspace/8bcb32f0ca833be152e907d3f27bfa6d1b7085b3 | Version Fixed In: | 4.9.2 |
| Sentry Crash Report: | |||
typo, thanks for noticing Git commit 8bcb32f0ca833be152e907d3f27bfa6d1b7085b3 by Martin Gräßlin. Committed on 04/09/2012 at 06:26. Pushed by graesslin into branch 'KDE/4.9'. Return displayHeight for property displayHeight and not displayWidth Typo in the scripting bindings. FIXED-IN: 4.9.2 M +1 -1 kwin/scripting/workspace_wrapper.cpp http://commits.kde.org/kde-workspace/8bcb32f0ca833be152e907d3f27bfa6d1b7085b3 |
the value of workspace.displayHeight is actually displayWidth although displaySize is correct. from output of print(JSON.stringify(workspace)); "displaySize":{"w":1920,"h":1200},"displayWidth":1920,"displayHeight":1920," (not sure if it is fixed in git or 4.9.1 but at least it is wrong for 4.9.0) Reproducible: Always