Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

setElementParent on GUI elements is slightly broken #314

Open
patrikjuvonen opened this issue Aug 7, 2018 · 1 comment
Open

setElementParent on GUI elements is slightly broken #314

patrikjuvonen opened this issue Aug 7, 2018 · 1 comment
Labels
bug Something isn't working help wanted Extra attention is needed
Milestone

Comments

@patrikjuvonen
Copy link
Contributor

patrikjuvonen commented Aug 7, 2018

Describe the bug

  • It does not seem to be possible to set the parent of a GUI element to guiRoot. Also tested on root, resourceRoot and getResourceDynamicElementRoot(resource).
  • It does not seem to reset the position relative to the new parent when set (from window to another, for example). Repositioning does not reload the position.

To reproduce

  1. Create a GUI element and optionally another GUI element as the parent.
  2. setElementParent(guiElement, guiRoot) -> doesn't do anything
  3. setElementParent(guiElement, guiSecondWindow) -> sets, but doesn't reposition relative to the new parent

Expected behavior
Set parent as requested and reset position relative to the new parent.

MTA Client:
Multi Theft Auto v1.5.5-release-13968

Additional context

@patrikjuvonen patrikjuvonen added bug Something isn't working help wanted Extra attention is needed labels Aug 7, 2018
@botder botder added this to the Backlog milestone Mar 3, 2019
@Lpsd
Copy link
Member

Lpsd commented May 4, 2021

Will be resolved by #1459 along with other non backwards-compatible changes.

Due to this apparently not repositioning elements it could break some existing GUIs when fixed.

@Lpsd Lpsd linked a pull request May 4, 2021 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working help wanted Extra attention is needed
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants