Skip to content
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

Allow viewing photos after attaching to note #4797

Merged
merged 4 commits into from
Feb 7, 2023

Conversation

Helium314
Copy link
Collaborator

@Helium314 Helium314 commented Feb 7, 2023

Fixes #2497.

See discussion in Helium314#392. The photo is shown in a dialog with transparent background.
I hope the delete button is now big enough.

screen

@westnordost
Copy link
Member

Nice! Hmm, the delete button is indeed a bit big now (also, scaling it will make it blurry, the actual graphic either have to be made larger or at least the view displaying it). I propose 32dp size for the delete button and 72dp or so for the image thumbnail.

How come the third picture is cut off that way? I wonder if there is a way it can be made clear that this scrolls vertically. Maybe always show the scroll bar? Or show some kind of gradient from background-color to transparent at its edges?

@Helium314
Copy link
Collaborator Author

scaling it will make it blurry

This is weird, I wonder why Android Studio can scale it without blurring (in the preview)

How come the third picture is cut off that way?

That's normal, but previously I think 4 or 5 pictures were necessary.
Is there really anything to do? When the users see that the 3rd photo they attach is cut off, they might try to scroll.
But a scrollbar might still help. Can the scrollbar be always shown, but only if scrolling is actually possible?

@Helium314
Copy link
Collaborator Author

Ok, the scrollbar shows up for a few seconds once it becomes necessary. That's hopefully enough.

@westnordost
Copy link
Member

This is weird, I wonder why Android Studio can scale it without blurring (in the preview)

Android has some optimizations built-in for vector graphics. It usually caches a bitmap variant of the vector graphic after rendering it once. The size of that bitmap depends on the defined size of the vector graphic.

@westnordost
Copy link
Member

I made some improvements I didn't want to bother you with

@westnordost westnordost merged commit 74cba55 into streetcomplete:master Feb 7, 2023
@westnordost westnordost deleted the note_zoom branch February 7, 2023 16:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Option to view photo before uploading it to new note.
2 participants