Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improve inputs in ACP setting page #17

Open
Eldenroot opened this issue Sep 18, 2019 · 3 comments
Open

Improve inputs in ACP setting page #17

Eldenroot opened this issue Sep 18, 2019 · 3 comments
Labels
enhancement New feature or request

Comments

@Eldenroot
Copy link
Member

Here for this field:

image

One word per line would be much better as I had suggested before. Much easier for import/export.

Same for:
image

@Eldenroot Eldenroot added the enhancement New feature or request label Sep 18, 2019
@effone effone added this to the 1.2.0 - Feature Release milestone Sep 18, 2019
@effone
Copy link
Member

effone commented Sep 18, 2019

I guess comma separated user ids are fine.
It is being used that way in many plugins for ages.

String list can be modified, yes.

@WildcardSearch
Copy link

I have to agree with @effone here. IDs are fine as comma-separated list. With the strings, I agree with @Eldenroot because what if the string needs to include a comma?

@Eldenroot
Copy link
Member Author

OK, leave user IDs as it is now but lets discuss about the string list; one word per line and you can use whatever you want (including comma).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants