Skip to content

chore(frontend): create PasswordInput wrapper with stories DEV-2727 - #7481

Open
magicznyleszek wants to merge 6 commits into
mainfrom
leszek/dev-2727-password-input-wrapper
Open

chore(frontend): create PasswordInput wrapper with stories DEV-2727#7481
magicznyleszek wants to merge 6 commits into
mainfrom
leszek/dev-2727-password-input-wrapper

Conversation

@magicznyleszek

@magicznyleszek magicznyleszek commented Aug 25, 2026

Copy link
Copy Markdown
Member

🗒️ Checklist

  1. run linter locally
  2. update developer docs (API, README, inline, etc.), if any
  3. for user-facing doc changes create a Zulip thread at #Support Docs Updates, if any
  4. draft PR with a title <type>(<scope>)<!>: <title> DEV-1234
  5. assign yourself, tag PR: at least Front end and/or Back end or workflow
  6. fill in the template below and delete template comments
  7. review thyself: read the diff and repro the preview as written
  8. open PR & confirm that CI passes & request reviewers, if needed
  9. act on any greptile review below a 5/5 score or leave comment explaining why you won't
  10. delete this checklist section from the final squash commit before merging

💭 Notes

Adds the wrapper so we can have nice stories file for Chromatic.

Also updated how we set colors for disabled inputs and placeholder.

👀 Preview steps

  1. ℹ️ Run storybook
  2. Go to Design System → PasswordInput
  3. Verify it looks good

@magicznyleszek
magicznyleszek marked this pull request as ready for review August 25, 2026 12:14
@magicznyleszek
magicznyleszek removed the request for review from Akuukis August 25, 2026 12:14
@greptile-apps

greptile-apps Bot commented Aug 25, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

The PR adds a shared PasswordInput wrapper and Storybook examples, migrates RESTServicesForm to the wrapper, enforces its use through ESLint, and updates disabled and placeholder input colors.

  • Adds PasswordInput wrapper and stories.
  • Migrates RESTServicesForm away from the direct Mantine import.
  • Centralizes disabled and placeholder colors through Mantine input variables.

Confidence Score: 5/5

The PR appears safe to merge.

No blocking failure remains.

Reviews (2): Last reviewed commit: "typo" | Re-trigger Greptile

Comment thread jsapp/js/components/common/PasswordInput.stories.tsx
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant