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

Since placeholder text is dynamically generated, make it optional via env var #107

Closed
asimons04 opened this issue Aug 16, 2023 · 0 comments

Comments

@asimons04
Copy link

asimons04 commented Aug 16, 2023

Is your feature request related to a problem? Please describe.
The text fields in post and comment forms are pretty self explanatory. The placeholder text makes them too busy and adds unnecessary confusion especially since it is randomly selected.

Describe the solution you'd like
Ideally, just remove the randomly selected placeholder text as it's not really adding much value.

If removing them entirely is off the table, then add some kind of global setting (e.g. env var DISABLE_PLACEHOLDERS=true) that will cause the placeholders.get() function in utils.ts to return an empty string.

Describe alternatives you've considered
I do a custom build and neuter the placeholders object in src/lib/util.ts as such:

export const placeholders = {
  get: (type: 'url' | 'post' | 'comment') => {
    return ''
  },
}
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant