search

AllYourBot / hostedgpt

An open version of ChatGPT you can host anywhere or run locally.
MIT License
304 stars 122 forks source link

Some visual fixes for PWA (no hover, no scroll) #371

Closed jasonpaulso closed 1 month ago

jasonpaulso commented 2 months ago

Fixes:

krschacht commented 2 months ago

Hey @jasonpaulso, I pulled down the changes and I'm not sure I'm experiencing them all. The scrolling on login/signup is definitely gone for me. But after I login, I still have to tap the conversations twice to get them to select (the hover state thing) and if I accidentally long tap it still does a flyout. Are they working differently for you?

krschacht commented 1 month ago

Hi @jasonpaulso I should probably just go ahead and merge this in. But I wanted to check and see if you saw my comment: when I tested out the PR I saw that you had fixed the login page scroll but I wasn’t able to experience the other things you mentioned.

jasonpaulso commented 1 month ago

Sorry for my late reply—I took last week off from email, etc. I’ll be having a look at this a bit later this morning, I have a guess at where I belied my PR.On Jun 2, 2024, at 9:52 PM, Keith Schacht @.***> wrote: Hi @jasonpaulso I should probably just go ahead and merge this in. But I wanted to check and see if you saw my comment: when I tested out the PR I saw that you had fixed the login page scroll but I wasn’t able to experience the other things you mentioned.

—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: @.***>

krschacht commented 1 month ago

I merged main back into here and resolved the merge conflicts, then I tested your branch out and all the fixes worked for me!

I was getting ready to merge in and I noticed that CI system tests are failing on this branch and I'm baffled by it. When I run the tests locally, everything works fine. But in this branch CI runs the system tests and a dozen of them fail. It's so odd!

I'm going to try slowly reverting everything one by one in order to isolate what could possibly be causing things to work fine for us locally but not in CI.

krschacht commented 1 month ago

@jasonpaulso I've narrowed the issue down to this change in the tailwind.config:

+  future: {
+    hoverOnlyWhenSupported: true,
+  },

It causes all of these broken test cases in the CI environment (but not locally): https://github.com/AllYourBot/hostedgpt/actions/runs/9438596249/job/25995746220

Any idea why? I did a little digging online and don't have a good idea for how to fix. I know we want this change because I assume this is what fixes the hover issue on mobile. I finally came up with a solution which is super hacky: in the CI script it simply uses sed to quickly edit the file and flip that flag.