1313e/CMasher

create_cmap_overview should have a "as input" sort

endolith opened this issue · 9 comments

create_cmap_overview can take a list as input, and the list has an order already, so sort= should have an option for listing the colormaps in the order they were in the list.

1313e commented

That's not a bad idea actually.
Wonder how I did not think of that.

1313e commented

@endolith I hope you don't mind that sort=None is not the default, as I have the feeling that sorting alphabetically is slightly more logical than not sorting at all.

1313e commented

I saw yes, and immediately realized why you asked for this feature.

1313e commented

I think I will also add the feature tomorrow that one can set the title of the overview (or remove it altogether).

That would be good too. Did you see that I rewrote the names of the colormaps so they would display alongside? And I had to add a space after _r names because it filters those out automatically. I don't care that much, but maybe something to think about changing.

1313e commented

Ah, that's the reason why that works.
Yes,I had already an idea how to deal with that.

1313e commented

@endolith Can you check ed80773 and see if that works better?