[ProgressIndicator] Property indicatorWidth redundant/missleading

Is your feature request related to a problem? Please describe.

The following applies to the LINEAR indicator:

Using the ProgressIndicator property indicatorWidth changes the height of the ProgressIndicator which is not obvious from its name. In addition the height should rather be controlled by the regular height property on the given layout params. Atm the indicatorWidth is used as a fix constant wich makes it unnecessarily complicated to control the View height with layout params.

Describe the solution you’d like

Remove misleading indicatorWidth in favor of the layout params height

[hunterstich]

Hey @mpost,

Thanks for the feedback. Can definitely see where you’re coming from. If I remember correctly, I think indicatorWidth was the most universal property name considering it also maps to the “strokeWidth” of a circular indicator, but @pekingme might have some more info. I’ll bring this up with the team.

[mpost]

Thanks for the additional info. I understand the constraints applied to the layout behavior in regards to the drawable used by the view. Therefore it could be argued that the intrinsic size has precedence over any layout height given.

Renaming the property to indicatorSize is a good step.