Skip to content

Fix personal key block display#7062

Merged
aduth merged 4 commits intomainfrom
aduth-personal-key-block
Sep 30, 2022
Merged

Fix personal key block display#7062
aduth merged 4 commits intomainfrom
aduth-personal-key-block

Conversation

@aduth
Copy link
Contributor

@aduth aduth commented Sep 29, 2022

🛠 Summary of changes

This fixes an issue where the personal key page layout was malformed, as an unintended regression of #6986 due to accidental name collision of the .separator-text class added there.

📜 Testing Plan

  • Confirm that the identity verification "Personal key" page appears as expected

👀 Screenshots

Desktop:

Before After
localhost_3000_manage_personal_key (2) localhost_3000_manage_personal_key (1)

Mobile:

Before After
localhost_3000_manage_personal_key(iPhone XR) localhost_3000_manage_personal_key(iPhone XR) (1)

changelog: Bug Fixes, Identity Verification, Fix display of personal key page layout
@aduth aduth requested a review from jmdembe September 29, 2022 20:30
Comment on lines -20 to -27
@include at-media('tablet') {
.separator-text > div {
&::after {
color: #000;
padding: 0 0.5rem;
}
}
}
Copy link
Contributor Author

Choose a reason for hiding this comment

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

As can be seen in the actual implementation, there are no <div> elements inside this element, so these styles were not doing anything and can be removed.

Copy link
Contributor

@jmdembe jmdembe left a comment

Choose a reason for hiding this comment

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

LGTM

@aduth aduth merged commit b1361b4 into main Sep 30, 2022
@aduth aduth deleted the aduth-personal-key-block branch September 30, 2022 19:58
@aduth aduth mentioned this pull request Oct 3, 2022
jskinne3 pushed a commit that referenced this pull request Oct 12, 2022
* Fix personal key block display

changelog: Bug Fixes, Identity Verification, Fix display of personal key page layout

* Fix padding-y at all viewport sizes

* Remove unused key-badge styles

* Update spec selectors
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