Made UIRect initialisation functions const

[Kees-van-Beilen]

Objective

Destructuring in const code blocks isn't allowed, thus using UIRect in const code can be a hassle as it initialisation function aren't const. This Pr makes them const.

Solution

Removed all destructuring in the UIRect implementation

Testing

• I've ran a few ui examples to check if i didn't make a mistake,

---

[ickshonpe]

Looks good, all the fields match up correctly and the examples still work.
Strongly prefer struct update syntax for the constructor functions though.

[ickshonpe]

I'd prefer that we used struct update syntax for the constructor functions, like:

Suggested change

	pub const fn left(value: Val) -> Self {
	let mut rect = UiRect::DEFAULT;
	rect.left = value;
	rect
	}
	pub const fn left(left: Val) -> Self {
	Self {
	left,
	..Self::DEFAULT
	}
	}

is simpler and clearer.

[Kees-van-Beilen]

That's the entire reason I've made this PR, ..Self::DEFAULT cannot be used in const functions. I also would love to use that notation, but it is going to be a while before rust will support this Sorry I assumed that that wasn't possible as the exact same scenario didn't work on my computer, but i just checked your code change suggestion and that does work. Will refactor and update 👍 thank you

[ickshonpe]

I thought you can, can't you? Self::DEFAULT is a const value, it's not Self::default()

[Kees-van-Beilen]

Apparently when a struct implements Drop or one of its field implements Drop then you cannot use the struct updater syntax. So for instance you can't write:

const MY_NODE:Node = Node {..Node::DEFAULT};

because Node has a field of type Vec which implements Drop. However this shouldn't be a problem in UIRect's case.

[ickshonpe]

Everything looks good now.