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

CP Entries filter, Comment column can be non-removable #3135

Closed
robinsowell opened this issue Mar 17, 2023 · 0 comments · Fixed by #3140
Closed

CP Entries filter, Comment column can be non-removable #3135

robinsowell opened this issue Mar 17, 2023 · 0 comments · Fixed by #3140
Labels
Bug: Accepted Bug has been confirmed, is reproducible, and ready to work on.

Comments

@robinsowell
Copy link
Contributor

So by default, the Comments column shows on the entry filter. Make sure it's showing- go to Settings- Comment Settings and disallow comments. Go back to your entries edit page- the comment column still displays. Click the controller so you can remove it and... Comments is no longer avaible as a checkbox option.

Once you click something else- like Status- and load the page - Comments will disappear.

I think the column is getting stored as one to show and when you disable them, it keeps showing. Which I don't think is a problem. But when you disable them, they no longer show in the checkboxes so now you're stuck with the column showing until you resave the filters, at which point it looks like it gets washed out.

IDK how hard it would be to either clear it from the saved columns when you change the allow comments setting or- and I kinda prefer this- IF the comment column is showing, then the checkbox should still be included.

In any case, it confused a user because they couldn't get rid of it. EE 7.2.11

@robinsowell robinsowell added the Bug: Accepted Bug has been confirmed, is reproducible, and ready to work on. label Mar 17, 2023
intoeetive added a commit that referenced this issue Mar 20, 2023
intoeetive added a commit that referenced this issue Mar 20, 2023
intoeetive added a commit that referenced this issue Apr 3, 2023
…available-columns

Resolved #3135, #3047 where the columns in Entry Manager did not dynamically update if some column is not available
intoeetive added a commit that referenced this issue Apr 3, 2023
…available-columns

Resolved #3135, #3047 where the columns in Entry Manager did not dynamically update if some column is not available
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug: Accepted Bug has been confirmed, is reproducible, and ready to work on.
Projects
None yet
1 participant