Skip to content
This repository was archived by the owner on Jan 6, 2025. It is now read-only.

GENERAL NOTICE: changes to ObservableMedia #885

Closed
CaerusKaru opened this issue Nov 11, 2018 · 4 comments · Fixed by #892
Closed

GENERAL NOTICE: changes to ObservableMedia #885

CaerusKaru opened this issue Nov 11, 2018 · 4 comments · Fixed by #892
Labels
discussion Further discussion with the team is needed before proceeding

Comments

@CaerusKaru
Copy link
Member

Due to the impending changes to RxJS in v7, there will be some changes made to ObservableMedia. This is because in the next version, RxJS no longer allows extending of Observable (since it's being changed to a function instead of a class).

One option we will explore is simply migrating to the CDK's BreakpointObserver. Another will be wrapping the observable in another class (possibly still called ObservableMedia just with a different way of accessing the Observable).

This notice is as much open for discussion as it is for getting the news out there. RxJS v7 is still a ways away, but the earlier we plan and explore options, the better the user experience for everyone.

@CaerusKaru CaerusKaru added the discussion Further discussion with the team is needed before proceeding label Nov 11, 2018
@mackelito
Copy link

I really like the CDK BreakpointObserver.. except that id does not contain "Window" Breakpoint Range from https://material.io/design/layout/responsive-layout-grid.html#breakpoints

It that could be added it would be super awesome! :)

@adamdport
Copy link

@CaerusKaru I can't find any info on RxJS v7. Can you link us to the change you're talking about?

@CaerusKaru
Copy link
Member Author

See here for the design doc

ThomasBurleson pushed a commit that referenced this issue Nov 30, 2018
BREAKING CHANGE:

`ObservableMedia` is now deprecated in anticipation of RxJS v7.
The new API is called **`MediaObserver`**, and provides the exact same functionality as ObservableMedia, except you cannot directly subscribe to it,
You can subscribe to MediaObserver's `media$` property; in place of subscribing directly to ObservableMedia.

Fixes #885.
ThomasBurleson pushed a commit that referenced this issue Nov 30, 2018
BREAKING CHANGE:

`ObservableMedia` is now deprecated in anticipation of RxJS v7.
The new API is called **`MediaObserver`**, and provides the exact same functionality as ObservableMedia, except you cannot directly subscribe to it,
You can subscribe to MediaObserver's `media$` property; in place of subscribing directly to ObservableMedia.

Fixes #885.
ThomasBurleson pushed a commit that referenced this issue Nov 30, 2018
BREAKING CHANGE:

`ObservableMedia` is now deprecated in anticipation of RxJS v7.
The new API is called **`MediaObserver`**, and provides the exact same functionality as ObservableMedia, except you cannot directly subscribe to it,
You can subscribe to MediaObserver's `media$` property; in place of subscribing directly to ObservableMedia.

Fixes #885.
liwde added a commit to Aam-Digital/ndb-core that referenced this issue Jan 14, 2019
liwde pushed a commit to Aam-Digital/ndb-core that referenced this issue Jan 17, 2019
* updated the deprecated HttpModule to HttpClientModule

* removed deprecated RxJS 6 features and rxjs-compat

* updated dependencies (doesn't compile; some dependency is using typescript 3.2.2)

* further dependency updates and fixes for breaking changes

see #187

* remove "typings" dependency. unneeded as we are using @types

* fixed lint errors

* Update some more dependencies and fix breaking change in flex-layout

There was an undocumented breaking change. See:
angular/flex-layout#885
angular/flex-layout@1205588#diff-9caf78031951b821123260bf7bc8e031
@angular-automatic-lock-bot
Copy link

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Sep 5, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
discussion Further discussion with the team is needed before proceeding
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants