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

Make it an option whether navigation of ItemList must be the same row #6798

Open
feois opened this issue May 3, 2023 · 2 comments
Open

Make it an option whether navigation of ItemList must be the same row #6798

feois opened this issue May 3, 2023 · 2 comments

Comments

@feois
Copy link

feois commented May 3, 2023

Describe the project you are working on

Not important

Describe the problem or limitation you are having in your project

When the selected item is at the start or the end of the row, event ui_left or ui_right respectively does nothing

Describe the feature / enhancement and how it helps to overcome the problem or limitation

See below

Describe how your proposal will work, with code, pseudo-code, mock-ups, and/or diagrams

When invoking event ui_right while the selected item is at the end of the row of ItemList, it should switch selection to the first item of next row, vice versa, when invoking event ui_left while the selected item is at the front of the row, selection should switch to the last item of previous row.

If this enhancement will not be used often, can it be worked around with a few lines of script?

Well yes, this can be implemented in scripts, it's inconvenient (plus you need to check if ItemList is the focus)

Is there a reason why this should be core and not an add-on in the asset library?

ItemList is core node

@feois
Copy link
Author

feois commented May 3, 2023

While reading ItemList source code, I found out that it's not a limitation but instead an intentional choice (!IS_SAME_ROW).
If so, I propose that it's made a choice.

@feois feois changed the title Enhance navigation of ItemList Make it an option whether navigation of ItemList must be the same row May 3, 2023
@Calinou
Copy link
Member

Calinou commented May 3, 2023

This can likely be done by adding an Allow Wraparound boolean property, similar to what I proposed in #5139.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants