-
Notifications
You must be signed in to change notification settings - Fork 1.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[BUG] Cannot view asset's info with qutebrowser or falkon #2606
Comments
Could it be those browsers don't support the specific API call to convert the date and time info to the appropriate locale? |
That's more than likely the root cause here. You could try running some of the example here and report back. |
Commented constructors below do not work in my browser, expected results generated on Firefox. const date = new Date(Date.UTC(2020, 11, 20, 3, 23, 16, 738));
console.log(new Intl.DateTimeFormat('en-US').format(date));
// Expected: "12/20/2020"
console.log(new Intl.DateTimeFormat('en-US', { timeZoneName: 'short' }).format(date));
// Expected: "12/20/2020, GMT"
console.log(new Intl.DateTimeFormat('en-US', { timeZoneName: 'long' }).format(date));
// Expected: "12/20/2020, Greenwich Mean Time"
//console.log(new Intl.DateTimeFormat('en-US', { timeZoneName: 'shortOffset' }).format(date));
// Expected: "12/20/2020, GMT"
//console.log(new Intl.DateTimeFormat('en-US', { timeZoneName: 'longOffset' }).format(date));
// Expected: "12/20/2020, GMT"
//console.log(new Intl.DateTimeFormat('en-US', { timeZoneName: 'shortGeneric' }).format(date));
// Expected: "12/20/2020, United Kingdom Time"
//console.log(new Intl.DateTimeFormat('en-US', { timeZoneName: 'longGeneric' }).format(date));
// Expected: "12/20/2020, United Kingdom Time" |
Didn't find any compatibility data for this and I had to search through the ECMAScript spec to even get some answers. In the 2021 spec The browsers you're trying to use are Chromium based, correct? Do you know which version perhaps? We could use |
Yes it's Chromium based. Backend: QtWebEngine 5.15.14, based on Chromium 87.0.4280.144 Full version info
There is some effort going on to port the browser to qt6 webengine, which I suppose will have a newer version of Blink. |
I gave this a go on the qt6 version, and can still reproduce the issue. Backend: QtWebEngine 6.3.2, based on Chromium 94.0.4606.126 (from api) Full version info
|
QtWebEngine 6.3.2 seems to be running the last version of Chromium that does not support the extensions. |
QtWebEngines 6.4 and onwards do support them and I can confirm that I cannot reproduce the bug with "Backend: QtWebEngine 6.5, based on Chromium 108.0.5359.220 (from api)". Happy to have this resolved as "wontfix, update your browser" if you'd rather not change the |
There haven't been any other reports for this error and you're using an older Chromium version, so I'll leave it for now. We can always revisit this if more people run into this issue. |
The bug
Clicking the "Info" button on an image produces the error below and nothing happens.
This happens consistently with my usual browser qutebrowser and other qtwebengine based browsers like falkon. I cannot reproduce it with Firefox or Chromium.
The OS that Immich Server is running on
Arch
Version of Immich Server
v1.58.0
Version of Immich Mobile App
N/A
Platform with the issue
Your docker-compose.yml content
N/A
Your .env content
Reproduction steps
1. Open qutebrowser or falkon 2. Click on picture 3. Click on "Info" 4. Nothing happens 5. Console now contains above errors
Additional information
No response
The text was updated successfully, but these errors were encountered: