Olaf Grabienski
Olaf Grabienski
This is a great idea! I've had a quick look at the default descriptions which look good to me. I'd suggest to also add a description for the Editor role,...
Hey @kiamlaluno - thanks for your PR! In Backdrop we try to avoid the term "user" for people, or: we try to use the term only for user *accounts*. How...
Thanks for the update, @kiamlaluno . I like the phrases you found, e.g. `Allows to edit already approved comments` instead of `Users could edit ...`. Update: also tested in the...
I like the radio buttons! There's a small typo, btw: "This setting applie**d** ...". However, in my opinion we don't need this sentence in the help text, it's perfectly fine...
> The default touch/hover change is caused by device screen size, not whether they are touch-screen devices or not. 👍 > I've created a PR, do you think this is...
> PR updated Looks good to me, only one thing: I think we don't use colons for the heading of a setting. `Dropdown behavior on large screens:` should be `Dropdown...
>> The default touch/hover change is caused by device screen size, not whether they are touch-screen devices or not. (...) Large iPads, for example, currently use the hover effect even...
Thanks for the updated PR! Here are the current state and the last suggestion for comparison: Last suggestion: ``` Dropdown behavior - Open with hover or touch - Open with...
> Can you think of a better word than "trigger" (...) that's specific to what opens the submenu? Let's see! Maybe including one of the terms "to open" or "submenu"...
Thanks @klonos. I'm not sure if I agree to emphasize the verb in the label here. The approach sounds generally elegant, but I think a sentence shouldn't be split up...