Icoon type SiteSetting

Ik probeer een SiteSetting toe te voegen aan mijn plugin die de iconen kan beheren die voor sommige componenten worden weergegeven. De component zelf zou sowieso al door de gebruiker aanpasbaar zijn, dus ik wilde de SiteSetting toevoegen als een manier voor medewerkers om het standaardicoon te definiëren. Ik weet dat er een emoji_list-type is voor SiteSettings, maar er lijkt er geen te zijn voor SVG-iconen?

Voor zover ik kan zien, gebruiken de meeste plugins/TC’s ofwel een afbeeldingupload of een platte string voor icooninstellingen. Mis ik een type/validator die ik kan gebruiken? Het voelt vreemd om mensen te vertellen dat ze de styleguide moeten raadplegen en de naam van het icoon moeten typen.

1 like

I think it’s a case where we don’t have any icon based site settings, so we haven’t added a type for it. We have an icon picker component that’s made its way into other parts of the app, but nothing currently in site settings.

Dat is logisch. Ik kan me geen specifiek scenario voorstellen waarin core dit zou vereisen, aangezien waarschijnlijk alleen Thema’s/TC’s en plugins een gebruik zouden hebben voor een extreem UI-gerichte instelling.

Ik gebruik het icoonkiezercomponent op andere plaatsen die gemakkelijker toegankelijk zijn voor gewone gebruikers, maar het specifieke gebruiksscenario dat ik in gedachten heb, zou zijn voor zaken als het instellen van een standaardicoon. In combinatie met het object SiteSetting type, zou het het templaten van bepaalde functies voor beheerders gemakkelijker maken.

Zou het team openstaan om een icon_type type toe te voegen aan SiteSettings in core, zelfs zonder dat core site-instellingen dit gebruiken? Het maakt me niet uit om dit onderwerp opnieuw te openen als een Feature request indien nodig.

Is er een update hierover? Ik merkte dat reacties aan de kern zullen worden toegevoegd, en dat de plugin momenteel de instelling Discourse reactions like icon heeft waarbij je de icoonaam handmatig moet invoeren.

3 likes

Momenteel geen plannen, maar misschien is dat een goed moment daarvoor. Ik geef het idee door.

2 likes