Skip to content

In the object group from the panel: Move the select list of object on top to get a static position for picking. - #9057

Open
Bouh wants to merge 2 commits into
masterfrom
select-list-object-group-on-top
Open

In the object group from the panel: Move the select list of object on top to get a static position for picking.#9057
Bouh wants to merge 2 commits into
masterfrom
select-list-object-group-on-top

Conversation

@Bouh

@Bouh Bouh commented Sep 3, 2026

Copy link
Copy Markdown
Collaborator

Why:
Picking an object make the select list moving with the new object added to the list, the layout shifting is anoying.

This PR:
image

On the modal, the select list also moved on top.
image

Before:
image

@Bouh
Bouh requested a review from 4ian as a code owner September 3, 2026 23:05
@ClementPasteau

Copy link
Copy Markdown
Collaborator

What if you scroll down, is the field still visible?
I know it wasn't the case before, but wondering if we should make the field sticky

@4ian

4ian commented Sep 4, 2026

Copy link
Copy Markdown
Owner

Using a select for adding an object is a weird pattern (though kind of efficient in its own way :)), but I would prefer we avoid to stack "extra weirdness" on top of it by making it sticky?

@Bouh

Bouh commented Sep 4, 2026

Copy link
Copy Markdown
Collaborator Author

The weirdness is that you have to scroll all the way to the bottom of the added items to click the select and add a new object. After each addition, the select moves, so when the list gets long, you have to scroll down again to find and click it.

With the select list at the top, you stay at the top. The select doesn’t move, items are added to the bottom, and there’s no need to keep scrolling back and forth.

Try it out. I think you’ll be persuaded.

If we want to rethink this part of the UI more fundamentally, that can be a bigger challenge for another day. For now, I’d suggest trying this simpler, more usable approach.

@4ian

4ian commented Sep 4, 2026

Copy link
Copy Markdown
Owner

Agree that in this case we should just put it on top and that's it, it works better :)

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.

3 participants