Skip to content
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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

馃殌 [Feature] Image cropping is optional #1934

Open
mattwr18 opened this issue Oct 16, 2019 · 6 comments
Open

馃殌 [Feature] Image cropping is optional #1934

mattwr18 opened this issue Oct 16, 2019 · 6 comments

Comments

@mattwr18
Copy link
Member

mattwr18 commented Oct 16, 2019

馃殌 Feature

The image cropping solution is great for some, but could be annoying for others

Design & Layout

maybe someone can add?
my idea was to have a normal preview of what the image would look like, with a button to crop an image, if one so chooses

Additional context

we have an additional issue that could be solved in another way that the cropped images are very big in comparison with ones that are not.

see #1933

@sushidave
Copy link

@mattwr18 Hi Matt. I agree with your suggestion. Cropping is the exceptional use case while not cropping is the standard one. Currently, we force all users to perform these steps after uploading an image:

  1. Increase image to 100%: Drag and drop at least two handles of the frame.
  2. Clicking the Confirm button.

Issues:

  • Some users forget to click the Confirm button and wonder why their uploaded image disappeared after saving the post.
  • The Confirm button is located on top instead of at the bottom. On smaller screens the button is out of sight after manipulating the frame. So it's prone to be forgotten.

The standard procedure should be like in Alpha where the default size was 100% and saving the post also saved the featured image.
Cropping (including clicking on the Confirm button) should be an option.

@datenbrei datenbrei added this to New in Roadmap via automation Nov 13, 2019
@datenbrei datenbrei moved this from New to Roadmap in Roadmap Nov 13, 2019
@Tirokk
Copy link
Member

Tirokk commented Nov 18, 2019

@DennisHack agrees with the idea of first put the post image in the post and then have a button for cropping. (Nov. 18, 2019)

As well see bug #1952 .

@DennisHack DennisHack moved this from Roadmap to Next Features in Roadmap Dec 8, 2019
@DennisHack DennisHack moved this from Next Features (Essential Features) to PRIOLIST in Roadmap Jan 7, 2020
@mattwr18 mattwr18 added the bounty For fixing this bug or implementing this feature you will receive a small financial compensation. label Jan 14, 2020
@alina-beck alina-beck removed the bounty For fixing this bug or implementing this feature you will receive a small financial compensation. label Feb 3, 2020
@stale
Copy link

stale bot commented Apr 3, 2020

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale label Apr 3, 2020
@sushidave
Copy link

See you later, stale bot 馃構

@stale stale bot removed the stale label Apr 3, 2020
@stale
Copy link

stale bot commented Jun 2, 2020

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale label Jun 2, 2020
@Tirokk
Copy link
Member

Tirokk commented Jun 3, 2020

Not stale

@stale stale bot removed the stale label Jun 3, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Roadmap
  
PRIOLIST (ALWAYS TO FINALISE)
Development

No branches or pull requests

4 participants