Skip to content

feat(date-formatter): update range dates formatter to show current year in some cases #137

Merged
merged 2 commits into from
Jun 7, 2019
Merged

feat(date-formatter): update range dates formatter to show current year in some cases #137

merged 2 commits into from
Jun 7, 2019

Conversation

roll314
Copy link
Contributor

@roll314 roll314 commented Jun 5, 2019

@fost
@lskramarov
@mikeozornin

TFS - 185214

Паста от Мишы

По нашему формату мы не пишем год, когда он текущий. Это приводит к таким картинкам:
http://screenshots.ptsecurity.com/mozornin-2019-06-04_14-11-36.png
Догадаться, что это 2019 невозможно, это небольшая катастрофа. Хочется переработать диапазоны.

Сейчас: Если год текущий на одном из концов, то его не пишем
Надо бы: Если год текущий на обоих концах совпадает, то их оба не пишем.

Примеры здесь: wiki - 20332985 исправлены

@pimenovoleg
Copy link
Member

@roll314 добавь пжл более ясное описание в PR и название

@roll314 roll314 changed the title feat(date-formatter): update Update range dates in date formatter to show current year in some cases Jun 5, 2019
@roll314 roll314 changed the title Update range dates in date formatter to show current year in some cases feat(date-formatter): update range dates formatter to show current year in some cases Jun 5, 2019
@mikeozornin
Copy link
Contributor

mikeozornin commented Jun 5, 2019

А почему здесь в английском другой год?

http://d.mikeozornin.ru/jhaUuG
http://d.mikeozornin.ru/KespBr

@roll314
Copy link
Contributor Author

roll314 commented Jun 6, 2019

косяк примера, поправлю

@roll314
Copy link
Contributor Author

roll314 commented Jun 6, 2019

обновил пример

Copy link
Contributor

@mikeozornin mikeozornin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ага

@pimenovoleg pimenovoleg merged commit fd51697 into positive-js:master Jun 7, 2019
pimenovoleg pushed a commit that referenced this pull request Jun 7, 2019
pimenovoleg pushed a commit that referenced this pull request Jun 7, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants