Skip to content

Feature/multi photo - #143

Open
zabaznov-roman wants to merge 15 commits into
Notely-Voice:mainfrom
zabaznov-roman:feature/multi-photo
Open

Feature/multi photo#143
zabaznov-roman wants to merge 15 commits into
Notely-Voice:mainfrom
zabaznov-roman:feature/multi-photo

Conversation

@zabaznov-roman

Copy link
Copy Markdown

No description provided.

@tosinonikute

Copy link
Copy Markdown
Member

@zabaznov-roman Thanks, I appreciate the PR and I'll give this a test first & then do some reviews :-)

Also in future its good if you have a screenshot or video that helps to quickly see how it works before running it.

@tosinonikute
tosinonikute self-requested a review August 21, 2026 12:40
@tosinonikute

Copy link
Copy Markdown
Member

This works well. A couple of suggestions:

  1. The delete icon for each item in the horizontal list should have a round background to make it more visually pleasing.
  2. When a user taps an image to view it larger, there should be a back button they can tap to close that view. Right now, a user who doesn't know where to tap to close it might get confused.

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.

2 participants