Should ZoomButtonGroup support magnifying glass icons?

[samreid]

From phetsims/circuit-construction-kit-common#620 "Zoom buttons need magnifying glasses".

Circuit construction kit is using ZoomButtonGroup for the chart zoom in/out buttons, in the aforementioned issue, we said:

From today's design meeting, "+" and "-" are overloaded in CCK, and showing those as button icons without other context will be confusing. We need the magnifying glass icons. This is for the charts and for the play area zoom buttons.

It currently looks like this:

Assigning to the primary developer for ZoomButtonGroup for discussion.

[samreid]

I added a comment in phetsims/circuit-construction-kit-common#620 (comment) about what the other zoom buttons in CCK AC look like. Maybe we should wait for a conclusion on that conversation before going too far with this issue.

[pixelzoom]

Strong push back on adding magnifying glasses to ZoomButtonGroup. This was specifically designed for cases where you do not need (heavyweight) magnifying glasses. If you need magnifying glasses, use ZoomButton.

[samreid]

Are you recommending a parallel implementation of ZoomButtonGroup that uses the magnifying glass icons?

UPDATE: Why not make the icons options that default to "+" and "-"?

[pixelzoom]

UPDATE: Why not make the icons options that default to "+" and "-"?

More complexity, more cost. Imo unnecessary. That's my opinion. If someone wants to add it, go for it -- and please add yourself as @author.

[samreid]

On hold until we have a consensus in phetsims/circuit-construction-kit-common#620

[samreid]

We added support in phetsims/circuit-construction-kit-common#620, closing.