Remove the rest of Bindable<> properties from Room

[smoogipoo]

Prereqs:

• Make RoomID non-bindable #30598

This covers most of the other less-important/"visual" properties from the room, building on the structure set by #30598. The first 1-2 commits are a bit chonky but the rest should be easy to parse commit-by-commit (NRT-related changes stopped happening).

BindableList<> properties are not included here because they're a bit more complicated.

In general, most relevant implementations take the form:

protected override void LoadComplete()
{
    base.LoadComplete();

    room.PropertyChanged += onRoomPropertyChanged;
    updateRoomName();
}

private void onRoomPropertyChanged(object? sender, PropertyChangedEventArgs e)
{
    switch (e.PropertyName)
    {
        case nameof(Room.Name):
            updateRoomName();
            break;
    }
}

private void updateRoomName()
{
}

protected override void Dispose(bool isDisposing)
{
    base.Dispose(isDisposing);
    room.PropertyChanged -= onRoomPropertyChanged;
}

I've done by best to ensure every += is matched with a -=, and to do this I've kept everything structurally identical so it can be searched by text. You're going to have to excuse the method duplication for now, I don't know whether it's best to separate it like that or to go with something like:

protected override void LoadComplete()
{
    base.LoadComplete();

    room.PropertyChanged += onRoomPropertyChanged;
    onRoomPropertyChanged(null, nameof(Room.Name);
}

private void onRoomPropertyChanged(object? sender, PropertyChangedEventArgs e)
{
    switch (e.PropertyName)
    {
        case nameof(Room.Name):
            /* update */
            break;
    }
}

Will figure that part out as a follow up.

[smoogipoo]

Test failures are not... relevant 😅