-
-
Notifications
You must be signed in to change notification settings - Fork 70
Feat/attachment improvements #856
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
base: dev
Are you sure you want to change the base?
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Deployed to https://msg-adamant-pr-856.surge.sh 🚀 |
|
Test failed: 18. Make previews size smaller for better looking
Expected: Message width = image preview size. |
|
Styling changes: 3. Show/Hide previews option on Settings page 3.1 If received an image and preview downloading is disabled, show background text PWA:
iOS for comparison:
Also, let's use not black background in the dark theme, but
Same colors both for images and file attachments.
|
|
Building Android App for this branch: |
|
✅ Test passed: 4. Slider improvements Chrome on Android
Android app built from this branch
|
|
Please describe how to test "5. Validate transaction structure using Zod" and provide link to code changes for this part. |
|
Test failed: 6. Display snackbar with error if the file does not exists on IPFS network
Proposed scenarioWhen unable to download a file
When all IPFS nodes are disabled by userIt's a special case. Check it before trying to download, priority check.
Preview of unavailable imagesIf a message has several attachments, and some are images that are not available, you can scroll to in preview. It's good, but preview is shown "blank".
Proposed solution: Show "Broken image" icon, and text
AdditionallyWhen we have "Download" button in Preview, don't duplicate it with "Download arrow" in the top right corner, it's excessive.
Texts |
|
Test link: https://adamant-2etqu1ijp-adamant-team.vercel.app/
✅ 2- Use color filters for svg to fit our design. Grayscale? Show some options, let us know how to play with it in debugger.
|
|
❌ Test failed: 11. Update Send file/ Send image button in Menu and actions ✅ 1- Update icons. Make them same size, emphasize (show some options) to make them a bit different
✅ 3- On desktops, these two are excessive, they do the same:
|
|
❌ 30. No Welcome coins screen when sending a file from new account Test link: https://adamant-2etqu1ijp-adamant-team.vercel.app/
Expected: App shows Welcome coins screen, and message with attachment preserved in the input field (User can send it after receiving free coins. |
|
❌ Test failed: 12. Update picked but not sent files selector Test link: https://adamant-2etqu1ijp-adamant-team.vercel.app/ ✅ 1- Make quote and files panels of the same size (I think full size is the best, like it’s file panel now) ✅ 2- For file panel, remove yellow bar to the left and make different background. I suppose we can remove background totally but keep thick left, top, right borders. ✅ 3- Disallow adding one image/file twice (currently you can add the same file several times)
|
|
❌ Test failed: 18. Make previews size smaller for better looking Test link: https://adamant-2etqu1ijp-adamant-team.vercel.app/ May be connected with: "31. Wrong scroll position after initial image sending to a static chat" (see below) Current preview size rules:
What's wrong
Expected: preview of 6 images in chat
Proposed solution
|
|
31. Wrong scroll position after initial image sending to a static chat
Expected: Scroll position is in the bottom after sending an image (1) and after receiving reply (2) Video: bad-scroll.mov |
|
Test link: https://adamant-2etqu1ijp-adamant-team.vercel.app/ Add spaces after file/image count.
|
|
❌ Test failed: 5. Validate transaction structure using Zod Test link: https://adamant-2etqu1ijp-adamant-team.vercel.app/ If the transaction doesn’t match the structure described in AIP-18 it must be marked as “Unsupported“. Test case:
To reproduce, quote the very first message in this chat, and reply with attaching a file.
Sender: Message is good. After re-login: Message mistakenly treated as not supported. |
|
✅ Test passed:
Test link: https://adamant-2etqu1ijp-adamant-team.vercel.app/ After choosing a file:
⟶
|
|
✅ Test passed:
Test link: https://adamant-2etqu1ijp-adamant-team.vercel.app/ After sending files:
|
Environment: Android app, Web app in Safari on iPhone/iOS How it's in Android app:
Compare how it's in Chrome on Android:
Can we do in similar way, with a possibility to show Downloads folder? Same is in the Web app in Safari on iPhone/iOS — can we provide a possibility to show Downloads folder? |
|
Test link: https://adamant-2etqu1ijp-adamant-team.vercel.app/
Expected: 250 Please describe how to test "16. Check file size before and after encryption" and provide link to code changes for this part. |
|
"17. Fix scroll when adding/removing attached files" is probably connected with "31. Wrong scroll position after initial image sending to a static chat" |
|
✅ Test passed:
Test link: https://adamant-2etqu1ijp-adamant-team.vercel.app/ When Full media auto-download are not enabled in settings, app downloads only previews. Full media available by request.
|
|
Please describe how to test "22. Better native support for download/preview files on Android" and provide link to code changes for this part. Is it connected with "32. Downloading files don't have clear UX on mobile devices"? |
|
Please describe how to test "23. Get CID from IPFS instead of generating it locally" and provide link to code changes for this part. |
|
✅ Test passed: 24. Move the download tasks to the background instead of running them only while the modal is open Test link: https://adamant-2etqu1ijp-adamant-team.vercel.app/
Same with images, app downloads them in the background. |
|
✅ Test passed: 28. Now 250mb is a barrier for every single file instead of combination of all files attached to the message Test link: https://adamant-2etqu1ijp-adamant-team.vercel.app/ Added and sent 4 files ~270 MB total.
⟶
|


































Testing
Additionally:
Currently has issues
For details, read comments below.