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

789 - Collapsible panels #790

Open
wants to merge 13 commits into
base: develop
Choose a base branch
from

Conversation

matt852
Copy link
Contributor

@matt852 matt852 commented Aug 9, 2024

This adds support for the Configuration Compliance panels on a Device page to be collapsible.

You can click anywhere on the panel heading and it will toggle between collapsed and expanded. Settings are saved via local cookie. This means that if you leave the page or refresh it, whatever you collapsed will remain collapsed.

These preferences are saved by platform and feature. This means that if you collapse the AAA box for IOS, then all IOS AAA boxes will remain collapsed. This can be changed if another method is preferred.

I've also updated the formatting slightly on the top green/red boxes to match the colors on the rest of the page. The green and red now matches the greens on the other buttons/labels on the page. I also made the boxes have slightly softer edges.

Screenshot 2024-08-09 at 3 23 08 PM

@glennmatthews
Copy link
Contributor

pytz is deprecated; under django 4 we should be using zoneinfo instead. https://docs.djangoproject.com/en/4.2/releases/4.0/#zoneinfo-default-timezone-implementation

@matt852
Copy link
Contributor Author

matt852 commented Aug 14, 2024

I'm seeing a bunch of test fixes in PR #788. I'm going to rebase my changes onto that branch, or wait until it is merged into develop before continuing to troubleshoot these test issues.

@itdependsnetworks
Copy link
Contributor

@matt852 can you confirm you have tested this extensively? Hard to see some of the changes in the template based on how it shows up in Git.

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.

3 participants