Skip to content
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

Issues 159 and 160 #161

Merged
merged 3 commits into from
Jul 26, 2019
Merged

Issues 159 and 160 #161

merged 3 commits into from
Jul 26, 2019

Conversation

danielwiehl
Copy link
Collaborator

PR Checklist

Please check if your PR fulfills the following requirements:

  • The commit follows our guidelines
  • Tests for the changes have been added (required for routing or view grid-related issues)
  • Public API has been documented

PR Type

What kind of change does this PR introduce?

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] Documentation content changes
[ ] Other... Please describe:

Issue

Issue Number: #159 #160

Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information

The dropdown sometimes showed even though no view tab was overflowing - this because the adjusted viewport width was not yet flushed to the DOM when checking for overflowing view tabs.

fixes #159
…ng to the observable

The request was immediately posted when invoking 'requestReceive$' method, which is wrong. Instead, it is now posted upon subscription.
It is unlike the 'requestReply' method, which starts resolving the promise immediately.

fixes: #160
ReToCode
ReToCode previously approved these changes Jul 26, 2019
ReToCode
ReToCode previously approved these changes Jul 26, 2019
…ecified via options object

The injection token {USE_NATIVE_RESIZE_OBSERVER} can be provided to control if to use native resize observable by default.
The flag has no effect if not supported by the user agent, or if explicitly specified via options object when creating the resize observable.

fixes: #156
@ReToCode ReToCode merged commit 0594320 into master Jul 26, 2019
@ReToCode ReToCode deleted the issue/159_160 branch July 26, 2019 10:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants