Bug 306225 - workspace.displayHeight is wrong
Summary: workspace.displayHeight is wrong
Status: RESOLVED FIXED
Alias: None
Product: kwin
Classification: Plasma
Component: scripting (show other bugs)
Version: 4.9.0
Platform: Arch Linux Linux
: NOR normal
Target Milestone: 4.9.2
Assignee: KWin default assignee
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-09-04 01:48 UTC by Yichao Yu
Modified: 2012-09-04 04:28 UTC (History)
0 users

See Also:
Latest Commit: http://commits.kde.org/kde-workspace/8bcb32f0ca833be152e907d3f27bfa6d1b7085b3
Version Fixed In: 4.9.2
Sentry Crash Report:

Attachments
Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
Description Yichao Yu 2012-09-04 01:48:34 UTC 
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
Comment 1 Martin Flöser 2012-09-04 04:18:43 UTC 
typo, thanks for noticing
Comment 2 Martin Flöser 2012-09-04 04:28:08 UTC 
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