Opened on Aug 12, 2012 at 4:59:16 AM
Closed on Aug 14, 2012 at 7:32:22 PM
Last modified on Aug 22, 2012 at 4:12:36 AM
#2247 closed Bug (Fixed)
Documentation needs change for @Desktop macros
| Reported by: | Owned by: | trancexx | |
|---|---|---|---|
| Milestone: | 3.3.9.5 | Component: | Documentation |
| Version: | 3.3.8.1 | Severity: | None |
| Keywords: | Cc: |
Description
The macro reference for @DesktopWidth and @DesktopHeight creates confusion by describing these macros as being the size of the desktop screen. Both these macros return the size of Primary Monitor not the Desktop. This created a situation where I was spending ages looking for bugs that did not exist when using Melba23's ExtMsgBox because he's obviously taken the documentation at face value.
The desktop is the entire area that Windows allows you to use across all monitors, while each monitor has it's own individual area. These macro names should be changed, but obviously that's going to break a lot of existing code. Under the circumstances, I beleive the text in the documentation should be changed to point out the issue.
Best Regards,
Richard.
Attachments (0)
Change History (4)
comment:1 by , on Aug 14, 2012 at 7:32:22 PM
| Milestone: | → 3.3.9.5 |
|---|---|
| Owner: | set to |
| Resolution: | → Fixed |
| Status: | new → closed |
comment:2 by , on Aug 14, 2012 at 11:42:10 PM
The bit depth (@DesktopDepth) and refresh rate (@DesktopRefresh) are also per screen. I tested it, and they returned the primary screen for me. Could you please verify their workings and document that too?
comment:3 by , on Aug 16, 2012 at 9:01:04 PM
Yes, I can confirm that. It's the same thing, the info is for primary monitor.
Somebody with better English do it.
...Anonymous my ass.
comment:4 by , on Aug 22, 2012 at 4:12:36 AM
Err I tried. It became a bit long the way you wrote it so shortened and added the note to the other two.
Fixed by revision [7171] in version: 3.3.9.5