integreat-cms icon indicating copy to clipboard operation
integreat-cms copied to clipboard

Published 20 hours ago • verified publisher icon digitalfabrik

Make it easier to set exact pixel sizes for images

Open timobrembeck opened this issue 3 years ago • 3 comments

Motivation

At the moment, the UI for setting the exact pixel sizes for images is very hard to find:

  1. One has to right-click on the image:

  2. left-click somewhere else in the browser (but not on the page) to hide the browser's inbuilt context menu while tinymce's context menu is still visible:

  3. Then open the option "Image..." in the tinymce context menu and only then, this window appears:

Proposed Solution

Come up with a good solution to make this window more accessible - maybe add a hover effect on the image with a small "edit" button or similar

Alternatives

Additional Context

timobrembeck avatar Oct 28 '21 08:10 timobrembeck

Unfortunately I can not reproduce this behavior. I am on a debian 10 and tested setting the pixel size in: chrome 95.0.4638.69 firefox 78.15.0esr in both cases only the tinymce menu shows up On which system did you encountered this behavior?

melegiul avatar Nov 03 '21 10:11 melegiul

Ok, I only experience the overlaying context-menu on my Firefox 94.0.2 (64-Bit) on arch linux. Then it's probably a weird browser issue which none of our users have.

Still, the way of right-clicking on the image to change the size is different than in wordpress and might lead to problems with our user base (e.g. the testing team from the municipality team was unable to change the pixel-size for images because they just didn't think of right-clicking on the image and then selecing "image..."). Instead, in wordpress, the following UI pops up when left-clicking on the image: wordpress-screenshot So even if this is a bit different, maybe it's enough to change the behaviour in a way that our context menu appears when the user left-clicks on the image instead of right-clicking? but this needs further clarification with UI/UX in my opinion. Probably, this isn't as concerning anymore and can be moved to the Version 1.1/1.2 milestone.

timobrembeck avatar Nov 20 '21 22:11 timobrembeck

I guess that the new pixel-sizing can be introduced in a new tutorial for the page editor, but we should introduce some new images sizes that we can deliver images that fit the selected pixel sizes. This means that we need to introduce another image class if we upload a image and add another HTML-Check if we save a page.

ulliholtgrave avatar Jan 06 '22 09:01 ulliholtgrave

I noticed this was a configuration problem on my side - I somehow had dom.event.contextmenu.enabled set to false in my firefox config.

timobrembeck avatar Mar 13 '23 13:03 timobrembeck