Skip to content

Display tooltip with repository item name for repository ID#278

Merged
OrfeasZ merged 7 commits into
masterfrom
feat/repository-item-tooltip
Jun 2, 2026
Merged

Display tooltip with repository item name for repository ID#278
OrfeasZ merged 7 commits into
masterfrom
feat/repository-item-tooltip

Conversation

@pavledev

Copy link
Copy Markdown
Collaborator

No description provided.

Comment on lines +965 to +971
if (m_RepositoryResource.GetResourceInfo().status != RESOURCE_STATUS_VALID) {
return "";
}

const auto s_RepositoryData = static_cast<THashMap<
ZRepositoryID, ZDynamicObject, TDefaultHashMapPolicy<ZRepositoryID>>*>(m_RepositoryResource.
GetResourceData());

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Could probably also check if GetResourceData() is null here too.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

I don't think that's needed since RESOURCE_STATUS_VALID status is set when resource data is assigned to resourceData field

@OrfeasZ OrfeasZ merged commit add244d into master Jun 2, 2026
2 of 3 checks passed
@OrfeasZ OrfeasZ deleted the feat/repository-item-tooltip branch June 2, 2026 15:51
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