Go to cell defining something

[Eloitor]

Description

Quickly go to the cell defining a variable or a function.

Suggested solution

• An item in the right click menu.
• A keyboard shortcut.

Alternative

No response

Additional context

No response

[alefminus]

Could this be a simple matter to code now that #1365 is fixed?

[robmck1995]

I didn't factor #1365 in when making these changes.

Here's the plan I used to make the change:
https://glide.agenticlabs.com/sharing?userId=65bae76b7fbfb1c20ac1f3b8&taskId=ghM1mqX

Do we think this needs to be updated based on the new functionality of goToDefinition? It looked like from my testing that goToDefinition just focused on the first instance of the variable within the cell, not across cells?

[alefminus]

I thought thanks to marimo's design we already know the cell defining a variable, so the remainder is going to the (single) definition in that cell. Thanks for introducing me to glide, but I think I don't have the time and understanding to contribute right now!

…

On Sun, May 19, 2024 at 3:15 PM Robert McKnight ***@***.***> wrote: I didn't factor #1365 <#1365> in when making these changes. Here's the plan I used to make the change: https://glide.agenticlabs.com/sharing?userId=65bae76b7fbfb1c20ac1f3b8&taskId=ghM1mqX Do we think this needs to be updated based on the new functionality of goToDefinition? It looked like from my testing that goToDefinition just focused on the first instance of the variable within the cell, not across cells? — Reply to this email directly, view it on GitHub <#1126 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AYWFZPD2VSJJH6W7IPOVXMTZDCJU7AVCNFSM6AAAAABGEPXL76VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMJZGIYTIOJZGA> . You are receiving this because you commented.Message ID: ***@***.***>

-- Alon Levy Software Developer Greenvibe 054-2395317