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.

Sign up for GitHub

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

Jump to bottom

Inspector scrolls away from focused item automatically #96489

Closed
rick551a opened this issue Sep 2, 2024 · 2 comments · Fixed by #96542
Closed

Inspector scrolls away from focused item automatically #96489

rick551a opened this issue Sep 2, 2024 · 2 comments · Fixed by #96542
Labels
bug regression topic:editor
Milestone
4.4

Comments

@rick551a
Copy link

rick551a commented Sep 2, 2024 •
edited
Loading

Tested versions

4.3 stable mono

System information

Win10, Nvidia

Issue description

When clicking a checkbox the inspector scrolls up, away from the clicked item

godot4_3_inspector_jump_focus.mp4

See video clip for example,
Whenever I click 'Fog' inside the WorldEnvironment, the inspector scrolls back up to the top each time i click it.
Very annoying, as I have to scroll back down to 'Fog' again to press it again

This did not happen in 3.5, the focus remained on item (just tested it)

Steps to reproduce

Create a WorldEnvironment -> New Environment
Expand a few options ,so the inspector height is long.
Click an option near the end (like 'Fog') and watch the inspector scroll back up when you click it

Minimal reproduction project (MRP)

None
@aXu-AP
Copy link
Contributor

aXu-AP commented Sep 2, 2024 •
edited
Loading

To be more accurate, this happens when:

  • Toggled property opens up new options, making the inspector longer/shorter
  • The inspector is currently scrolled further down past a certain point

My guess is that while the inspector is reloading, it is temporarily shorter and the scrollbar adjusts to the new size limiting the max value.

@matheusmdx
Copy link
Contributor

Bisecting points to #78960 as the culprit, @kleonc

image


We already had a similar issue caused by this pr but with OptionButton instead, #91287

@akien-mga akien-mga added this to the 4.4 milestone Sep 3, 2024
@Maran23 Maran23 mentioned this issue Sep 3, 2024
Merged
@AThousandShips AThousandShips mentioned this issue Oct 8, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug regression topic:editor
Projects
None yet
Milestone
4.4
5 participants
@aXu-AP @akien-mga @matheusmdx @rick551a @AThousandShips