Codeinwp / Codeinwp/visualizer
Use default values instead of placeholders ( or remove placeholders )
Nobody has claimed this yet.
- Dominant language
- PHP
- Stars
- 108
- Forks
- 29
- Avg merge
- 1d 9h
- Merged PRs (30d)
- 7
Description
This is coming from me, as I've always found the options using placeholders confusing.
Right now, for the majority of options, we are using placeholders ( for e.g. for the Minor Grid Lines it's 0 ), and I think that can create confusion. I have the feeling that's the current value for that field. I think we should either have that as a default value or not have it at all, as it doesn't help, from my point of view.
Moved from here https://github.com/Codeinwp/visualizer/issues/418
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start with the original issue referenced at https://github.com/Codeinwp/visualizer/issues/418, then inspect how chart option fields currently present placeholders. Decide whether the affected options should expose defaults or omit placeholders, and consider the work done when the confusing placeholder behavior is handled consistently across the relevant options.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- php, wordpress
- Domain
- frontend
- Issue type
- Feature
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 25/100