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.

Sign up for GitHub

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

Jump to bottom

use title case in default module label #1089

Open
chlebowa opened this issue Feb 6, 2024 · 0 comments · May be fixed by #1459
Open

use title case in default module label #1089

chlebowa opened this issue Feb 6, 2024 · 0 comments · May be fixed by #1459
Labels
core good first issue Good for newcomers

Comments

@chlebowa
Copy link
Contributor

chlebowa commented Feb 6, 2024

The default label for the previewer module is in sentence case "Report previewer", while (nearly) all other modules use title case, e.g. "Variable Browser".

Update the reporter module label to title case.
@chlebowa chlebowa added the core label Feb 6, 2024
@vedhav vedhav added the good first issue Good for newcomers label May 27, 2024
meldolan added a commit to meldolan/teal that referenced this issue Jun 11, 2024
@meldolan
Update reporter_previewer_module.R
b80be98 
changing "Report previewer" to "Report Previewer" so its using title case like the other modules

use title case in default module label insightsengineering#1089

Signed-off-by: meldolan <[email protected]>
@meldolan meldolan mentioned this issue Jun 11, 2024
Closed
@jimrothstein jimrothstein linked a pull request Jan 22, 2025 that will close this issue
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
core good first issue Good for newcomers
Projects
None yet
Milestone
No milestone
2 participants
@vedhav @chlebowa