fix: use translation instead of hardcoded loading string

[secondl1ght]

🌐 Replace hardcoded loading text with i18n translation in InputField

Summary

Replaced the hardcoded 'Loading...' default value for the loadingText prop in the InputField molecule with a proper next-intl translation using the existing common.loading key from en.json.

Changes

• src/components/molecules/InputField/InputField.tsx
  • Added 'use client' directive (required for the useTranslations hook, consistent with other molecules)
  • Imported useTranslations from next-intl
  • Removed hardcoded 'Loading...' default from the loadingText prop
  • Added useTranslations('common') hook and resolved loading text via loadingText ?? t('loading')
  • Callers can still override loadingText explicitly if needed

No Test Changes Required ✅

The global next-intl mock in src/config/test.ts already uses the actual en.json messages, so all 12 existing tests pass without modification. Snapshots updated cleanly since the translated value matches the previous hardcoded string.

---

Related

• Follow-up from feat(ui): add skeleton loaders for all remaining loading states #1584.

Screenshots

• N/A — no visual changes

Notes

• Add any additional notes here

---

This pull request summary was generated, for full implementation details please reference the file changes.

[greptile-apps]

Greptile Summary

This PR replaces a hardcoded 'Loading...' default with a next-intl translation lookup (common.loading) in InputField, and adds the required 'use client' directive so useTranslations can be used. The change is minimal, consistent with how other molecules handle translations, and all existing tests continue to pass.

Confidence Score: 5/5

Safe to merge — the change is a minimal, correct i18n substitution with no behavioral or structural risks.

No P0 or P1 findings. The 'use client' directive is necessary and correct for useTranslations, consistent with other molecules. The fallback logic using ?? is correct. All existing tests pass. No custom rules are violated.

No files require special attention.

Important Files Changed

Filename	Overview
src/components/molecules/InputField/InputField.tsx	Added 'use client' directive and useTranslations('common') hook to resolve loadingText via translation fallback; no logic changes beyond the hardcoded string removal.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[InputField render] --> B{loadingText prop provided?}
    B -- Yes --> C[Use provided loadingText]
    B -- No --> D[Lookup common.loading via t]
    C --> E[resolvedLoadingText]
    D --> E
    E --> F{loading === true?}
    F -- Yes --> G[Display resolvedLoadingText in input]
    F -- No --> H[Display value prop in input]

Loading

_{Reviews (1): Last reviewed commit: "fix: use translation instead of hardcode..." | Re-trigger Greptile}

[github-actions]

🚀 Preview Deployment

URL: https://franky-preview-pr-1683-fzxmjul7ya-oa.a.run.app

franky-preview-pr-1683 • 980442d

[Orlandohub]

LGTM