aequery
aequery copied to clipboard
ScriptUI Capitalization Consistency
Original report by vaporstack (Bitbucket: vaporstack, GitHub: vaporstack).
There are a couple of minor inconsistencies in the casing of ui/container.js -
Notably, AddEditText / AddStatictext have dissimilar casing. Easy enough to fix but probably should be done with a deprecation warning to avoid breaking existing scripts.
Recommend we use the naming and casing as described in JavaScript Tools Guide -
* AddStatictext -> AddStaticText
* AddTreeview -> AddTreeView
* AddListbox -> AddListBox* *
A little bit annoying to decipher since the tools guide has some inconsistencies in itself, for example
#!javascript
ProgressBar / Scrollbar
Original comment by Rune Gangsø (Bitbucket: runegan, GitHub: runegan).
How should the deprecation warning work? Just throw an error stating: "addListbox has changed name to addListBox"?
Original comment by Rune Gangsø (Bitbucket: runegan, GitHub: runegan).
ScriptUI Capitalization Consistency
See #12 I don't know if there is any more inconsistency, that needs to be checked. The previous method names (addListbox, addStatictext and addTreeview)now points to the renamed methods (addListBox, addStaticText and addTreeView) to be backwards compatible.