-
Notifications
You must be signed in to change notification settings - Fork 2.9k
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
Web - Attachment - No magnifying glass seen when hoovered over image in chat #6075
Comments
Triggered auto assignment to @CortneyOfstad ( |
Triggered auto assignment to @aldo-expensify ( |
Hey @mvtglobally , can you help me reproduce this bug? Does the step (1) include clicking the "Send" button? When you say "Select image that was uploaded", are you referring to clicking on its thumbnail on the chat?, thanks! :) |
@aldo-expensify Yes. You need to send the image to another user first. |
Issue reproducible during KI retests. |
Thanks, so... are we missing the magnifying glass on the thumbnail image? |
I'm not 100% convinced about that the cursor over the thumbnail should be a magnifying glass. It makes sense to me once the image is "opened" and clicking it zooms in, but here you are not exactly just zooming in, you are opening a modal with the image which has a download button. Where does the information for the "Expected result" come from? Is it written in some specification? For example, in Whatsapp, it works exactly like that. You get the hand (pointer?) cursor when hovering the thumbnail and you get the magnifying glass after you open it. |
I'm closing this as I don't see it as a problem. |
If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!
Action Performed:
Expected Result:
Magnifying glass would be visible when hovering over image
Actual Result:
No magnifying glass
Workaround:
n/a
Platform:
Where is this issue occurring?
**Version Number:v1.1.10-0
**Reproducible in staging?:Y
**Reproducible in production?:Y
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
Video
Expensify/Expensify Issue URL:
Issue reported by:
Slack conversation:
View all open jobs on GitHub
The text was updated successfully, but these errors were encountered: