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

Case-insensitive tag lookup #50

Closed
neocotic opened this Issue Nov 30, 2011 · 2 comments

Comments

Projects
None yet
1 participant
@neocotic
Member

neocotic commented Nov 30, 2011

Basically, template tags are forced in to lower case before being looked up on the contextual data object.

Unfortunately, I'm not sure if this is currently possible with mustache.js but some further bespoke customisation may provide a solution.

Embedded documentation will also need to be updated to state that they are case-insensitive.

Predefined templates should be changed to reflect these changes.

@ghost ghost assigned neocotic Nov 30, 2011

@neocotic

This comment has been minimized.

Member

neocotic commented Dec 12, 2011

I'm hoping this could be as simple as changing the bespoke version of mustache.js to transform the contents of each variable to lowercase. This could possibly be done here;

      name = this.trim(name).toLowerCase();

Obviously, the properties will now need to be hard coded in lowercase instead. I think this is limited to the following methods;

@neocotic

This comment has been minimized.

Member

neocotic commented Dec 21, 2011

Worth noting (though internal docs cover this) that sections must be opened and closed using the same case.

@neocotic neocotic closed this Dec 21, 2011

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment