Update color scheme to latest version

florianfelsing commented 3 months ago

Some more polishing to do here @krschacht, but feel free to take a glimpse whenever you feel like it.

krschacht commented 3 months ago

@florianfelsing I just pulled it and toggled back-and-forth between chatgpt and hostedgpt, and it looks great! The only thing I notice is that our left-column is too wide. To be honest, I think I inadvertently changed this in a recent PR. Could you change our left-column to match the width of ChatGPT?

I'll take a look at your diff and see if I have any feedback on implementation.

florianfelsing commented 3 months ago

Three things that come to mind that would still be open:

The actual ChatGPT theme also makes use of gradients in different places, which gives their interface some special touch. Not sure if we also wanna handle these within the scope of this task, though
The "Send message" should not be rounded, but have cornered edges. This was a little bit more tricky since I thought at first, though, because it seems like the tooltip is overwriting the icon's stroke width somewhere. Got a couple of hacky solutions in mind there, but nothing that I really liked so far
ChatGPT applies blur to their sidebar when you hover over the "Close sidebar" button - would probably move that into a separate issue

krschacht commented 3 months ago

@florianfelsing You have a great eye! These are really good things to notice. Let's defer these for now, I'm going to go ahead and merge in your PR. It's a great set of polish improvements.

I created an Issue to capture the items you just mentioned. And since I'm closing this PR, I started a discussion to talk about what's next. Let's hop over there: https://github.com/the-dot-bot/hostedgpt/discussions/151

AllYourBot / hostedgpt

Update color scheme to latest version #148