Skip to content

Conversation

yaira2
Copy link
Member

@yaira2 yaira2 commented Dec 17, 2020

Please note, hiding a column won't work unless you toggle the item and then hide it again afterwards, this is a known issue with the setup of the SettingsViewModel and it shouldn't hold up merging this pull request.
Fixes #380

Copy link
Member

@gave92 gave92 left a comment

Choose a reason for hiding this comment

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

Works good with the exception of that known bug 👍

@yaira2 yaira2 added the ready to merge Pull requests that are approved and ready to merge label Dec 17, 2020
@yaira2 yaira2 merged commit 654fe56 into master Dec 17, 2020
@yaira2 yaira2 deleted the EditColumns branch December 17, 2020 17:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready to merge Pull requests that are approved and ready to merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add the option to hide columns in the details layout mode

3 participants