Follow-up commit for 4879dc4720.
---
Disables concatenation of multiple leading comments for a TypeScript node. Since all shipped source files have a license banner at top, the license banner comment would be incorrectly considered as "comment" for the first TypeScript node of a given file. Since there are various files in the Material project where the first node of a source file is exported and should only use the first leading comment, we need to disable comment concatenation.
See for example: https://github.com/angular/material2/blob/master/src/cdk/coercion/boolean-property.ts
* Patch releases should be published from patch publish branches with the format: `{}.{}.x`
* Minor releases should be published from minor publish branches with the format: `{}.x`
* Major releases should be published from the `master` branch.
Closes#12655
* Similarly as the TSLint rule, we should only warn (on `gulp docs`) if there is a `@deprecated` tag without a `@breaking-change` tag. If there is a `@breaking-change` tag, but no `@deprecated` tag, it's valid because a breaking change does not always involve a deprecation (e.g. "to be made required").
* Updates the log patching for the Dgeni package to also hide warnings for mixins that don't end with `MixinBase`.
Removes type checking from the `switchIdentifiersRule` because we don't want to depend on types for older versions when running the schematics. The rule still maintains the explicit checks that ensure that only identifiers from Material or the CDK are being updated,
* Switches back from `dart-sass` to `node-sass` due to a pretty huge slow-down we noticed (can be revisited)
* No longer copies all SCSS files to the temporary `package` output just in favor of the demo-app (slows down unnecessarily). This now allows us to live-reload theme changes within a second.
* No longer rebuilds all package SCSS files twice for the ES5 output (rel: https://github.com/angular/material2/pull/7403)
As discussed, renames the `@deletion-target` tag to `@breaking-change` to make the intentions of the tag clearer. Also adds an extra check to the tslint rule to catch any leftover `@deletion-target` tags.
* Switches to dart-sass in order to be consistent with the Bazel output. That way we can identify potential issues with the SASS output (e.g. demo-app) before we at some point make the full-switch.
* Also speeds-up the project installs because libsass bindings are no longer needed (trade-off with compilation time though; but pretty minimal due to our amount of Sass files)
* Shows method overloads in the documentation instead of showing the overload-base method that has most likely no `JSDoc` nor good-explaining parameter names (parameters have multiple purposes and types)
* Depending on the timing for building the assets and building the ESM output (kind of randomly), building the Material package will result in NGC failures. This happens when running the `material:build-no-bundles` task (e.g. unit testing; serving the dev-app) because the assets are built in parallel with the ESM output.
* Removes a duplicated function in the package-tools.
This is necessary so that, when generating ngInjectableDefs, ngc can
correctly determine the import location for injected dependencies.
Without this, the generated imports use a relative path to dist instead
of e.g. "@angular/cdk/overlay". In real life it's not a problem because
the packages are resolved to node_modules.
Between rxjs 5.5 and 6.0 the global name changed from `Rx` to `rxjs`, however ours is still set to `Rx`. These changes switch to using the correct name.
Fixes#11299.
* feat(virtual-scroll): fixed size virtual scroll (#9316)
* feat(virtual-scroll): fixed size virtual scroll
* address some comments
* change VirtualScrollStrategy interface a bit
* address some more comments
* fix lint & build
* chore: move virtual-scroll to cdk-experimental (#9974)
* virtual-scroll: simplify scroll listener logic (#10102)
* virtual-scroll: only move views that need to be moved (#10099)
* virtual-scroll: only move views that need to be moved
* address comments
* virtual-scroll: switch `throttleTime` to `sampleTime` (#10179)
* virtual-scroll: switch throttleTime to sampleTime
* add comment
* virtual-scroll: allow user to pass `Observable<T[]>` (#10158)
* virtual-scroll: rename `Range` to `ListRange` to avoid confusion with native `Range` (#10220)
* virtual-scroll: add autosize scroll strategy (#10219)
* rename fixed size virtual scroll directive
* add autosize virtual scroll strategy
* add item size estimator class
* add logic for jumping rendered content based on scroll position
* address comments
* virtual-scroll: add `onContentRendered` hook to `VirtualScrollStrategy` (#10290)
* virtual-scroll: add `onContentRendered` hook to `VirtualScrollStrategy`
* address comemnts
* virtual-scroll: add incremental scroll logic in `AutosizeVirtualScrollStrategy` (#10504)
* virtual-scroll: add incremental scroll logic in `AutosizeVirtualScrollStrategy`.
This still has a couple issues that need to be ironed out and it doesn't
have the code for correcting the error between the predicted and actual
scroll position. (See various TODOs for additional things that need
work).
* fix lint
* address comments
* address comments
* fix bazel
* fix devapp
* fix lint
* cleanup
* virtual-scroll: rewrite offset in terms of "to-top" and fix a bug where items were removed too soon (#10986)
* rewrite offsets to the end of the rendered content as offsets to the
start
* add some more autosize demos for testing
* make sure not to remove too many items
* address comments
* virtual-scroll: address amcdnl's feedback (#10988)
* rewrite offsets to the end of the rendered content as offsets to the
start
* add some more autosize demos for testing
* make sure not to remove too many items
* virtual-scroll: address amcdnl's feedback
* virtual-scroll: fix updating when data changes and add trackBy demos (#11085)
* virtual-scroll: add logic to correct the scroll position as user move… (#11137)
* virtual-scroll: add logic to correct the scroll position as user moves toward the top
* address comments
* fix(scrolling): adds right to fix pushed content (#11192)
* fix(scrolling): adds right to fix pushed content
* chore: address feedback
* chore: address pr feedback
* virtual-scroll: add tests for fixed size items (#11255)
* virtual-scroll: add tests for fixed size items
* address comments
* fix bug in fixed size strategy
* fix bazel build
* virtual-scroll: add tests for `cdkVirtualFor` logic (#11275)
* merge fixed size test components into one
* add tests for cdkVirtualFor logic
* allow undefined to be explicitly passed as trackBy
* fix bazel build
* address comments
* add some basic tests (#11295)
* fix lint
* virtual-scroll: add e2e tests for autosize scroll strategy (#11345)
* set up virtual scroll page in e2e app
* add gulp task for e2e:watch
* add e2e tests for autosize
* address comments
* address comments
* fix lint and tests
* fix bad rebase
* fix(scrolling): scrollbar jump when data change on scroll (#11193)
* fix(scrolling): data change not updating size
* chore: remove todo
* chore: more performant fix for jump
* Bumps the dashboard to the latest Firebase version to fix some TS compilation errors.
* Adds a package-lock for the dashboard for consistency with everything else and to help avoid versioning issues.
* Bumps the screenshot tool to the latest version of Firebase.
Replaces the `selector-nested-pattern-scoped` Stylelint rule with a new rule called `no-ampersand-beyond-selector-start`. The new rule will enforce that ampersands can only be used at the start of selectors in theme mixins. This will catch issues like #10889 and #10904 where having an ampersand anywhere else in the selector will work in most cases, but will break down once the mixin is wrapped in a selector.
* refactor (MatCalendarHeader): remove Host decorator
* fix (remove unused import)
* feature (MatCalendarHeader): add custom header using the standard header to demo
* fix (ts lint)
* test (MatCalendarHeader): add a unit test for custom header input (ongoing)
* refactor (styling)
* refactor (styling)
* test (MatCalendarHeader): pass header component to datepicker
* test (MatCalendarHeader): remove header component to fix test (temporary)
* test (MatCalendarHeader): provide entryComponents
* test (MatCalendarHeader): add custom header component to declarations
* fix (MatCalendarHeader): fix issue with rollup-globals
* test (MatCalendarHeader): get rid of declarations param
* test (MatCalendarHeader): add test to check for standard calendar header selector
* test (MatCalendarHeader): rename test
* test (MatCalendarHeader): type annotations
This includes:
* Adding missing schema json to package
* Add missing packageGroup entries to material, cdk, and moment-adapter
* Move schematics for migration to the migrate-specific schema
* Bump version to 6.0.0-rc.11
This change enables material and cdk packages to take advantage of webpacks more advanced optimizations
documented at https://github.com/webpack/webpack/tree/master/examples/side-effects
Our code is already side-effect free because otherwise @angular-devkit/build-optimizer would break it, so it's safe
to turn this on.
* WIP: migrate migration tool to schematics
* address comments
* fix bazel
* make released version work
* Add additional rules
* wip
* wip
* yay, working
* all rules migrated
* combine packages in single `npm i` call
* update schematics version and fix TODOs
Kristiyan Kostadinov