Skip to content

Conversation

@lancepioch
Copy link
Member

  1. Type in username/email
  2. Hit tab to go to password input
  3. Notice it works now

@lancepioch lancepioch self-assigned this Jan 30, 2026
@coderabbitai
Copy link

coderabbitai bot commented Jan 30, 2026

📝 Walkthrough

Walkthrough

Adds a protected getPasswordFormComponent() method to Login that wraps the parent implementation and sets the password field's metadata tabindex to 2; no other form behavior or attributes were changed.

Changes

Cohort / File(s) Summary
Login page — password component
app/Filament/Pages/Auth/Login.php
Added protected function getPasswordFormComponent(): Component which calls the parent component and adjusts its metadata/attributes to include tabindex: 2. No other changes to form inputs or imports were made.
🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main change—fixing tab navigation from username to password on the login form, which matches the code modification.
Description check ✅ Passed The description is related to the changeset, providing reproduction steps that demonstrate the tab behavior fix being implemented.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@lancepioch lancepioch merged commit 57c2aa6 into main Feb 2, 2026
32 checks passed
@lancepioch lancepioch deleted the lance/login-tab-fix branch February 2, 2026 13:40
@github-actions github-actions bot locked and limited conversation to collaborators Feb 2, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants