feat(UploadPicker): Allow to set color to primary

[susnux]

Required for nextcloud/server#48527

[Pytal]

Could also work with a buttonType prop and pass it as is?

[susnux]

Could also work with a buttonType prop and pass it as is?

IMHO to many options as there is no sense in making it error or warning state.

[artonge]

I think I agree with Pytal, even if there is no reason for now, it also lowers the complexity by making it transparent what we are doing under the hood.

[susnux]

I think I agree with Pytal, even if there is no reason for now, it also lowers the complexity by making it transparent what we are doing under the hood.

Not sure as then some will use weird combinations which do not want. We want consistent design across all apps so we should limit what dev can do here to sane options.

[artonge]

so we should limit what dev can do here to sane options.

I don't think I agree here, and in any case, devs can still bypass the property by overriding the CSS.
A sane default is enough in my opinion, and we if you really want to limit options, we can hard code the possible values.

[skjnldsv]

I do get both side, and i'll raise another reason to go this way: if the vue button API changes, we suddenly have to also update this API here.
While keeping a boolean, we get to decide how it will impact the uploader design.

devs can still bypass the property by overriding the CSS.

If a dev decide to go this way, they'll already lost.
We can't control everything, but devs should know that if things are getting difficult for what they're trying to achieve, they're probably doing it wrong. 😉