Change cursor shape for disabled widgets

[amedoeyes]

Objective

• Fixes Feathers: Cursors for disabled widgets #20499

Solution

• Added EntityCursor re-insertion in set_*_colors functions with corresponding cursor shape depending on whether the widget is disabled or not.

Testing

• Tested the changes in the feathers example.

[amedoeyes]

I think function names should change because they don't just change colors anymore.

[github-actions]

It looks like your PR has been selected for a highlight in the next release blog post, but you didn't provide a release note.

Please review the instructions for writing release notes, then expand or revise the content in the release notes directory to showcase your changes.

[alice-i-cecile]

I think function names should change because they don't just change colors anymore.

I agree, can you update this? Can you also add this PR to the feathers release notes?

[ickshonpe]

It's not a problem to match on a bool I guess, but it's not idiomatic and looks a bit odd:

Suggested change

	let cursor_shape = match disabled {
	true => bevy_window::SystemCursorIcon::NotAllowed,
	false => bevy_window::SystemCursorIcon::Pointer,
	};
	let cursor_shape = if disabled {
	bevy_window::SystemCursorIcon::NotAllowed
	} else {
	bevy_window::SystemCursorIcon::Pointer
	};

[amedoeyes]

I agree, but I saw that it was already done like that in other parts of the crate so for consistency I just followed it.

[ickshonpe]

Changes look good to me.

[amedoeyes]

I think function names should change because they don't just change colors anymore.

I agree, can you update this? Can you also add this PR to the feathers release notes?

On it.

[amedoeyes]

Is this good?

[alice-i-cecile]

Thanks! Nice work :)