Fix #108632: Icon preview fix for material utils plugin. #104675

Closed
YimingWu wants to merge 1 commits from ChengduLittleA/blender-addons:main into main

When changing the target branch, be careful to rebase the branch in your fork to match. See documentation.
Member

The original material.preview.icon_id call will fail if preview is
None, now fixed with icon_value = layout.icon(material)

The original `material.preview.icon_id` call will fail if `preview` is `None`, now fixed with `icon_value = layout.icon(material)`
Author
Member

Looks like the ID can't span to other repositories...

Looks like the ID can't span to other repositories...
YimingWu force-pushed main from 4b69f70602 to 20752d5381 2023-07-30 16:11:28 +02:00 Compare
Member

@ChengduLittleA I think this can be closed, as the issue was fixed by blender/blender-addons@932ed7422d. (Also the content of the PR doesn’t seem to be expected?)

@ChengduLittleA I think this can be closed, as the issue was fixed by blender/blender-addons@932ed7422dd9ddfbb2c6bf7e599de4d423a37372. (Also the content of the PR doesn’t seem to be expected?)
Author
Member

@pioverfour Looks like so. Thanks for the following up.

@pioverfour Looks like so. Thanks for the following up.
YimingWu closed this pull request 2024-01-25 03:09:06 +01:00

Pull request closed

Sign in to join this conversation.
No reviewers
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: blender/blender-addons#104675
No description provided.