-
Notifications
You must be signed in to change notification settings - Fork 7.7k
Provide i18n for InlineBezierCurveEditor #5553
Conversation
|
||
InlineBezierCurveEditor = require("InlineBezierCurveEditor").InlineBezierCurveEditor, | ||
BezierCurveUtils = require("BezierCurveUtils"); | ||
BezierCurveUtils = require("BezierCurveUtils"), | ||
EmbeddedStyles = require("text!EmbeddedStyles.css"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Name of new css file should have better description -- something like localized.css
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
But maybe we want to add some more CSS rules later, wo this is a general name for the file.
But I can rename it if you still want to.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There's already main.css
for general rules. This new css file is only for localized content and the name should reflect that to help prevent other rules from getting put in it.
@SAplayer Brilliant! Done with code review. |
Just done the rename. Hope it's ready for merging now. |
Thanks. Merging. |
Provide i18n for InlineBezierCurveEditor
Provide a translation (currently english + german) for the InlineBezierCurveEditor which was introduced with #5550.
CC @redmunds
Check german translation: @ingorichter @couzteau @pthiess