Skip to content
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
Closed

Case-insensitive tag lookup #50

neocotic opened this issue Nov 30, 2011 · 2 comments
Assignees
Labels
Milestone

Comments

@neocotic
Copy link
Member

@neocotic 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
Copy link
Member Author

@neocotic 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
Copy link
Member Author

@neocotic 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
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
1 participant