-
-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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 Photo Editor #9575
Web Photo Editor #9575
Conversation
Closing as stale/superseded by #10989 |
This is for the web while #10989 is for the mobile app |
Will reopen when the work restart |
how come this got closed? |
@byjlw as stated above, there is an opened pr that address this feature and likely to be merged |
Good Afternoon! I know this is an old PR, but I was trying to understand if there is an active plan to add basic edit mode (crop/rotate) to the web version and NOT just the mobile version? Or is there, and I just missed it? The referenced PR ( #10989 ) seems to be mobile only. Thanks! ~Lex |
@lex-obrien If I've been following along correctly, #11074 already added a (hidden for now) web editor, which is waiting for a server side implementation to save edited photos, which is a draft in #11658 so that's where I'm following along. |
Ahhhaa! Thank you @stephen304 , that was the PR that I was missing! |
Based on the work started by @faupau03 here: #3271
The vast majority of the work is theirs, opening a new PR so that I can keep up-to-date with upstream.
There's a bit of work required to get this ready for review. I'm going to focus on some refactoring for maintainability and tests, and finishing off the crop feature before submitting for review.
TODO:
Out of scope for this version: