[ENG-1637] Add Hover info text on base icon

[trangdoan982]

Before:

After:

---

[linear-code]

ENG-1637 Add Hover info text on base icon

[supabase]

This pull request has been ignored for the connected project zytfjzqyijgagqxrzbmz because there are no changes detected in packages/database/supabase directory. You can change this behaviour in Project Integrations Settings ↗︎.

---

Preview Branches by Supabase.
Learn more about Supabase Branching ↗︎.

[devin-ai-integration]

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 1 additional finding.

[graphite-app]

The tooltip uses nodeType.name directly, but line 158 shows that nodeType.name can be falsy (using nodeType.name || "Unnamed Node Type"). When nodeType.name is undefined/null/empty, the tooltip will display inconsistent text like "Create Base view for undefined nodes" while the UI shows "Unnamed Node Type".

Fix:

ref={(el) => {
  if (el) {
    const name = nodeType.name || "Unnamed Node Type";
    setTooltip(el, `Create Base view for ${name} nodes`);
  }
}}

Suggested change

	ref={(el) => {
	if (el)
	setTooltip(el, `Create Base view for ${nodeType.name} nodes`);
	}}
	ref={(el) => {
	if (el)
	const name = nodeType.name || "Unnamed Node Type";
	setTooltip(el, `Create Base view for ${name} nodes`);
	}}

Spotted by Graphite



Is this helpful? React 👍 or 👎 to let us know.

[mdroidian]

Probably should deal with the graphite suggestion, or maybe handle null nodeType.name higher up the chain.