Skip to content

Commit

Permalink
fix: (platform) Fixed detached menu while scrolling (#3195)
Browse files Browse the repository at this point in the history
  • Loading branch information
KevinOkamoto committed Sep 9, 2020
1 parent c2321a8 commit 049b6a3
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 9 deletions.
Original file line number Diff line number Diff line change
@@ -1,6 +1,5 @@
<fd-docs-section-title [id]="'def'" [componentName]="'menu'">Menu</fd-docs-section-title>
<description
>The Menu can be invoked from any element, by using the <code>fdpMenuTriggerFor</code> directive.
<description>The Menu can be invoked from any element, by using the <code>fdpMenuTriggerFor</code> directive.
</description>
<component-example>
<fdp-platform-menu-basic-example></fdp-platform-menu-basic-example>
Expand All @@ -10,10 +9,9 @@
<separator></separator>

<fd-docs-section-title [id]="'def'" [componentName]="'menu'">Menu with Horizontal Positioning</fd-docs-section-title>
<description
>The horizontal position of the menu can be controlled via the <code>xPosition</code> property, which can be set to
either 'after' or 'before'. The default value is 'after'.</description
>
<description>The horizontal position of the menu can be controlled via the <code>xPosition</code> property, which can be
set to
either 'after' or 'before'. The default value is 'after'.</description>
<component-example>
<fdp-platform-menu-x-position-example></fdp-platform-menu-x-position-example>
</component-example>
Expand All @@ -39,7 +37,8 @@
</p>
<p>
For containers that are scrollable (e.g. via the CSS <code>overflow: scroll</code> property), the user needs to
add the <code>cdk-scrollable</code> (or <code>cdkScrollable</code>) directive to that container.
add the <code>cdk-scrollable</code> (or <code>cdkScrollable</code>) directive to that container, and import the
CDK ScrollingModule.
</p>
</description>
<component-example>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,7 @@ import { PlatformMenuScrollingExampleComponent } from './platform-menu-examples/
import { PlatformMenuXPositionExampleComponent } from './platform-menu-examples/platform-menu-x-position-example.component';
import { PlatformMenuModule, PlatformButtonModule, } from '@fundamental-ngx/platform';
import { AvatarModule } from '@fundamental-ngx/core';
import { ScrollingModule } from '@angular/cdk/scrolling';


const routes: Routes = [
Expand All @@ -31,7 +32,8 @@ const routes: Routes = [
SharedDocumentationPageModule,
PlatformMenuModule,
PlatformButtonModule,
AvatarModule
AvatarModule,
ScrollingModule
],
exports: [RouterModule],
declarations: [
Expand All @@ -43,4 +45,4 @@ const routes: Routes = [
PlatformMenuXPositionExampleComponent
]
})
export class PlatformMenuDocsModule {}
export class PlatformMenuDocsModule { }

0 comments on commit 049b6a3

Please sign in to comment.