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

Allow field formatter to work with list_text as well #9

Closed
herbdool opened this issue Mar 2, 2021 · 2 comments · Fixed by #10
Closed

Allow field formatter to work with list_text as well #9

herbdool opened this issue Mar 2, 2021 · 2 comments · Fixed by #10

Comments

@herbdool
Copy link
Contributor

herbdool commented Mar 2, 2021

In my use case I want to preset a bunch of icons for a field, and not leave it up to the author to know what classes mean.

This is easily done with the list_text field type. I've got it working so I can provide a pr soon.

By the way, it wasn't clear at first where this formatter would be available and how one should use it. I had to look in the code. (Oops, I missed that part of the readme).

@herbdool
Copy link
Contributor Author

herbdool commented Mar 3, 2021

@BWPanda I've got a PR.

@ghost ghost closed this as completed in #10 Mar 3, 2021
ghost pushed a commit that referenced this issue Mar 3, 2021
@ghost
Copy link

ghost commented Mar 3, 2021

Thanks @herbdool! I made a small change to fix the line length in the README, but otherwise merged your PR as-is.

This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant