Fix password field height on login page in Safari#1036
Open
Conversation
Co-authored-by: maebeale <7607813+maebeale@users.noreply.github.com>
Co-authored-by: maebeale <7607813+maebeale@users.noreply.github.com>
Copilot
AI
changed the title
[WIP] Fix password field height issue on login page in Safari
Fix password field height rendering in Safari
Feb 15, 2026
Collaborator
@copilot try again. see photo.
|
Co-authored-by: maebeale <7607813+maebeale@users.noreply.github.com>
Contributor
Author
Copilot
AI
changed the title
Fix password field height rendering in Safari
Fix password field width on login page in Safari
Feb 15, 2026
Collaborator
|
@copilot it's still broken. looks the same as earlier pic. |
…ield Co-authored-by: maebeale <7607813+maebeale@users.noreply.github.com>
Contributor
Author
Fixed in commit 8ddd242. The issue was the password field was using |
Copilot
AI
changed the title
Fix password field width on login page in Safari
Fix password field height on login page in Safari
Feb 15, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.

What is the goal of this PR and why is this important?
The password field on the login page was not displaying at the same height as the email field above it in Safari, appearing visibly shorter. This inconsistency affected the visual appearance and user experience of the login form.
How did you approach the change?
Updated the password field's padding in
app/views/devise/sessions/new.html.erbto match the email field's default SimpleForm wrapper padding:class: "w-full form-control password-field rounded-md border-gray-300 shadow-sm focus:ring-blue-500 focus:border-blue-500 px-3 py-2 pr-10"The root cause was that when custom
input_htmlclasses are provided to SimpleForm, they completely override the default wrapper classes. The email field receivespx-3 py-2padding from the default wrapper configuration, but the password field was explicitly usingp-2padding, resulting in less vertical padding (p-2= 0.5rem all around vspy-2= 0.5rem vertical withpx-3= 0.75rem horizontal).Anything else to add?
Initial approaches attempted:
height: 2.5remrule - didn't solve the issuew-fullclass for width - the fields were already the same widthThe correct fix was matching the vertical padding by using
px-3 py-2instead ofp-2, ensuring both fields have identical height rendering across all browsers including Safari.💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.