Add --default-to-base option for code command? about bartycrouch HOT 4 CLOSED

Thanks @Dschee.

The reason is simple: The best practices I know all recommend to use structured keys like "HOME_SCREEN.NETWORK_ALERT.TITLE" in code and such a key simply wouldn't make any sense.

I'm a bit confused, copying across the keys would be the default-to-keys option, which already exists for code.

Also there are tools like Linguan that can show you a list of all your empty localization Strings, so empty Strings are easier to check then pre-filled Strings.

That's true, though we use WebTranslateIt which keeps track of what has and hasn't been translated for us.

We are using structured keys, so as you say the default-to-keys option isn't a good fit for our app. I'll see if I can schedule some time to create a PR.

Cheers.

from bartycrouch.

The reason is simple: The best practices I know all recommend to use structured keys like "HOME_SCREEN.NETWORK_ALERT.TITLE" in code and such a key simply wouldn't make any sense. Also there are tools like Linguan that can show you a list of all your empty localization Strings, so empty Strings are easier to check then pre-filled Strings.

The reason the interface subcommand has that option is that usually in Interface Builder (even considering best practices) you already have some kind of meaningful text. So, using that kind of makes sense (compared to the keys in code).

But having that all said, I'm not against adding that option – feel free to implement it and post a PR, I sure will have a look as soon as I can.

from bartycrouch.

Ahh, I see now what you mean. Yes, now you reminded me: default-to-keys is kind of like the equivalent to default-to-base for code as it uses the thing you usually have to provide anyways (the default value in the Base Storyboard/XIB or the localization key) and uses that to prefill values.

I did never think about adding a Base language to the Localizable.strings file though to use a default translation, which is, if I understood you correctly what you're trying to do. That of course is a new feature then and would also make sense. If you find the time to do it, it would also be great to explain a usage example this in detail within the README file.

from bartycrouch.

I'm closing this due to inactivity. Feel free to send a PR with such a feature and reference this issue so I can reopen it until the PR is merged.

from bartycrouch.