Limit users from flipping the model upside-down

[ijlal99]

Fixes #1020
Fixes #1737

[paireks]

Hello @ijlal99

it doesn't work on my end, but maybe I miss something, I don't know the reason yet. To reproduce:

1. Take any example file (I took this one: https://xeokit.github.io/xeokit-sdk/examples/buildings/#dotbim_BlenderHouse)
2. Use navcube to show the model from Top (Click Top on the navcube)
3. Rotate

Before the changes I could rotate it however I want (also upside down), right now model disappears after rotation, like this:

[ijlal99]

Hi @paireks!

Thank you for catching the bug. I have update the PR with the fix. Can you please have a look at it again?

[MichalDybizbanskiCreoox]

Not strictly technical, but I'd suggest including issue/ticket references in commits for context and traceability.

[ijlal99]

Hi @MichalDybizbanskiCreoox!

Once the PR is approved, I'll change the commit messages to contain the ticket reference.

[MichalDybizbanskiCreoox]

Hi @MichalDybizbanskiCreoox!

Once the PR is approved, I'll change the commit messages to contain the ticket reference.

Ok, but keep in mind some PRs get merged skipping their approval.
Also not sure if changing a PR doesn't invalidate an approval.

[paireks]

I checked it on a few examples, and right now it seems to work on my end as expected.

[ijlal99]

Hi @xeolabs!

I believe this PR is ready to be merged now.

[xeolabs]

Hi @xeolabs!

I believe this PR is ready to be merged now.

Great! Could you just address @paireks comment above then we should be good to go.

[ijlal99]

I have already taken care of that in the latest push.