Skip to content

[Fix] push_id() for ui sections in table #26

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

MickHarrigan
Copy link

Fixes what I would consider a bug with adding anything requiring a salt to a region of the table that is not the bottom right.
This is due to the nature of reusing the header_ui() and region_ui() functions.

@lucasmerlin
Copy link
Contributor

I don't see why this would be necessary. Can you make a small repro example that shows whats wrong without these changes?

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.

2 participants