insightsengineering / insightsengineering/teal.osprey

[Feature Request]: srv_g_decorate and decorators

Open
#352 0 comments 0 reactions 0 assignees View on GitHub
core enhancement
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

Open the contributing 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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.