-
Notifications
You must be signed in to change notification settings - Fork 88
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Use the default_theme value from the login #1709
Conversation
Do you have time today to rebase this PR then provide an example ? (I'm supposed to review this PR) |
// currentTheme = gmf.Themes.findThemeByName(this.themes, this.defaultTheme); | ||
// } | ||
|
||
// this.setTheme(/** @type {GmfThemesNode} */ (currentTheme), true); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Shouldn't we remove this commented code?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
still in "work in progress" ...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ok :-)
It can't be tested because this pull request depends on camptocamp/c2cgeoportal#2456 and geomapfish-demo.camptocamp.net is not yet updated |
a876a0c
to
281c426
Compare
Demo updated :-) |
4fe6271
to
9243050
Compare
And check the user functionalities from a 'default_theme'
9243050
to
5fdba09
Compare
@sbrunner still no |
But we can merge and check later with the demo |
@sbrunner ready to be reviewed |
ping @sbrunner |
For me that's looks good :-) |
Thanks for the review |
fixes #654