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

treeview: missing columns #10580

Merged
merged 2 commits into from
Nov 23, 2024
Merged

Conversation

eszkadev
Copy link
Contributor

jsdialog: treeview: put dummy columns at correct position

It was a regression from commit aee68f732003d02bdf7c9b0768df8ca0a3880123
browser: treeview: align listbox with headers using subgrid

If we used autofilter in calc which has icons at the end of an entry,
we got dummy column at the beginning of the row...

This adds preprocessing of columns so we always have all of them
unified (the same number of cols in every row) with dummy ones at
the correct position. Exception is separator which has to be only
one in the row to be full width.

It was a regression from commit aee68f7
browser: treeview: align listbox with headers using subgrid

If we used autofilter in calc which has icons at the end of an entry,
we got dummy column at the beginning of the row...

This adds preprocessing of columns so we always have all of them
unified (the same number of cols in every row) with dummy ones at
the correct position. Exception is separator which has to be only
one in the row to be full width.

Signed-off-by: Szymon Kłos <szymon.klos@collabora.com>
Change-Id: I8f3ed68100ca7f6b8c299e18e3867a4654d3a8c4
@eszkadev eszkadev force-pushed the private/eszkadev/treeview-missing-columns branch from f6cf1ed to ca430f3 Compare November 22, 2024 12:20
@eszkadev
Copy link
Contributor Author

passed but change id check didn't work

@eszkadev eszkadev closed this Nov 22, 2024
@eszkadev eszkadev reopened this Nov 22, 2024
Because we preprocess data to have all the entries with equal number
of the columns already

Signed-off-by: Szymon Kłos <szymon.klos@collabora.com>
Change-Id: Icda50b2c378c1e39b3847bd18e58e35c2c0de015
@eszkadev eszkadev force-pushed the private/eszkadev/treeview-missing-columns branch from ca430f3 to 4a51dc3 Compare November 22, 2024 13:07
Copy link
Contributor

@gokaysatir gokaysatir left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the fix.

@eszkadev eszkadev merged commit 210607f into master Nov 23, 2024
13 checks passed
@eszkadev eszkadev deleted the private/eszkadev/treeview-missing-columns branch November 23, 2024 04:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants