Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

undefined var 'bounds' on type #184

Closed
Kaiochao opened this issue May 14, 2020 · 1 comment · Fixed by #185
Closed

undefined var 'bounds' on type #184

Kaiochao opened this issue May 14, 2020 · 1 comment · Fixed by #185
Labels
enhancement good first issue
Milestone
suite v1.5

Comments

@Kaiochao
Copy link

There is a built-in variable /atom/movable/var/bounds which takes a string. Yeah, it's undocumented...

http://www.byond.com/forum/post/227863#comment1877753
Cyberboss added a commit to Cyberboss/SpacemanDMM that referenced this issue May 14, 2020
@Cyberboss
Add /atom/var/bounds to builtins
134f362 
Fixes SpaceManiac#184
@Cyberboss Cyberboss mentioned this issue May 14, 2020
Merged
SpaceManiac pushed a commit that referenced this issue May 15, 2020
@Cyberboss
Add /atom/var/bounds to builtins (#185)
35915f7 
Fixes #184.
@Kaiochao
Copy link
Author

Kaiochao commented May 16, 2020 •
edited
Loading

Also, dreamchecker doesn't seem to know about client.bound_x/y/width/height which are technically documented under client.bounds. And movable.bounds, not sure if this fixes that.

@SpaceManiac SpaceManiac added this to the suite v1.5 milestone Jul 11, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement good first issue
Projects
None yet
Milestone
suite v1.5
Development

Successfully merging a pull request may close this issue.

Add /atom/var/bounds to builtins Cyberboss/SpacemanDMM
2 participants
@SpaceManiac @Kaiochao