deepfriedmemes.com icon indicating copy to clipboard operation
deepfriedmemes.com copied to clipboard

The site does not work for me.

Open JonathanGorr opened this issue 1 year ago • 8 comments

I upload an image and nothing happens, png and jpg types.

JonathanGorr avatar Feb 22 '25 19:02 JonathanGorr

Thanks for reporting. Browser, OS?

I've recently had someone email me about this too but it started working for them again somehow. I could never reproduce it on any of my devices/browsers.

To be clear, there's no upload happening. The entire image loading+processing routine runs locally in javascript, and I haven't changed that since 2017, so this tells me that something is changing in people's clients.

If possible please provide any relevant error messages from the browser javascript console.

efskap avatar Feb 22 '25 23:02 efskap

It didn't work initially on Edge and Chrome. After some minutes or hours it did work when I loaded the page again. I cleared the local storage, cache, cookies etc and again it stopped working; doesn't do anything after selecting an image to upload. There doesn't appear to be anything of interest in the console; no errors just info logs.

Issues tab:

Accessibility

Form elements must have labels: Element has no title attribute Element has no placeholder attribute

Compatibility '-ms-text-size-adjust' is not supported by Chrome, Chrome Android, Edge, Firefox, Safari. Add 'text-size-adjust' to support Chrome 54+, Chrome Android 54+, Edge 79+. '-webkit-text-size-adjust' is not supported by Chrome, Chrome Android, Edge 79+, Firefox, Safari. Add 'text-size-adjust' to support Chrome 54+, Chrome Android 54+, Edge 79+. 'content-type' header charset value should be 'utf-8'.

Performance A 'cache-control' header is missing or empty. A 'cache-control' header contains directives which are not recommended: 'must-revalidate' Resource should use cache busting but URL does not match configured patterns. The target should not be cached or have a 'cache-control' header with 'max-age=180' or less.

Security Response should include 'x-content-type-options' header. The 'Expires' header should not be used, 'Cache-Control' should be preferred. The 'X-Frame-Options' header should not be used. A similar effect, with more consistent support and stronger checks, can be achieved with the 'Content-Security-Policy' header and 'frame-ancestors' directive.

Other 'touch-action' should be listed after '-ms-touch-action'. 'transform' should be listed after '-webkit-transform'.

JonathanGorr avatar Feb 23 '25 01:02 JonathanGorr

Thanks for helping debug, that's very bizarre!

In the broken state, does the upload button say "Browse (or paste)" or just "Browse"? I'm curious because the "or paste" part is added on by the javascript (if the user agent isn't a mobile one), and this would clarify if the JS is failing to run entirely. That's the only reason that comes to mind for nothing happening without errors being printed.

I still can't reproduce on Firefox and Chromium even after clearing site data

efskap avatar Feb 23 '25 02:02 efskap

Hi, uh, the website isn't working for me as well. btw, the button says "Browse" instead of "Browse (or paste)".

OwenF2019 avatar Jul 26 '25 15:07 OwenF2019

Maybe the remote javascript libraries aren't being fetched correctly sometimes. Considering that it still worked fine on all my devices and I haven't touched the code in a long time, this is the only thing that comes to mind at the moment.

I downloaded the remote dependencies to the vendor directory in 55db6d6bea0007a6b016102eb5d1ad119bc50ca2

Let me know if the issue persists

efskap avatar Aug 03 '25 04:08 efskap

Site doesn't seem to work for me either, it does show Browse (or paste) though.

LiverNOnions25 avatar Oct 18 '25 04:10 LiverNOnions25