decimal for config(locale = "nl")
Nobody has claimed this yet.
Assessment
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Newbie friendliness
- 55/100
- Issue type
- Bug
- Clarity
- Mostly clear
- Activity status
- Quiet
- Tech stack
- javascript, r
- Domain
- data-visualization
Research direction
Reproduce the example with config(locale = "nl") and inspect how the R package applies the locale from plotly-locale-nl.js. Trace where decimal formatting is selected, then verify that the chart displays Dutch decimal commas without affecting other locales.
Written by the indexing model from the issue text.
Description
When using config(locale = "nl") (Dutch) for data with decimals, "." is used as a separator instead of "," (Dutch default).
plot_ly(data.frame(x = c("A", "B", "C"), y = c(1.2, 1.5, 1.6)), x = ~x, y = ~y, type = "bar") %>% config(locale = "nl")
When I look in plotly-locale-nl.js it seems to be correct (decimal:","). Is this something that can be fixed?
Thank you.
- Dominant language
- R
- Stars
- 2.7k
- Forks
- 641
- PR merge metrics
- No merged PRs in 30d
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.
More from plotly/plotly.R
-
Difficulty 2/5 1-3 hours Newbie friendliness 62/100
-
Difficulty 4/5 3-5 days Newbie friendliness 35/100
-
Difficulty 3/5 1-2 days Newbie friendliness 68/100
-
Difficulty 3/5 1-2 days Newbie friendliness 72/100
-
save_image Error Open
Difficulty 3/5 1-2 days Newbie friendliness 52/100
Similar issues
-
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
easystats/performance#950 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 70/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
briandconnelly/airnow#9 ·
-
bug
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
OHDSI/CohortConstructor#774 ·