테마의 변환 버튼에 큐브 아이콘이 없습니다

[Moin]

I noticed that the convert button in the theme controls for themes does not have an icon,

whereas it is displayed for theme components.

[sam (Sam Saffron)]

Good catch, going to put a pr-welcome on this one.

[tgxworld (Alan Tan)]

app/assets/javascripts/admin/addon/controllers/admin-customize-themes-show-index.js

The change seems to be intentional. @Osama Do you happen to remember why?

[Osama]

I’ve dug into the history here and I believe we decided not to use an icon simply because we couldn’t find/come up with an icon to represent components.

This is a screenshot from when the concept of themes and components was introduced (2018). There’s no icon for the components tab for the same reason.

---

Given all the recent changes we’ve made around themes/components, I’m not sure if it even makes sense to have the convert button anymore — I don’t think there’s a use case for it these days? I certainly don’t remember the last time I needed to use it.

[tgxworld (Alan Tan)]

I think a simple change we can make here is just to use a consistent icon for the button which I am proposing in this PR:

https://github.com/discourse/discourse/pull/34317

There should be no need for the button’s icon to convey the kind of conversion that is happening for the theme because that is handled by the confirmation dialog when the button is clicked.

Pinging @hugh and @awesomerobot for a quick review.

[hugh (Hugh Lashbrooke)]

This looks good to me