Cleanup: Remove old vertex/weight paint brush icons, reuse texture brush instead
Needs ReviewPublic
Actions

Authored by Yevgeny Makarov (jenkm) on Feb 6 2020, 10:04 AM.

Details

Reviewers

William Reynish (billreynish)
Julian Eisel (Severin)
Pablo Vazquez (pablovazquez)

Group Reviewers

User Interface

Summary

Remove old vertex and weight paint brush icons and reuse the existing texture brush instead.

ICON_BRUSH_BLUR > soften.png
ICON_BRUSH_MIX > texdraw.png

../release/datafiles/brushicons/blur.png
../release/datafiles/brushicons/mix.png
../release/datafiles/brushicons/multiply.png

Diff Detail

Repository: rB Blender

Event Timeline

Yevgeny Makarov (jenkm) created this revision.Feb 6 2020, 10:04 AM

I had hoped we would get around to completely removing these fixed brush icons in favour of dynamically generated brush previews sooner.

But until then this is an improvement that removes the ridiculous eye sore that are the old vertex and weight paint brush icons at least.

This revision is now accepted and ready to land.Feb 6 2020, 11:24 AM

William Reynish (billreynish) added a reviewer: Pablo Dobarro (pablodp606).Feb 6 2020, 11:24 AM

xrg (xrg) awarded a token.Feb 6 2020, 1:01 PM

updated to reflect current state of master

Yevgeny Makarov (jenkm) retitled this revision from Cleanup Brush Icons to Remove old vertex/weight paint brush icons, reuse texture brush instead.Jul 17 2020, 5:59 PM

Yevgeny Makarov (jenkm) edited the summary of this revision. (Show Details)

Yevgeny Makarov (jenkm) added a reviewer: User Interface.

Yevgeny Makarov (jenkm) updated this revision to Diff 30514.Oct 29 2020, 9:47 PM

Yevgeny Makarov (jenkm) retitled this revision from Remove old vertex/weight paint brush icons, reuse texture brush instead to Cleanup: Remove old vertex/weight paint brush icons, reuse texture brush instead.

Yevgeny Makarov (jenkm) edited the summary of this revision. (Show Details)

Yevgeny Makarov (jenkm) edited reviewers, added: Julian Eisel (Severin); removed: Campbell Barton (campbellbarton), Pablo Dobarro (pablodp606).

This patch has an approved status however it was not committed yet. So I'm assuming the other reviewers are blocking. Updating the reviewers list to reflect this. This way the patch status still show as Need Review.

This revision now requires review to proceed.Mar 26 2021, 6:11 PM