This is actually intended behavior, but one that's not clearly explained; if the size parameters are invalid, the media edit dialog will apply all other changes except for size (so to allow other changes like image type, alternate text, etc)

There are two ways to solve this bug imho:

1. Adjust the message to "Size values are invalid and will not be applied."

2. Make the 'apply' button inactive if size is invalid, forcing the user to find valid parameters or cancel -- but this will also mean that if other changes were made in the dialog, and the user doesn't know or want to change the size after seeing it is invalid (or wants to revert to the previous dimensions) we force them to cancel *all* changes and redo them again.

I am leaning towards #1, but we should discuss and see if we want to apply #2.

I am leaning towards #1, but we should discuss and see if we want to apply
#2.

Wont #2 be more obvious ?

The problem in my opinion with #2 is that if a user spent time working on new settings (caption / alternate text / position etc) we force them to deal with size before applying the rest. In general I think that might be less convenient in the long run.

I think the better approach (giving the user more leeway) is to alert the user that the parameter is wrong and won't be applied, save everything else, and allow the user to go back in and deal with the size widget specifically if they want to (or decide they don't want to deal with numerical values and resize with the mouse)

I think that while it's less like "others" are doing, this might give the user more leeway when fixing and working on stuff. Instead of forcing the user to deal with a problem they might just want to undo, we let the user save everything else and choose if they want to go back to retrying the size specifically.

I think this can be best "fixed" by altering the design and copy so that it's clear that invalid changes won't be put into effect.