Add NoHeader() to TableWidget #830
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
when the noHeader flag is set the columns are created as normal by the Build() function. however the imgui.TableHeadersRow() function is not called this means that table columns can be set up in the normal way (widths, resizeable, etc.) but without the column names being shown and without the table context menu if the column headers nned to be shown then the application can draw them manually as required
gucio321
requested changes
Jul 27, 2024
TableWidgets.go
Outdated
| return t | ||
| } | ||
|
|
||
| // NoHeader indicates that the column header should not be shown. |
There was a problem hiding this comment.
Add here that "it allows to setup column flags in Columns method" and say something about this method in a comment over Columns method - just for documentation
There was a problem hiding this comment.
Okay. New change committed
gucio321
approved these changes
Jul 27, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Discussion here:
#829
The SimpleHeader() idea in the first post is too inflexible. This pull request is for the NoHeader() idea from the second post.