Skip to content

Panel starter: icons RustGIS has - #1

Merged
giswqs merged 1 commit into
mainfrom
panel-icons
Oct 19, 2026
Merged

giswqs merged 1 commit into
mainfrom
panel-icons

Conversation

@giswqs

@giswqs giswqs commented Oct 19, 2026 •

Copy link
Copy Markdown
Member

The panel starter's refresh, info and layers icons aren't RustGIS icon names, so the Refresh and Show Layer buttons and the panel's own button showed the puzzle piece (seen in a ui_shot screenshot). They're now update, identify and contents.

Summary by CodeRabbit

  • Style
    • Updated the panel and two command icons.

refresh, info and layers aren't RustGIS icon names, so their buttons
showed the puzzle piece.
@coderabbitai

coderabbitai Bot commented Oct 19, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: Organization UI
  • Review profile: ASSERTIVE
  • Plan: Advanced
  • Run ID: c2241ff7-3fdf-44c4-97d3-735a1bd68eba
📥 Commits

Reviewing files that changed from the base of the PR and between 22d6292 and c087395.

📒 Files selected for processing: 1
  • starters/panel/plugin.json

Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.

📝 Walkthrough

The panel starter updates the icons for the panel-state command, the show-layer command, and the panel. Other command metadata remains unchanged.

Changes: Panel icon metadata

Layer / File(s) Summary
Update panel icon values
starters/panel/plugin.json
The panel-state icon changes from refresh to update. The show-layer icon changes from info to identify. The panel icon changes from layers to contents.

Priority: ⬇️ Low

Estimated code review effort: 1 (Trivial) | ~5 minutes

Change: Bug fix

Merge Risk: ⚪ Minimal · up to c0873

The replacement icons are available in the starter’s minimum supported RustGIS version. No icon-related issue remains to resolve before merge.

Pre-merge checks: Passed 5
Check name Status Explanation
Description Check Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check Passed The title identifies the panel starter icon change and relates to the main pull request objective. The wording is awkward but remains understandable.
Docstring Coverage Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
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.
Full details: Docstring Coverage

Explanation

No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (1 skipped: 1 unsupported.)

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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

I’m a rabbit, pleased to hop,
Past three icons in a tidy swap.
refresh becomes update today,
info and layers make way.
New icons greet my ears at play.

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

@giswqs
giswqs merged commit 3f370f6 into main Oct 19, 2026
6 checks passed
@giswqs
giswqs deleted the panel-icons branch October 19, 2026 19:43
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.

1 participant