Skip to content

Commit 2f88fd9

Browse files
authored
[A11y]Fixing a missed horizontal scroll bar case (#8353)
* Adding some style workarounds for narrow screen * Update similar snippet for 2FA dialog * Remove display flex from login account row
1 parent 1d671e0 commit 2f88fd9

3 files changed

Lines changed: 6 additions & 11 deletions

File tree

src/Bootstrap/dist/css/bootstrap-theme.css

Lines changed: 0 additions & 4 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

src/Bootstrap/less/theme/page-account-settings.less

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,6 @@
4040

4141
.login-account-row {
4242
padding-top: 10px;
43-
display: flex;
4443
align-items: center;
4544
justify-content: space-between;
4645
}

src/NuGetGallery/Views/Users/_UserAccountChangeExternalCredential.cshtml

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -30,10 +30,10 @@
3030
@Html.AntiForgeryToken()
3131

3232
<div class="login-account-row">
33-
<span>
33+
<span class="col-sm-9" style="padding: 0;">
3434
Microsoft account: <b>@cred.Identity</b>
3535
</span>
36-
<span class="col-sm-3">
36+
<span class="col-sm-3" style="padding: 0;">
3737
<button class="btn btn-default btn-block" type="submit"
3838
data-confirm="This action will replace all of your linked accounts and password login with the new login account. Are you sure you want to change your login account?">
3939
Change Account
@@ -69,10 +69,10 @@
6969
<div class="login-account-row">
7070
@if (Model.User.EnableMultiFactorAuthentication)
7171
{
72-
<span>
72+
<span class="col-sm-9" style="padding: 0;">
7373
Two-factor authentication is currently enabled <i class="ms-Icon ms-Icon--CheckMark checkmark-icon" aria-hidden="true"></i>
7474
</span>
75-
<span class="col-sm-3">
75+
<span class="col-sm-3" style="padding: 0;">
7676
<button class="btn btn-danger btn-block" type="submit"
7777
data-confirm="This action will disable 2FA, making your account less secure. Are you sure you want to disable this setting?">
7878
Disable
@@ -81,10 +81,10 @@
8181
}
8282
else
8383
{
84-
<span>
84+
<span class="col-sm-9" style="padding: 0;">
8585
Two-factor authentication is currently disabled <i class="ms-Icon ms-Icon--Warning warning-icon" aria-hidden="true"></i>
8686
</span>
87-
<span class="col-sm-3">
87+
<span class="col-sm-3" style="padding: 0;">
8888
<button class="btn btn-default btn-block" type="submit">Enable</button>
8989
</span>
9090
}

0 commit comments

Comments
 (0)