Skip to content

7683 Revert IPP CTA change (while preserving troubleshooting-options refactor)#7134

Merged
allthesignals merged 1 commit intomainfrom
wmg/7683-ipp-move-up-reversion
Oct 12, 2022
Merged

7683 Revert IPP CTA change (while preserving troubleshooting-options refactor)#7134
allthesignals merged 1 commit intomainfrom
wmg/7683-ipp-move-up-reversion

Conversation

@allthesignals
Copy link
Contributor

@allthesignals allthesignals commented Oct 12, 2022

🎫 Ticket

Link to the relevant ticket.

#7129
https://cm-jira.usa.gov/browse/LG-7683

🛠 Summary of changes

This reverts commit feb84af. I was supposed to leave this as an open PR until designers were ready to merge this.

This change reverts some of feb84af. We will keep changes to troubleshooting-options, but hold off on moving up the IPP CTA.

👀 Screenshots

Before: Screen Shot 2022-10-12 at 10 25 53 AM
After: Screen Shot 2022-10-12 at 3 31 20 PM

@allthesignals allthesignals requested a review from aduth October 12, 2022 19:11
…ut keeps refactoring of troubleshooting-options
@allthesignals allthesignals force-pushed the wmg/7683-ipp-move-up-reversion branch from 3532e5f to 2f93fe7 Compare October 12, 2022 19:31
@allthesignals allthesignals changed the title Revert "7683 Make divider explicit and configurable (#7129)" 7683 Revert IPP CTA change (while preserving troubleshooting-options refactor) Oct 12, 2022
heading={formatHTML(t('idv.troubleshooting.headings.are_you_near'), {
wbr: 'wbr',
})}
divider={false}
Copy link
Contributor Author

@allthesignals allthesignals Oct 12, 2022

Choose a reason for hiding this comment

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

Note that this uses the new divider={false} prop to preserve intended behavior

@allthesignals allthesignals merged commit 92d9347 into main Oct 12, 2022
@allthesignals allthesignals deleted the wmg/7683-ipp-move-up-reversion branch October 12, 2022 20:00
jskinne3 pushed a commit that referenced this pull request Oct 12, 2022
…ut keeps refactoring of troubleshooting-options (#7134)
@jmdembe jmdembe mentioned this pull request Oct 13, 2022
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.

2 participants