We probably need this since it's used with image.

The event names need to be tensed, ratio probably needs a rename to aspect_ratio?

Agreed, and ratio should be aspect_ratio for sure.

More change proposals:
border --> border_size
border_center_fill --> center_fill_mode

Small question

(p) image_size, does it mean original image size, or image size after applying ScaleType ?
I am asking this, because sometimes user need to calculate scale-factor for image. (origin vs current)

As the documentation says: This represents the size of the original image in pixels

I suggest small Idea
Property to read scale_factor (Width, Height).

In T7875#141363, @segfaultxavi wrote:

More change proposals:
border --> border_size

Maybe border_insets or something? This is a pretty bizarre property...really wish it could just be specifying the solid region's geometry instead.

border_center_fill --> center_fill_mode

Approved.

image,resize needs to be image,resized and have size event info...

@segfaultxavi border_insets ?

Now that I think about it, the other events are wrong too. They should be at minimum preloaded and unloaded. I'm wondering whether preloaded makes sense though. Why isn't this just loaded?

Taking it even further, if we have load and unload, why not just have load_state,changed or similar and just pass a bool?

border_insets sounds about right, since that's what this property is.

All events should be in the past tense, sure. I would need to know what the preload event represents, though.
Why do we have preload and unload, instead of just load and unload? In that case, the load_state, changed would make a lot of sense.

preload is supposed to mean that the image data has been loaded and can be displayed. In a sense, this is an image domain specific term (i.e., "image preloading" is a thing), but for the purposes of this API, preload and unload are opposing states. I suppose maybe preload_state,changed would be the most accurate, or perhaps even preload,changed?

Thinking more about it, we use the ,changed suffix for events when there's an associated property. I think it would be weird to use it in this case.
When's the unload event emitted, anyway? do we have an unload method?

unload is called internally when the image's data is unloaded (e.g., if it's being reloaded) and can no longer be displayed.

We use changed when a state has changed. This seems like a state?

Yeah, but it's an internal state, not accessible through any property. Anyway, no point in discussing it any further since I have no strong opinion on this, either way works for me.
I prefer preload_state,changed over preload,changed, though.

My thinking is primarily that anyone who is interested in a preload event is also likely to be interested in an unload event, which means they need 2 event subscriptions. More event subscriptions = bad.

I already greenlighted thiiiiiiiiiiiiiis :D

I'M TRYING TO DOCUMENT IT FOR WHEN SOMEONE GETS ANGRY LATER WTF

This is done ?

OP updated