fix(styleguide): fixed theme switching in style guide

torkelo · torkelo · commit 7b7ba46f123c · 2016-09-21T09:10:25.000+02:00
diff --git a/public/app/features/styleguide/styleguide.ts b/public/app/features/styleguide/styleguide.ts
@@ -13,7 +13,7 @@ class StyleGuideCtrl {
   pages = ['colors', 'buttons'];
 
   /** @ngInject **/
-  constructor(private $http, $routeParams) {
+  constructor(private $http, private $routeParams, private $location) {
     this.theme = config.bootData.user.lightTheme ? 'light': 'dark';
     this.page = {};
 
@@ -37,8 +37,11 @@ class StyleGuideCtrl {
   }
 
   switchTheme() {
-    var other = this.theme === 'dark' ? 'light' : 'dark';
-    window.location.href = window.location.href + '?theme=' + other;
+    this.$routeParams.theme = this.theme === 'dark' ? 'light' : 'dark';
+    this.$location.search(this.$routeParams);
+    setTimeout(() => {
+      window.location.href = window.location.href;
+    });
   }
 
 }

Original file line number	Diff line number	Diff line change
`@@ -13,7 +13,7 @@ class StyleGuideCtrl {`
`13`	`13`	`pages = ['colors', 'buttons'];`
`14`	`14`
`15`	`15`	`/ @ngInject /`
`16`		`- constructor(private $http, $routeParams) {`
	`16`	`+ constructor(private $http, private $routeParams, private $location) {`
`17`	`17`	`this.theme = config.bootData.user.lightTheme ? 'light': 'dark';`
`18`	`18`	`this.page = {};`
`19`	`19`
`@@ -37,8 +37,11 @@ class StyleGuideCtrl {`
`37`	`37`	`}`
`38`	`38`
`39`	`39`	`switchTheme() {`
`40`		`- var other = this.theme === 'dark' ? 'light' : 'dark';`
`41`		`- window.location.href = window.location.href + '?theme=' + other;`
	`40`	`+ this.$routeParams.theme = this.theme === 'dark' ? 'light' : 'dark';`
	`41`	`+ this.$location.search(this.$routeParams);`
	`42`	`+ setTimeout(() => {`
	`43`	`+ window.location.href = window.location.href;`
	`44`	`+ });`
`42`	`45`	`}`
`43`	`46`
`44`	`47`	`}`