Athou / commafeed

Google Reader inspired self-hosted RSS reader.
https://www.commafeed.com
Apache License 2.0
2.59k stars 362 forks source link

Text Zoom on mobile #1462

Open FrozenSource opened 2 weeks ago

FrozenSource commented 2 weeks ago

Is your feature request related to a problem? Please describe. Hi I have been trying out the webapp and have been impressed. There is one issue and that is that on mobile (adding commafeed to the homescreen) I am unable to set the text size.

Describe the solution you'd like While it is doable with a lot of effort (since I am very short sighted) I would love to have a text size settings for the titles and text.

Describe alternatives you've considered I tried out a custom css with "* { font-size: 24pt; }" but it doesn't seem to work and I feel stupid.

Would love to hear from you guys whether I am stupid and haven't found the setting or otherwise.

Crul commented 2 weeks ago

Note: I'm not the developer, just a random user.

I tried out a custom css with "* { font-size: 24pt; }" but it doesn't seem to work and I feel stupid.

I think you need to add !important, this works for me:

* { font-size: 24pt !important; }

FrozenSource commented 2 weeks ago

I tried it on iPhone btw, even with the important it does not seem to work.

Crul commented 2 weeks ago

That's weird, it seems to work emulating the iPhone with Firefox. See screenshot. image

The only thing I can think of is reloading the page (which you probably have already done it, but just in case).

FrozenSource commented 2 weeks ago

Using Brave I also see that custom css is empty with cache disabled:

image
FrozenSource commented 2 weeks ago

When loading with Safari (which I never used it with) it also has the custom css empty.


services:
  commafeed:
    image: athou/commafeed:latest
    restart: unless-stopped
    environment:
      - CF_DATABASE_DRIVERCLASS=org.postgresql.Driver
      - CF_DATABASE_URL=jdbc:postgresql://postgresql:5432/commafeed
      - CF_DATABASE_USER=commafeed
      - CF_DATABASE_PASSWORD=commafeed
    volumes:
      - ./data:/commafeed/data
    networks:
    - default
    - traefik-public
    labels:
    - traefik.enable=true
    - traefik.http.routers.commafeed.rule=Host(`rss.something.org`)
    - traefik.http.services.commafeed.loadbalancer.server.port=8082

  postgresql:
    image: postgres:latest
    restart: unless-stopped
    environment:
      POSTGRES_USER: commafeed
      POSTGRES_PASSWORD: commafeed
      POSTGRES_DB: commafeed
    volumes:
      - ./db:/var/lib/postgresql/data

networks:
  traefik-public:
    external: true```
Crul commented 2 weeks ago

That's weirder.

I will let this one for Athou. I'm testing in a local instance, version 4.4.1 (cd06055). It's possible it's not the latest one and there is a bug somewhere.

FrozenSource commented 2 weeks ago

Alright I'll await any questions... creating a new user also doesn't do it as I changed the username of admin I thought it might have something to do with that, but that doesn't seem to be the case.

For note I pulled it yesterday (also got watchtowerr), about gives me: CommaFeed version 4.4.1 (cd06055).

So that ought to be the same version you were using....

Athou commented 2 weeks ago

Maybe a caching issue, I think custom js and css endpoints are not in the cache busting filter. @FrozenSource does it work if you hard reload the page by right clicking on the reload icon while the developer console is open?

image

FrozenSource commented 2 weeks ago

It is not the browsers caching.... it's cloudflare's. I cleared and disabled the cache which does its job for now. Might it be an option to either increase font-size through a setting+JS. Or something that might not be as amazing provide a no-cache header for the custom theme?

Athou commented 1 week ago

Sure, I'll keep this issue open to:

FrozenSource commented 1 week ago

Great!