Pholio currently doesn't show images or thumbnails on mobile.
Description
Revisions and Commits
rP Phabricator | |||
D9562 | rP4ea14ae37677 Make Pholio work a little better on mobile | ||
Restricted Differential Revision | rP5ba5cb567565 Add "thread" role and allow threads to render without thread lists | ||
Restricted Differential Revision | rP1f51d023af12 Improve Pholio on mobile |
Status | Assigned | Task | ||
---|---|---|---|---|
Restricted Maniphest Task | ||||
Resolved | epriestley | T1315 Phacility Launch Status | ||
Resolved | epriestley | T2795 Phacility (Mid Priority) | ||
Restricted Maniphest Task | ||||
Restricted Maniphest Task | ||||
Duplicate | None | T2644 Make Pholio work better on mobile | ||
Restricted Maniphest Task |
Event Timeline
What still needs to be done here? This is the last task under 'unbeta pholio', which itself is #2 on our roadmap.
@chad might have the best answer here, but let me dig through the security queue and I'll see what I can come up with.
I can't immediately find too many issues. Here's the stuff I see and/or remember after playing around with it a bit:
- There's no animation when swiping images.
- When you tap to zoom, we don't focus on the part of the image you tapped.
- Inline comment reticles are shown, but there is no way to interact with them (you can't see the associated comments for a given area).
- There is no way to add or edit inline comments.
- When editing a mock, the UI for image changes doesn't fit on screen. See screenshot below, the "X" is cut off.
- When editing a mock, the UI for adding images can't be interacted with (no way to "Drag and drop images here...").
Some reasonable remedies might be:
- Call swiping/zooming post-beta features.
- Don't render the inline rectangles on mobile.
- No adding/editing inlines via mobile for now.
- Make the edit UI a little smaller so it fits?
- Hide the add UI (no updating via mobile).
- Error on the "Create" UI?
I'm going to do these for now:
- Make the edit UI a little smaller so it fits?
- Hide the add UI (no updating via mobile).
And push everything else to v2.
With iOS 6+ it looks like we can add files from the phone, but we need to do all the post-to-hidden <iframe /> junk to make the uploads work.
(Or, apparently on recent-ish browsers, we can maybe use FormData to ajax <input type="file" ... /> controls. Let me see how far I can get with that.)
FormData does work (see T5187) but I think creating mocks from your phone is a v2+ feature since I can't really imagine any use cases which aren't off-label unless maybe you're drawing stuff out with a pencil and paper, which is something I actually do, but I am firmly not a designer so presumably real designers would find my methods and practices abhorrent.
My assumption is the add image from phone is an all around Phabricator feature, nothing specific with Pholio.
Yeah -- you can already do it in Files (since we have a normal upload there). T5187 will let you do it in comment boxes, and then we can extend that into Pholio eventually.