Skip to content

Commit e7105c6

Browse files
somombovalorkin
authored andcommitted
fix(sortable): Changed reference to BrowserModule to CommonModule instead (#1503)
* Library Modules should not import BrowserModule anywhere Caused error: `BrowserModule has already been loaded. If you need access to common directives such as NgIf and NgFor from a lazy loaded module, import CommonModule instead.` After trying on lazy loaded app, that had `Ng2BootstrapModule.forRoot()` imported in root App Module. * changed code styling to be consistent with the rest Hopefully it will pass CI tests now!
1 parent 651fcb3 commit e7105c6

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

src/sortable/sortable.module.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,12 @@
11
import { NgModule, ModuleWithProviders } from '@angular/core';
2-
import { BrowserModule } from '@angular/platform-browser';
2+
import { CommonModule } from '@angular/common';
33

44
import { SortableComponent } from './sortable.component';
55
import { DraggableItemService } from './draggable-item.service';
66

77
@NgModule({
88
declarations: [SortableComponent],
9-
imports: [BrowserModule],
9+
imports: [CommonModule],
1010
exports: [SortableComponent]
1111
})
1212
export class SortableModule {

0 commit comments

Comments
 (0)