I have a question regarding Previews in Jetpack Co...

# compose

[Rene Win]

I have a question regarding Previews in Jetpack Compose:
if i use the locale parameter within the Preview annotation like this:

@Preview(locale = "en")

then i assume that some code (in this example Locale.getDefault()) would also return this locale.
but it doesnt matter what i put in their, the Locale.getDefault() always returns the (i assume) system language
full preview code for completeness:

@Preview(locale = "en")
@Composable
fun Test() {
    Text("${Locale.getDefault()}")
}

The reason behind this is that we use the previews for screenshot testing and we have a datetimeformatter which depends on the current locale.. (which is calling Locale.getDefault() in the background)

[Stylianos Gakis]

I think the locale is passed inside the LocalConfiguration from this annotation and not to wherever java.util.Locale.getDefault() is looking at to fetch the locale.

Try adding a function like this

@Composable
@ReadOnlyComposable
fun getLocale(): Locale {
    val configuration = LocalConfiguration.current
    return ConfigurationCompat.getLocales(configuration).get(0) ?: LocaleListCompat.getAdjustedDefault()[0]!!
}

And see if this returns the right locale, I’m also curious

[Rene Win]

yes, this function does indeed return the locale from the locale preview parameter.
but where the Locale.getdefault is currently used i have no compose context - so therefore i cant call this function.
but it should be the same with the context right?
so context.resources.configuration should be the same as LocalConfiguration.current

[ephemient]

it should be the case that

LocalView.current.context == LocalContext.current
LocalContext.current.resources.configuration == LocalConfiguration.current

[Stylianos Gakis]

Yeah I think those two should return the same thing. It's just that for compose you also get for free that if the configuration changes, you will get the updated value. If you're not in a composable context, you need to make sure you get the refreshed configuration. Normally though, if you're not overriding this in your manifest the activity will be recreated and you will be forced to get the latest configuration that way.

[ephemient]

☝️ on configuration change, LocalView.current and LocalContext.current won't change, but LocalConfiguration.current will

[Stylianos Gakis]

Does this then mean that if you rely on LocalContext.current.resources.configuration you're going to be getting stale information about the configuration? Or did I misunderstand you?

[ephemient]

yes, it means that LocalContext.current.... may be outdated because it doesn't that part of the context has mutated

[yschimke]

There is a test harness module in Accompanist that can override with a fake Context. Maybe try that.

[ephemient]

that doesn't change the global JVM Locale either

[ephemient]

basically it's a bad fit for testability and you should avoid APIs that use the default locale

[ephemient]

(it's also a bad fit for Compose because Locale.setDefault() isn't observable)

[yschimke]

With the test harness module you could have a single preview, with a parameter provider with 20+ locales, and all the forms should match.

[Stylianos Gakis]

Thinking about all this, I remember another discussion revolving around the same exact thing tbh. I think a lint against using Locale.getDefault should be a great idea.

[Stylianos Gakis]

Yeah, it was this discussion https://kotlinlang.slack.com/archives/CJLTWPH7S/p1661546818567459?thread_ts=1661524832.340219&cid=CJLTWPH7S