unkeyed / unkey

Open source API management platform
https://go.unkey.com
Other
4k stars 469 forks source link

Login page shows weird, when logging out #522

Closed bipulpoudel closed 11 months ago

bipulpoudel commented 11 months ago

Preliminary Checks

Reproduction / Replay Link (Optional)

No response

Description

The login page is shown weird css when you get redirected after signout.

image

Environment

No response

perkinsjr commented 11 months ago

This is because of Dark mode. We never do any dark mode for our sign in pages. Should be fairly simple to hard code the classnames to use bg white

bipulpoudel commented 11 months ago

I can pick this up? I have fixed it.

bipulpoudel commented 11 months ago

I have added dark mode in sign-in page, should we not add it? Because, the ThemeProvider cache is giving this error.

bipulpoudel commented 11 months ago

I have created this:

image

I guess, it looks good though?

perkinsjr commented 11 months ago

Dark mode should not be added to the sign in and sign up

bipulpoudel commented 11 months ago

Aah okayy, I would fix it by removing cache then :)

perkinsjr commented 11 months ago

Or you could just set the colors to bg-white and bg-black across the board and leave the cache alone so the user next log in doesn't get reverted

bipulpoudel commented 11 months ago

Sounds like a plan :), Thanks

bipulpoudel commented 11 months ago

Hi @perkinsjr , one silly question, is it okay to add theme provider as light in layout for auth?

It is working perfectly fine, no change in code needed at all, but is it something we can do?

Something like this:

image

Code wise, it's much cleaner but not sure, if this is recommended, didn't find anything in next documentation

perkinsjr commented 11 months ago

There is no need to add a theme provider. We have zero plans to add light or dark mode.

bipulpoudel commented 11 months ago

@perkinsjr , I have created PR :)

bipulpoudel commented 11 months ago

This is fixed :), closing issue now.