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

NLS Enablement of Core Node Categories #679

Closed
wants to merge 0 commits into from
Closed

NLS Enablement of Core Node Categories #679

wants to merge 0 commits into from

Conversation

kallenboone
Copy link
Contributor

Includes NLS enablement of core node categories. NLS enablement of user node categories is still in progress.

@@ -267,9 +267,13 @@ RED.palette = (function() {
function init() {
$(".palette-spinner").show();
if (RED.settings.paletteCategories) {
RED.settings.paletteCategories.forEach(createCategoryContainer);
RED.settings.paletteCategories.forEach(function(category){
createCategoryContainer(category, category); // TODO NLS enable categories from settings
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fyi, this can be handled by trying to lookup a message, but fall back to category if it doesn't exist:

RED._("palette.core."+category,{defaultValue:category})

Will add that when I get this merged.

@knolleary
Copy link
Member

I've manually cherry picked the relevant commit from your repository and force-pushed the rebased version of the code. Which basically means, this PR is no longer needed, and you'll need to refresh your fork to the latest code. The quickest way is to delete your fork and create a new one.

@knolleary knolleary closed this Jun 17, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants