Skip to content

Commit a48e6b1

Browse files
authored
Merge branch '8.2.x' into amarinov/issue6081_82
2 parents fffe3e8 + fcb3b88 commit a48e6b1

File tree

8 files changed

+29
-19
lines changed

8 files changed

+29
-19
lines changed

projects/igniteui-angular/migrations/update-8_2_6/index.spec.ts

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -67,7 +67,9 @@ describe('Update 8.2.6', () => {
6767
it('should update igx-grid-paginator-theme', done => {
6868
appTree.create(
6969
'/testSrc/appPrefix/component/test.component.scss',
70-
`$dark-grid-paginator: igx-grid-paginator-theme($color: black);
70+
`@import '~igniteui-angular/lib/core/styles/components/grid-paginator/grid-paginator-component';
71+
@import '~igniteui-angular/lib/core/styles/components/grid-paginator/grid-paginator-theme';
72+
$dark-grid-paginator: igx-grid-paginator-theme($color: black);
7173
@include igx-grid-paginator($dark-grid-paginator);
7274
.igx-grid-paginator__pager {
7375
@include igx-button($dark-button);
@@ -77,7 +79,9 @@ describe('Update 8.2.6', () => {
7779
const tree = schematicRunner.runSchematic('migration-12', {}, appTree);
7880
expect(tree.readContent('/testSrc/appPrefix/component/test.component.scss'))
7981
.toEqual(
80-
`$dark-grid-paginator: igx-paginator-theme($color: black);
82+
`@import '~igniteui-angular/lib/core/styles/components/paginator/paginator-component';
83+
@import '~igniteui-angular/lib/core/styles/components/paginator/paginator-theme';
84+
$dark-grid-paginator: igx-paginator-theme($color: black);
8185
@include igx-paginator($dark-grid-paginator);
8286
.igx-grid-paginator__pager {
8387
@include igx-button($dark-button);

projects/igniteui-angular/migrations/update-8_2_6/index.ts

Lines changed: 14 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,11 +29,17 @@ export default function(): Rule {
2929
'$_square-shape-pagination'];
3030

3131
let globalStyleExt: string;
32+
const gridPaginatorComponentImport = '~igniteui-angular/lib/core/styles/components/grid-paginator/grid-paginator-component';
33+
const gridPaginatorThemeImport = '~igniteui-angular/lib/core/styles/components/grid-paginator/grid-paginator-theme';
34+
const paginatorComponentImport = '~igniteui-angular/lib/core/styles/components/paginator/paginator-component';
35+
const paginatorThemeImport = '~igniteui-angular/lib/core/styles/components/paginator/paginator-theme';
3236
const config = getWorkspace(host);
3337
const projects = getProjects(config);
3438

3539
context.logger.info(`Applying migration for Ignite UI for Angular to version ${version}`);
3640

41+
const update = new UpdateChanges(__dirname, host, context);
42+
3743
if (config.schematics && config.schematics['@schematics/angular:component']) {
3844
// updated projects have global prefix rather than per-project:
3945
globalStyleExt = config.schematics['@schematics/angular:component'].styleext;
@@ -56,12 +62,19 @@ export default function(): Rule {
5662
content = content.split(n).join(newThemes[i]);
5763
}
5864
});
65+
if (content.indexOf(gridPaginatorComponentImport) !== -1) {
66+
content = content.replace(gridPaginatorComponentImport, paginatorComponentImport);
67+
host.overwrite(path, content);
68+
}
69+
if (content.indexOf(gridPaginatorThemeImport) !== -1) {
70+
content = content.replace(gridPaginatorThemeImport, paginatorThemeImport);
71+
host.overwrite(path, content);
72+
}
5973
host.overwrite(path, content);
6074
}
6175
});
6276
}
6377

64-
const update = new UpdateChanges(__dirname, host, context);
6578
update.applyChanges();
6679
};
6780
}

projects/igniteui-angular/src/lib/core/styles/components/progress/_progress-component.scss

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -115,7 +115,6 @@
115115

116116
@include e(text, $m: hidden) {
117117
@extend %circular-text !optional;
118-
@extend %circular-text--hidden !optional;
119118
}
120119

121120
@include m(indeterminate) {
@@ -124,9 +123,5 @@
124123
@include e(outer) {
125124
@extend %circular-outer--indeterminate !optional;
126125
}
127-
128-
@include e(text) {
129-
@extend %circular-text--hidden !optional;
130-
}
131126
}
132127
}

projects/igniteui-angular/src/lib/core/styles/components/progress/_progress-theme.scss

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -311,10 +311,6 @@
311311
fill: --var($theme, 'text-color');
312312
}
313313

314-
%circular-text--hidden {
315-
visibility: hidden;
316-
}
317-
318314
@include keyframes('indeterminate-accordion') {
319315
from {
320316
stroke-dashoffset: 578;

projects/igniteui-angular/src/lib/core/styles/components/switch/_switch-theme.scss

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -173,7 +173,7 @@
173173
), map-get($theme, variant));
174174

175175
$switch-off-offset: map-get((
176-
material: 1px,
176+
material: -1px,
177177
fluent: $switch-thumb-width / 3
178178
), map-get($theme, variant));
179179

@@ -275,11 +275,11 @@
275275
transition: $input-transition;
276276

277277
@include if-ltr() {
278-
transform: translateX(-#{rem($switch-off-offset)});
278+
transform: translateX(#{rem($switch-off-offset)});
279279
}
280280

281281
@include if-rtl() {
282-
transform: translateX(#{rem($switch-off-offset)});
282+
transform: translateX(#{rem(-1 * $switch-off-offset)});
283283
}
284284

285285
&:hover {

projects/igniteui-angular/src/lib/directives/for-of/for_of.directive.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -746,6 +746,7 @@ export class IgxForOfDirective<T> implements OnInit, OnChanges, DoCheck, OnDestr
746746
const diffs = [];
747747
let totalDiff = 0;
748748
const l = this._embeddedViews.length;
749+
this._embeddedViews.filter(view => !view.destroyed).forEach(view => view.detectChanges());
749750
const rNodes = this._embeddedViews.map(view => view.rootNodes.find(node => node.nodeType === Node.ELEMENT_NODE));
750751
for (let i = 0; i < l; i++) {
751752
const rNode = rNodes[i];

projects/igniteui-angular/src/lib/progressbar/circularbar.component.spec.ts

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -322,7 +322,8 @@ describe('IgCircularBar', () => {
322322

323323
componentInstance.progressbar.textVisibility = false;
324324
fixture.detectChanges();
325-
expect(progressBarElem.children[2].classList.value).toMatch(CIRCULAR_HIDDEN_TEXT_CLASS);
325+
// Text is not rendered
326+
expect(progressBarElem.children[2]).toBeFalsy();
326327
});
327328

328329
it('When indeterminate mode is on value should not be updated', () => {
@@ -375,8 +376,8 @@ describe('IgCircularBar', () => {
375376

376377
componentInstance.progressbar.textVisibility = false;
377378
fixture.detectChanges();
378-
379-
expect(progressBarElem.children[2].classList.value).toMatch(CIRCULAR_HIDDEN_TEXT_CLASS);
379+
// Text is not rendered
380+
expect(progressBarElem.children[2]).toBeFalsy();
380381
}));
381382

382383
it('The max representation should respond correctly to passed maximum value', fakeAsync(() => {

projects/igniteui-angular/src/lib/progressbar/templates/circular-bar.component.html

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@
77
[attr.aria-valuenow]="value">
88
<circle class="igx-circular-bar__inner" cx="50" cy="50" r="46" />
99
<circle #circle class="igx-circular-bar__outer" cx="50" cy="50" r="46" />
10-
<text [class.igx-circular-bar__text--hidden]="!textVisibility" text-anchor="middle" x="50" y="60">
10+
<text *ngIf="textVisibility" text-anchor="middle" x="50" y="60">
1111
<ng-container *ngTemplateOutlet="textTemplate ? textTemplate.template : defaultTextTemplate; context: context">
1212
</ng-container>
1313
</text>

0 commit comments

Comments
 (0)