wagtail icon indicating copy to clipboard operation
wagtail copied to clipboard

Only override snippet chooser icon if it's different

Open RealOrangeOne opened this issue 1 year ago • 3 comments

This allows overriding the icon on a per-block basis.

RealOrangeOne avatar Jul 16 '24 10:07 RealOrangeOne

Manage this branch in Squash

Test this branch here: https://realorangeonesnippet-chooser-i-0486z.squash.io

squash-labs[bot] avatar Jul 16 '24 10:07 squash-labs[bot]

@RealOrangeOne how can I manually test this functionality?

Also, should we include a unit test for this?

lb- avatar Oct 11 '24 07:10 lb-

I just opened this PR to capture the issue I spotted. Yes I agree this should absolutely have a test.

RealOrangeOne avatar Oct 11 '24 11:10 RealOrangeOne

Opened a PR with unit tests for this fix.

joelwilliam2005 avatar Feb 16 '25 17:02 joelwilliam2005

Superseded by #12883

RealOrangeOne avatar Feb 18 '25 11:02 RealOrangeOne