-
Notifications
You must be signed in to change notification settings - Fork 70
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
VAGOV-TEAM-100670: Removes sidebar placeholder and two-column layout #20267
Conversation
Checking composer.lock changes... |
Checking composer.lock changes... |
Cypress Accessibility Violations
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I appreciate you cleaning up the CSS. Way too easy to forget about this and leave it there indefinitely.
Description
This PR removes the sidebar placeholder put in recently, as it has been removed from updated designs.
Closes department-of-veterans-affairs/va.gov-team#100670
Testing done
Screenshots
Before
After
QA steps
As a user with role
Form Builder user
:/form-builder
.Definition of Done
Select Team for PR review
CMS Team
Public websites
Facilities
User support
Accelerated Publishing
Form Engine
Is this PR blocked by another PR?
DO NOT MERGE
Does this PR need review from a Product Owner
Needs PO review
CMS user-facing announcement
Is an announcement needed to let editors know of this change?