… & add missing translation.
''empty option, use that in the dropdown. This still allows showing the placeholder if the value is set to, eg, null or undefined).
@Findus23 what do you think? Would this make selecting all sites clearer to you?
The placeholder font color is theme related so won't address in this issue. Though if it's a problem, perhaps a separate issue is called for.
fyi it worked for me and code looks 👍 .
From a UX point of view it might be worth thinking about having some content above this saying something like "Give access to all websites [permission selector] [confirm button]". Then we could also add a note that user will get only access to currently existing websites, not to websites created in the future. But we could always add this later in a later release as it doesn't seem too important maybe.
Works :+1: and nice to use. feel free to merge once tests pass