Skip to content

[IconPack] Replace code snippet tooltip on Package and IconPack name fields with a plain text description tooltip#1017

Merged
egorikftp merged 1 commit into
mainfrom
feature/idea/iconpack
Jun 6, 2026
Merged

[IconPack] Replace code snippet tooltip on Package and IconPack name fields with a plain text description tooltip#1017
egorikftp merged 1 commit into
mainfrom
feature/idea/iconpack

Conversation

@egorikftp

@egorikftp egorikftp commented Jun 5, 2026

Copy link
Copy Markdown
Member
Screenshot 2026-06-05 at 23 36 45

📝 Changelog

If this PR introduces user-facing changes, please update the relevant Unreleased section in changelogs:

…fields with a plain text description tooltip
@egorikftp egorikftp force-pushed the feature/idea/iconpack branch from ecf5ed4 to 68fc262 Compare June 5, 2026 20:38
@coderabbitai

coderabbitai Bot commented Jun 5, 2026

Copy link
Copy Markdown

Need an answer fast? Review this PR in Change Stack to ask focused questions about the PR or a changed range.

Review Change Stack

Walkthrough

This PR refactors the IconPack editor UI to replace code-snippet tooltips with plain-text descriptions. A new reusable LabelTooltipRow composable is introduced that pairs a label with an info icon tooltip. The IconPackEditor is updated to use this component for package and icon pack name fields instead of the previous CodeTooltipHeader. Supporting tooltip strings are added to the properties file, and the changelog documents both the tooltip change and a new unrelated Settings option for suppressing unused receiver parameter annotations.

🚥 Pre-merge checks | ✅ 4 | ❌ 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 (4 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and specifically describes the main change: replacing code snippet tooltips with plain text tooltips for Package and IconPack name fields.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Description check ✅ Passed Pull request description includes required changelog checklist with IntelliJ Plugin marked as updated, matching the file changes in CHANGELOG.md.

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

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feature/idea/iconpack

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.

@egorikftp egorikftp merged commit 1189fb0 into main Jun 6, 2026
4 checks passed
@egorikftp egorikftp deleted the feature/idea/iconpack branch June 6, 2026 07:37
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