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

Conversation

devversion
Copy link
Member

  • Add a public method, which allows developers to register their own theme styles for the app theming
  • Adds descriptions and examples to the service documentation.

Credits to @Emeegeemee for implementing the registerStyles functionality.

Closes #7708. Closes #7864.

@devversion devversion added the needs: review This PR is waiting on review from the team label Jun 3, 2016
inject(function($MD_THEME_CSS) { expect($MD_THEME_CSS).toBe('/**/'); });

// Find the string '/**//*test*/' in the head tag.
expect(document.head.innerHTML).toMatch(/\/\*\*\/\/\*test\*\//);
Copy link
Member

@crisbeto crisbeto Jun 3, 2016

Choose a reason for hiding this comment

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

This might be a little more readable with indexOf/toContain.

Copy link
Member Author

Choose a reason for hiding this comment

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

Agreed, this was directly ported from #7864

* Add a public method, which allows developers to register their own theme styles for the app theming
* Adds descriptions and examples to the service documentation.

Credits to @Emeegeemee for implementing the `registerStyles` functionality.

Closes angular#7708. Closes angular#7864.
@ThomasBurleson ThomasBurleson added the pr: merge ready This PR is ready for a caretaker to review label Jun 3, 2016
@devversion devversion deleted the submit/pr-7864 branch June 3, 2016 22:28
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
needs: review This PR is waiting on review from the team 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