Add Bosnian locale #2495
Add Bosnian locale #2495
Conversation
|
|
||
| switch (day) { | ||
| case 0: | ||
| return "'sledeće nedjelje u' p" |
There was a problem hiding this comment.
I think that sljedeće is correct one here. https://bs.wikipedia.org/wiki/Wikipedia:Pravilno-nepravilno_u_bosanskom_jeziku/2018.#S
There was a problem hiding this comment.
Fixed. Thank you!
src/locale/bs/index.js
Outdated
| match: match, | ||
| options: { | ||
| weekStartsOn: 1 /* Monday */, | ||
| firstWeekContainsDate: 1, |
There was a problem hiding this comment.
hi @BranislavLazic
Could you please provide a reference that confirms the value of 1 of firstWeekContainsDate?
According to Wikipedia, it is 4 for the majority of European countries.
Do you possibly know a source that we can refer to?
There was a problem hiding this comment.
Seems like you are right. Also, some of them share the same flaw. E.g. sr-Latn (which was my reference locale) or hr.
|
@BranislavLazic |
There was a problem hiding this comment.
Much appreciated! 👍 👍
I will schedule this for our next release given the go-ahead by @kossnocorp
This PR adds Bosnian locale and resolves this issue #2435.