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

The aggregates handling in components like the padding row, can cause the inspector controls to change underneath the user. #6249

Open
seanparsons opened this issue Aug 21, 2024 · 0 comments
Labels
Bug Something isn't working Inspector Affects the inspector, settings, commands

Comments

@seanparsons
Copy link
Contributor

seanparsons commented Aug 21, 2024

For example: https://github.com/user-attachments/assets/cf31a84a-3f28-4e05-9124-f1accb032038

While this is being triggered by the number input scrubbing, the actual bug lies with how the inspector controls check to see if all the "sides" are equal, if they are then equal then it will collapse them. This can also be triggered by bumping the values with the cursor keys.

Solution:

  • the problem: "if they are then equal then it will collapse them." - this should only happen when the element is first selected, ie the initial controls should display based on the algo
  • the manual setting for how to display the controls should be respected while the element is selected (so if you switch to eg all four paddings, having two or all four accidentally match doesn't change the controls at all)
@seanparsons seanparsons added Bug Something isn't working Inspector Affects the inspector, settings, commands labels Aug 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something isn't working Inspector Affects the inspector, settings, commands
Projects
None yet
Development

No branches or pull requests

1 participant