Alex

Results 1105 comments of Alex
trafficstars

I think it makes sense to include the save functionality in this PR as well, if we merge it to main and have to push out a release, there will...

I think we want to keep different editing features in separate PRs, but each should have a finished state. I am fine with hiding the button in this PR and...

Merge means getting this into `main` so that you don't have to keep working around merge conflicts, which happens often since many PRs get merged into `main` every day.

> if I have not forgotten anything, then I have fulfilled all the wishes from your review. but tomorrow I want to add a pop-up question when trying to close...

I would like to get this PR merged in `main` before it gets out of hand with the changes. Let's finish up the UI and then have it merged. Then...

If you guys temporarily disable hardware acceleration, does it work normally?

@dodg3r if you can include a sample file, that would be great

@jakub-antos are you using any hardware acceleration? Look like they are not compatible with the newer version of HEIC

@jakub-antos your docker compose file is out of date, there is no microservices anymore. I assume any photo you upload will not have thumbnail generated, not just HEIC

Perhaps some weirdness with iCloud/Shared Album. I'd suggest just let it run until it stops