Skip to content

EDM-3097: Disabled import option has a tooltip#476

Merged
celdrake merged 1 commit intoflightctl:mainfrom
celdrake:EDM-3097-disable-import-option-tooltip
Feb 5, 2026
Merged

EDM-3097: Disabled import option has a tooltip#476
celdrake merged 1 commit intoflightctl:mainfrom
celdrake:EDM-3097-disable-import-option-tooltip

Conversation

@celdrake
Copy link
Collaborator

@celdrake celdrake commented Feb 4, 2026

Added an explanation tooltip when the "Use an existing Git repository" option cannot be selected.

Summary by CodeRabbit

  • New Features

    • Added tooltip that displays when no Git repositories are available for selection
  • Style

    • Updated checkbox component styling for improved visual alignment and consistency

@coderabbitai
Copy link

coderabbitai bot commented Feb 4, 2026

Walkthrough

Added a new translation entry for "No Git repositories found" and integrated it into the ImportFleetWizard RepositoryStep via a tooltip. Additionally, applied CSS styling updates to the CreateRepositoryForm checkbox component.

Changes

Cohort / File(s) Summary
Internationalization
libs/i18n/locales/en/translation.json
Added translation entry "No Git repositories found" for UI text display.
ImportFleetWizard
libs/ui-components/src/components/Fleet/ImportFleetWizard/steps/RepositoryStep.tsx
Imported WithTooltip and wrapped "Use an existing Git repository" radio button with tooltip displaying "No Git repositories found" when no repositories exist.
CreateRepository Styling
libs/ui-components/src/components/Repository/CreateRepository/CreateRepositoryForm.css, libs/ui-components/src/components/Repository/CreateRepository/CreateRepositoryForm.tsx
Added new .fctl-create-repo__rs-checkbox CSS class with custom property for vertical alignment; applied className to Checkbox component for "Use resource syncs" option.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~15 minutes

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main change: adding a tooltip to a disabled import option, which is reflected in all modified files.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

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.

{showResourceSyncs && canCreateRS && (
<Checkbox
id="use-resource-syncs"
className="fctl-create-repo__rs-checkbox"
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

The checkbox was misaligned with the label.
Using CheckboxField or other solutions didn't work.

@celdrake celdrake merged commit a9fdb49 into flightctl:main Feb 5, 2026
6 checks passed
@celdrake celdrake deleted the EDM-3097-disable-import-option-tooltip branch February 5, 2026 14:37
celdrake added a commit to celdrake/flightctl-ui that referenced this pull request Feb 5, 2026
celdrake added a commit that referenced this pull request Feb 5, 2026
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.

2 participants