fix (design library): change to question mark

[kaeizen]

Summary by CodeRabbit

• Style
  • Updated the tooltip icon shown when items are selected from a lock to a help icon, providing a clearer visual cue.
  • Retains existing tooltip text and selection behavior; only the iconography has changed.
  • Improves clarity without altering workflows or functionality.

[coderabbitai]

Walkthrough

The tooltip icon in DesignLibraryListItem changes from "lock" to "editor-help" when selectedNum !== 0. No other logic, text, or conditions are modified. No public APIs are changed.

Changes

Cohort / File(s)	Summary
UI Icon Update src/components/design-library-list/design-library-list-item.js	Replace Dashicon icon="lock" with icon="editor-help" inside the tooltip when selectedNum !== 0; no other logic or text changes.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

I nudge a glyph, a gentle hop—
From lock to help, the hints now pop.
A whisper of UI delight,
Icons moonwalk in the night.
Carrots clicked, I sign and squeal:
“A tiny change, a polished feel!” 🥕✨

Pre-merge checks and finishing touches

✅ 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 and concisely describes the primary change: a fix in the design library that replaces the icon with a question-mark (the tooltip icon change), and it clearly scopes the change to the design library while indicating intent.
Docstring Coverage	✅ Passed	No functions found in the changes. Docstring coverage check skipped.

✨ Finishing touches

• 📝 Generate Docstrings

🧪 Generate unit tests

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch fix/minor-fix-design-library

---

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

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[github-actions]

🤖 Pull request artifacts

file	commit
pr3599-stackable-3599-merge.zip	dbc17c4

[github-actions]

Try this Pull Request in the WordPress playground: https://playground.wordpress.net/?mode=seamless#{"landingPage":"/wp-admin/post-new.php?post_type=page","preferredVersions":{"php":"latest","wp":"latest"},"steps":[{"step":"login","username":"admin","password":"password"},{"step":"installPlugin","pluginZipFile":{"resource":"url","url":"https://raw.githubusercontent.com/gambitph/Stackable/artifacts/pr3599-stackable-3599-merge.zip"}}]}

[coderabbitai]

Actionable comments posted: 0

🧹 Nitpick comments (2)

src/components/design-library-list/design-library-list-item.js (2)

136-140: Guard condition can render tooltip when not selected (selectedNum defaults to false).

selectedNum !== 0 is true when selectedNum === false, so the tooltip shows unintentionally on initial render. Prefer a numeric check.

Apply this diff:

-					{ selectedNum !== 0 &&
+					{ Number(selectedNum) > 0 &&
 						<Tooltip text={ __( 'Style options are locked for this design because it is selected.', i18n ) }>
 							<Dashicon icon="editor-help" size={ 16 } />
 						</Tooltip>
 					}

---

137-139: Optional: Align tooltip copy with the “help” icon.

Slightly tighten the message to read more like an informational hint.

Apply this diff:

-						<Tooltip text={ __( 'Style options are locked for this design because it is selected.', i18n ) }>
+						<Tooltip text={ __( 'Styles are locked while this design is selected.', i18n ) }>
 							<Dashicon icon="editor-help" size={ 16 } />
 						</Tooltip>

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 51e501e and dbc17c4.

📒 Files selected for processing (1)

• src/components/design-library-list/design-library-list-item.js (1 hunks)

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (6)

• GitHub Check: PHP 8.2 and WP latest
• GitHub Check: PHP 7.3 and WP latest
• GitHub Check: PHP 7.3 and WP 6.5.5
• GitHub Check: PHP 8.2 and WP 6.5.5
• GitHub Check: PHP 8.2 and WP 6.7.2
• GitHub Check: PHP 8.2 and WP 6.6.2

🔇 Additional comments (1)

src/components/design-library-list/design-library-list-item.js (1)

138-138: Icon swap LGTM.

Using Dashicon "editor-help" for the tooltip looks good.