-
Notifications
You must be signed in to change notification settings - Fork 74
Added docs to Scene_Base #90
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.
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
Conversation
Added documentations to the Scene_Base class.
Updated the jsdoc for the initialize for remove the constructor tags since it's provoking a bug to the jsdoc generations or intelisenses. The Also added "@instance" to method for tell the system it's not a static class.
|
We'll remove version info, because It is need in _header.js only. Is it ok? |
|
wether => whether? |
|
@liply Sure! I am sorry I forgot to answer was busy with stuff! as for the Whether sorry it's was a misspelling of myself do you want me to fix it? |
|
could you fix it? |
Removed the Header and corrected the misspelling error
|
@liply : Sure! |
js/rpg_scenes/Scene_Base.js
Outdated
| Scene_Base.prototype.update = function() { | ||
| this.updateFade(); | ||
| this.updateChildren(); | ||
| AudioManager.checkErrors(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I removed this line before.
Could you remove?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sure I guess my version was a little outdated!
Removed AudioManager.checkErrors();
krmbn0576
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice! ❤️
Added documentations to the Scene_Base class.