fix(calendarTitle): use start of iso week instead of week for the wee…

…k view title Closes #557
mattlewis92 · Mar 18, 2017 · 357cd28 · 357cd28
1 parent f7949cf
commit 357cd28
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/src/services/calendarTitle.js b/src/services/calendarTitle.js
@@ -13,7 +13,7 @@ angular
     function week(viewDate) {
       return calendarConfig.titleFormats.week
         .replace('{week}', moment(viewDate).isoWeek())
-        .replace('{year}', moment(viewDate).startOf('week').format('YYYY'));
+        .replace('{year}', moment(viewDate).startOf('isoweek').format('YYYY'));
     }
 
     function month(viewDate) {

diff --git a/test/unit/services/calendarTitle.spec.js b/test/unit/services/calendarTitle.spec.js
@@ -24,6 +24,10 @@ describe('calendarTitle', function() {
     expect(calendarTitle.week(new Date('2016-01-01'))).to.equal('Week 53 of 2015');
   });
 
+  it('should use the start of the iso week for the year number', function() {
+    expect(calendarTitle.week(new Date('2018-01-01'))).to.equal('Week 1 of 2018');
+  });
+
   it('should give the correct month title', function() {
     expect(calendarTitle.month(calendarDay)).to.equal('May 2015');
   });