Skip to content

block: Show tooltips for blocks #235

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Sep 27, 2024
Merged

block: Show tooltips for blocks #235

merged 1 commit into from
Sep 27, 2024

Conversation

wjt
Copy link
Member

@wjt wjt commented Sep 26, 2024

Prior to commit 7a6fe13, instantiate_block() would set the new Block's tooltip_text property to the BlockDefinition's description.

Since that commit, we just set a definition property on the Block to the BlockDefinition.

Implement Control._get_tooltip(), returning BlockDefinition.definition, which restores tooltips to blocks.

https://phabricator.endlessm.com/T35645

@wjt wjt requested a review from dylanmccall September 26, 2024 15:50
@wjt wjt force-pushed the push-mtvutpttoowm branch from 14c6533 to f3d0b7b Compare September 26, 2024 16:05
Copy link
Member

@dbnicholson dbnicholson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice! I was wondering how those were lost.

Prior to commit 7a6fe13, instantiate_block()
would set the new Block's tooltip_text property to the BlockDefinition's
description.

Since that commit, we just set a definition property on the Block to the
BlockDefinition.

Implement Control._get_tooltip(), returning BlockDefinition.definition, which
restores tooltips to blocks.

https://phabricator.endlessm.com/T35645
@wjt wjt force-pushed the push-mtvutpttoowm branch from f3d0b7b to b9fa778 Compare September 27, 2024 08:54
@dbnicholson dbnicholson merged commit 6a05c62 into main Sep 27, 2024
3 checks passed
@dbnicholson dbnicholson deleted the push-mtvutpttoowm branch September 27, 2024 13:20
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.

2 participants