For variable names, we're using an underscore between words. It's a bit strange that we're using capitalization to distinguish function names and underscores for variables, but just go with it, it's a legacy issue. Also, cblist doesnt' really tell me what this checkbox is for. In general avoid using names like "cb" (for example, this could be checkBox or comboBox). In this case, I would change this name to at least list_checkbox, and it would probably be a good idea to indicate what type of list it is. For example, is it a checkbox for a list of file names (I really don't know yet, because I haven't looked at the actual dialog, but if the name was clearer I would know)? If so, make it something like attachments_list_checkbox. In general, try to use long descriptive variable names for most things.