Skip to content

Add margin-top to password reset content - #7427

Open
idumlupinar wants to merge 3 commits into
dnnsoftware:developfrom
idumlupinar:bugfix-7065
Open

Add margin-top to password reset content#7427
idumlupinar wants to merge 3 commits into
dnnsoftware:developfrom
idumlupinar:bugfix-7065

Conversation

@idumlupinar

@idumlupinar idumlupinar commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

Fixes #7065.

When a user receives the password reset link and tries to set their password, the tooltip containing the complexity requirements is partially hidden. It is due to the margin of .dnnPasswordResetContent CSS class.

Summary

Initially, I added the value to \admin\Security\module.css with 60px specification.
.dnnPasswordResetContent { width:550px; margin-top: 60px; }

Per @valadas' suggestion I decided to try adding the value in Aperture theme's login component on \DNN Platform\Skins\Aperture\src\scss\components_login.scss
with a slight difference. Instead of 60px I used 4rem. Translates to 64px. Looked good on my desktop and mobile devices.

.dnnPasswordResetContent {
    margin-top: 4rem;
}

When a user receives the password reset link and tries to set their password, the tooltip containing the complexity requirements is partially hidden. It is due to the margin of .dnnPasswordResetContent

@valadas valadas left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This may be a problem using other themes. I would rather look into the default theme for a fix, maybe a z-offset issue or something about how the header is made. But only doing it here might cause it to look strange when used in other themes.

@idumlupinar
idumlupinar marked this pull request as draft August 5, 2026 05:48
Removed margin-top from .dnnPasswordResetContent as it might affect custom Themes. Planning to adjust Aperture theme instead.
Added custom margin for the password reset form to address bug dnnsoftware#7065.
@idumlupinar
idumlupinar marked this pull request as ready for review August 6, 2026 20:55
@idumlupinar
idumlupinar requested a review from valadas August 6, 2026 20:55
@idumlupinar

idumlupinar commented Aug 6, 2026

Copy link
Copy Markdown
Contributor Author

\DNN Platform\Skins\Aperture\src\scss\components\_login.scss might be the correct location?

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.

[Bug]: Password strength tooltip partially visible - very simple to fix

2 participants