insightsengineering / insightsengineering/teal.osprey
[Feature Request]: srv_g_decorate and decorators
- Dominant language
- R
- Stars
- 5
- Forks
- 2
- Avg merge
- 10h 8m
- Merged PRs (30d)
- 3
Description
### Feature description
This package has its own way to decorate plots with modules like `srv_g_decorate()` and `ui_g_decorate()`. They are custom modules not really decorators as they are inserted as part of the modules on the package.
Now that a general solution exists and it is implemented on #338, it would be great if the decorator is improved (see #46) and exported, or if it is removed and left for the user to specifiy it with decorators elsewhere (uteals?)
### Code of Conduct
- [x] I agree to follow this project's Code of Conduct.
### Contribution Guidelines
- [x] I agree to follow this project's Contribution Guidelines.
### Security Policy
- [x] I agree to follow this project's Security Policy.
Contributor guide
Research direction
Start by reviewing the decorator approach in #338 and the requested improvements in #46, then inspect srv_g_decorate() and ui_g_decorate() in the package. Determine whether the general decorator should be improved and exported or whether these custom modules should be removed so users specify decorators elsewhere; done requires a decided direction and corresponding package behavior.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- r
- Domain
- tooling
- Issue type
- Feature
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Quiet
- Clarity
- Needs clarification
- Newbie friendliness
- 30/100