Skip to content

Allow turntable to use Y axis.#1504

Draft
WickedSmoke wants to merge 1 commit intosolvespace:masterfrom
WickedSmoke:turntable_y
Draft

Allow turntable to use Y axis.#1504
WickedSmoke wants to merge 1 commit intosolvespace:masterfrom
WickedSmoke:turntable_y

Conversation

@WickedSmoke
Copy link
Contributor

As normal math conventions and OpenGL have the Y axis pointing up I would like an option to have turntable rotation use this axis.

This code currently just changes Z up to Y up. What would be the best way to add an option to the UI? Another checkbox would be easiest to implement, but a combo of (Free, Turntable Y, Turntable Z) would be less controls for the user to futz with. Is there combo box support with the current UI?

@WickedSmoke
Copy link
Contributor Author

I see many years ago @vespakoen tried to implement this in commit 97630bd.

@phkahler
Copy link
Member

Checkboxes and radio buttons work exactly the same way. Here is an example:

&TextWindow::ScreenChangeGroupOption,

We just added the checkbox for "skewed" extrusions, but it's functionally identical to the radio buttons - just looks different.

@WickedSmoke
Copy link
Contributor Author

Yeah, I saw the addition of "skewed" but I couldn't understand how to use that feature.

A proper combo widget would be more future proof as any number of rotation options could be added without taking up more screen space.

Turntable was added in commit e67f967.
@phkahler
Copy link
Member

@WickedSmoke there should be an option for which axis is up if we're going to do this. I'm guessing there are people in favor of all 3 being "up". So 4 options: Off, X, Y, Z ?

@WickedSmoke
Copy link
Contributor Author

So 4 options: Off, X, Y, Z ?

Yeah, that would work.

@ruevs ruevs marked this pull request as draft January 1, 2026 12:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants