Skip to content
This repository was archived by the owner on Sep 5, 2024. It is now read-only.

fix(theming): fix read-only $mdThemingProvider.configuration() #9389

Merged
merged 1 commit into from
Aug 23, 2016

Conversation

ThomasBurleson
Copy link
Contributor

  • insure we clone properly the a configuration map object.
  • implement unit test to verify.

@ThomasBurleson ThomasBurleson added the pr: merge ready This PR is ready for a caretaker to review label Aug 23, 2016
@ThomasBurleson
Copy link
Contributor Author

@jelbourn - ping

@ThomasBurleson ThomasBurleson force-pushed the wip/fix-themingProvider branch from 1787cf2 to 8758424 Compare August 23, 2016 21:24
return angular.extend( { }, config, locals );
var config = angular.extend( { }, themeConfig, locals );

config.registeredStyles = [].concat(config.registeredStyles);
Copy link
Member

Choose a reason for hiding this comment

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

Indention?

@jelbourn
Copy link
Member

LGTM aside from that one indentation thing

@ThomasBurleson ThomasBurleson force-pushed the wip/fix-themingProvider branch from 8758424 to f6d5c6b Compare August 23, 2016 22:56
@ThomasBurleson
Copy link
Contributor Author

@jelbourn - fixed.

@jelbourn jelbourn merged commit b328882 into master Aug 23, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
pr: merge ready This PR is ready for a caretaker to review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants