Custom theme token types not registering

[milesingrams]

Description

When changing theme tokens types stay with original theme

CodeSandbox/Snack link

No response

Steps to reproduce

1. Eject config
2. Change the tokens in the config to be something different than default

  tokens: {
     ...all other tokens,
    space: {
      '4xs': 2,
      '3xs': 4,
      '2xs': 6,
      xs: 8,
      sm: 12,
      md: 16,
      lg: 20,
      xl: 24,
      '2xl': 32,
      '3xl': 40,
    },
}

3. Use the theme overrides described in docs/ejected theme

type Config = typeof gluestackUIConfig; // Assuming `config` is defined elsewhere

type Components = typeof componentsConfig;

export type { UIConfig, UIComponents } from '@gluestack-ui/themed';

export interface IConfig {}
export interface IComponents {}

declare module '@gluestack-ui/themed' {
  interface UIConfig extends Omit<Config, keyof IConfig>, IConfig {}
  interface UIComponents extends Omit<Components, keyof IComponents>, IComponents {}
}

3. Pass the config to GluestackUIProvider

4. Types will throw errors when you try to use new tokens in components. Seems to only recognize old tokens

gluestack-ui Version

1.0.6

Platform

• [X] Expo
• [ ] React Native CLI
• [ ] Next
• [ ] Web
• [ ] Android
• [ ] iOS

Other Platform

No response

Additional Information

No response

[Viraj-10]

Hey @milesingrams, Thanks for reporting this issue. we will have a look. mean while can you please confirm if it is just typing that is not coming or tokens are not getting resolved as well.

[milesingrams]

The tokens still work as far as I can tell. I think the declare module '@gluestack-ui/themed' part isn't overriding the base types

[milesingrams]

So after looking into it more looks like overriding using declare module '@gluestack-style/react' works. Maybe the ejected config should use that instead of declare module '@gluestack-ui/themed'

[milesingrams]

I am still seeing a type issue with the ':transition' property which I believe might be unrelated but should be fixed:

Looks like it expects opacity to be number type but in the context of the :transition property it has a different structure

[milesingrams]

Same with duration on the :transition prop

[Viraj-10]

Hey @milesingrams, Thanks for reporting this issue. We will have a look.

[lhguerra]

Was any progress made on this?