-
-
Notifications
You must be signed in to change notification settings - Fork 291
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
[WIP] [iOS] Media Item Menu - Edit Item Images #1345
base: main
Are you sure you want to change the base?
Conversation
- Upload image isn't working - Only a single image is shown per section. Need to make this the HCollection of all images for the group
@LePips Absolutely no rush on this at all. I'm still a ways away from being ready but let me know if you have any feedback on the UI! Not in love with anything I've done so far but I don't have a great sense of what this should look like. The edit button on the poster is because you can have more than one banner, etc. so I make it so you edited at that level opposed to from the section as a whole. The remote image section, with the option from remote sources from the server, is paginated so it only loads in 50 at a time. Web is only 25 so I can limit that further if we want. Functionally, I think my only issue I'm not sure of is I am using Finally, I'm trying to upload from a file. I've got it so it picks the file using the file picker and I have the upload API working right but I think the formatting for the image is off? I'm going from a local 'URL' to a file path on the device -> Data -> UIImage to run the following:
|
… because I think that's better. Spacing on the add screen is still all wrong but we're getting closer
TODO: - Spacing for remote portrait images is wrong & cramped - Upload image from file browser never works & produces 400 error - Show all images for an item.imageType opposed to just the first - Setting image works but produces a 400 error - Error alert looks bad
For each grid, we can make assumptions on what the shape of the images are going to be and have the layouts match accordingly. ie: primary is portrait, backdrop is landscape, etc. While anyone can upload arbitrary images, users going against the grain isn't a concern. For the image contents, we can instead show nothing. When an image is selected, lets push to a confirmation view that shows the larger image with the size and metadata source details. |
Selecting a Remote image is now working without error and works consistently! Upload a local file is still broken Item types with multiple images is working as intended now! Overriding an image on index doesn't seem to work but it doesn't work for Web either so........ UI is way more jank but the hard parts are getting solved!
Summary
Creates an image section for editing items. Allows for adding, deleting, or editing images for an item. Remote searching is working with all options from server.
Uploading a file is not working. Not sure why. I have the file browser but the data format is wrong.
I also have an issue where the cached images aren't updating after changes.
Screenshots
All Item Images
Simulator.Screen.Recording.-.iPhone.16.Pro.-.2024-12-07.at.18.54.32.mp4
Select From Remote Images
Simulator.Screen.Recording.-.iPhone.16.Pro.-.2024-12-07.at.18.54.51.mp4
Upload an Image
Simulator.Screen.Recording.-.iPhone.16.Pro.-.2024-12-07.at.18.57.40.mp4