Fix: Add missing notebook tooltips #10635
Conversation
|
E2E Tests 🚀 |
dhruvisompura
left a comment
dhruvisompura
left a comment
There was a problem hiding this comment.
Thank you for adding tooltips! Overall, this looks great. I just have a couple suggestions for adding some more missing tooltips:
I noticed that the delete cell action doesn't have a tooltip so the action title is getting used. When I hover over the trash icon it says "Delete the Selected Cell(s)" but it should probably just say "Delete Cell" since you can't actually delete a selection of cells via the cell action bar anymore?
What do you think about updating the tooltip for the "run above" and "run below" cell actions to say "Run Cells Above" and "Run Cells Below"? I think that reads a bit smoother than the action title.
seeM
force-pushed
the
fix/notebook-cell-tooltips
branch
from
a96e36a to
a59bbb9
Compare
|
@dhruvisompura I like your suggestions! Added in a59bbb9. Will merge once CI is green |
Adds missing tooltips to buttons in the notebook cell action bar, addressing #9896.
I did not add tooltips to the run/interrupt buttons until we decide what to do with the existing hover (#10247).
Screen.Recording.2025-11-18.at.18.27.34.mov
Release Notes
New Features
Bug Fixes
QA Notes
Verify that helpful tooltips appear on all relevant notebook buttons.
@:positron-notebooks @:web