Skip to content

Conversation

chris1384
Copy link
Contributor

fixes:

  • fixed object not appearing when selecting certain model ids
  • object rotation in viewport is centered
  • removed zoom out limit
  • added doublesided property to previewed element

fixes:
- fixed object not appearing when selecting certain model ids
- object rotation in viewport is centered
- removed zoom out limit
- added doublesided property to previewed element
@Fernando-A-Rocha
Copy link
Contributor

Nice but some objects aren't supposed to be double sided?

@chris1384
Copy link
Contributor Author

chris1384 commented Oct 4, 2025

you're right, I thought about the outcomes of it, but I considered that there are more objects that need easier viewing (mostly landbits and vegetation) and there are only a few models that might look out of place when doublesided is applied to them

we'll have to see if it's required to either keep the property or remove it

@chris1384 chris1384 changed the title [editor] Improved object preview [editor_gui] Improved object preview Oct 4, 2025
@ArranTuna
Copy link
Contributor

What about a checkbox to view with / without double sided?

@chris1384
Copy link
Contributor Author

chris1384 commented Oct 9, 2025

Should be a lot better now @Fernando-A-Rocha @ArranTuna

image

@Fernando-A-Rocha
Copy link
Contributor

That's cool. Any other object properties you could put as check boxes next to that double-sided one?

@chris1384
Copy link
Contributor Author

chris1384 commented Oct 9, 2025

@Fernando-A-Rocha besides alpha, I don't think there's anything else to add. Perhaps we may include extra features (possibly regarding camera) in a separate PR until this one is reviewed

@PrimelPrime
Copy link
Contributor

Would be nice to be able to change the sky gradient inside the object browser that way weather and time could still stay the same but you dont have that bright glare while browsing right in your face

@ArranTuna
Copy link
Contributor

Everything looks right. Will merge unless anyone has any complaints.

@Fernando-A-Rocha
Copy link
Contributor

Great, more features can come in subsequent PRs

@ArranTuna ArranTuna merged commit 362f23c into multitheftauto:master Oct 11, 2025
1 check passed
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.

4 participants