"Filter" is the first full word after the acronym. That's why I capitalized this word. Also, it matches the "name" field in the .info file. Oh yes, I know CamelCase is not allowed here by Drupal coding standards, but this rule is broken by a lot of modules, so I see it mostly as a recommendation, not really an enforced rule.
Sorry, I don't feel the need to change this. I want to keep using "WYSIWYG Filter" as the module name, and "WYSIWYG Filter" is used in some other strings in the module. If I change the label of the filter, for consistency, I would have to change "WYSIWYG Filter" in those places as well, and it's just a personal preference that I like to use CamelCase here.
The time spent of this could be spent on something else more productive.
Comments
Comment #1
mattyoung commentedpatch
Comment #2
markus_petrux commented"Filter" is the first full word after the acronym. That's why I capitalized this word. Also, it matches the "name" field in the .info file. Oh yes, I know CamelCase is not allowed here by Drupal coding standards, but this rule is broken by a lot of modules, so I see it mostly as a recommendation, not really an enforced rule.
Comment #3
mattyoung commentedIn core:
HTML filter
URL filter
HTML corrector
all have acronyms in front and lower case words follow. So
WYSIWYG Filter
it just look out of place
Comment #4
mattyoung commentedreconsider?
Comment #5
markus_petrux commentedSorry, I don't feel the need to change this. I want to keep using "WYSIWYG Filter" as the module name, and "WYSIWYG Filter" is used in some other strings in the module. If I change the label of the filter, for consistency, I would have to change "WYSIWYG Filter" in those places as well, and it's just a personal preference that I like to use CamelCase here.
The time spent of this could be spent on something else more productive.