Skip to content

Commit

Permalink
fix(popover): allow popover to have an ion-content wrapping it
Browse files Browse the repository at this point in the history
  • Loading branch information
brandyscarney committed Jun 3, 2016
1 parent f8d1103 commit c801d18
Show file tree
Hide file tree
Showing 2 changed files with 61 additions and 60 deletions.
5 changes: 1 addition & 4 deletions src/components/popover/popover.scss
Original file line number Diff line number Diff line change
Expand Up @@ -36,10 +36,7 @@ ion-popover {

min-height: 200px;

ion-page {
scroll-content {
position: relative;
display: flex;

height: auto;
}
}
116 changes: 60 additions & 56 deletions src/components/popover/test/basic/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -4,58 +4,60 @@ import {ionicBootstrap, Popover, NavController, Content, NavParams, ViewControll

@Component({
template: `
<ion-list radio-group [(ngModel)]="fontFamily" (ionChange)="changeFontFamily()">
<ion-row>
<ion-col>
<button (click)="changeFontSize('smaller')" ion-item detail-none class="text-button text-smaller">A</button>
</ion-col>
<ion-col>
<button (click)="changeFontSize('larger')" ion-item detail-none class="text-button text-larger">A</button>
</ion-col>
</ion-row>
<ion-row class="row-dots">
<ion-col>
<button (click)="changeBackground('white')" category="dot" class="dot-white" [class.selected]="background == 'white'"></button>
</ion-col>
<ion-col>
<button (click)="changeBackground('tan')" category="dot" class="dot-tan" [class.selected]="background == 'tan'"></button>
</ion-col>
<ion-col>
<button (click)="changeBackground('grey')" category="dot" class="dot-grey" [class.selected]="background == 'grey'"></button>
</ion-col>
<ion-col>
<button (click)="changeBackground('black')" category="dot" class="dot-black" [class.selected]="background == 'black'"></button>
</ion-col>
</ion-row>
<ion-item class="text-athelas">
<ion-label>Athelas</ion-label>
<ion-radio value="Athelas"></ion-radio>
</ion-item>
<ion-item class="text-charter">
<ion-label>Charter</ion-label>
<ion-radio value="Charter"></ion-radio>
</ion-item>
<ion-item class="text-iowan">
<ion-label>Iowan</ion-label>
<ion-radio value="Iowan"></ion-radio>
</ion-item>
<ion-item class="text-palatino">
<ion-label>Palatino</ion-label>
<ion-radio value="Palatino"></ion-radio>
</ion-item>
<ion-item class="text-san-francisco">
<ion-label>San Francisco</ion-label>
<ion-radio value="San Francisco"></ion-radio>
</ion-item>
<ion-item class="text-seravek">
<ion-label>Seravek</ion-label>
<ion-radio value="Seravek"></ion-radio>
</ion-item>
<ion-item class="text-times-new-roman">
<ion-label>Times New Roman</ion-label>
<ion-radio value="Times New Roman"></ion-radio>
</ion-item>
</ion-list>
<ion-content>
<ion-list radio-group [(ngModel)]="fontFamily" (ionChange)="changeFontFamily()">
<ion-row>
<ion-col>
<button (click)="changeFontSize('smaller')" ion-item detail-none class="text-button text-smaller">A</button>
</ion-col>
<ion-col>
<button (click)="changeFontSize('larger')" ion-item detail-none class="text-button text-larger">A</button>
</ion-col>
</ion-row>
<ion-row class="row-dots">
<ion-col>
<button (click)="changeBackground('white')" category="dot" class="dot-white" [class.selected]="background == 'white'"></button>
</ion-col>
<ion-col>
<button (click)="changeBackground('tan')" category="dot" class="dot-tan" [class.selected]="background == 'tan'"></button>
</ion-col>
<ion-col>
<button (click)="changeBackground('grey')" category="dot" class="dot-grey" [class.selected]="background == 'grey'"></button>
</ion-col>
<ion-col>
<button (click)="changeBackground('black')" category="dot" class="dot-black" [class.selected]="background == 'black'"></button>
</ion-col>
</ion-row>
<ion-item class="text-athelas">
<ion-label>Athelas</ion-label>
<ion-radio value="Athelas"></ion-radio>
</ion-item>
<ion-item class="text-charter">
<ion-label>Charter</ion-label>
<ion-radio value="Charter"></ion-radio>
</ion-item>
<ion-item class="text-iowan">
<ion-label>Iowan</ion-label>
<ion-radio value="Iowan"></ion-radio>
</ion-item>
<ion-item class="text-palatino">
<ion-label>Palatino</ion-label>
<ion-radio value="Palatino"></ion-radio>
</ion-item>
<ion-item class="text-san-francisco">
<ion-label>San Francisco</ion-label>
<ion-radio value="San Francisco"></ion-radio>
</ion-item>
<ion-item class="text-seravek">
<ion-label>Seravek</ion-label>
<ion-radio value="Seravek"></ion-radio>
</ion-item>
<ion-item class="text-times-new-roman">
<ion-label>Times New Roman</ion-label>
<ion-radio value="Times New Roman"></ion-radio>
</ion-item>
</ion-list>
</ion-content>
`,
})
class PopoverRadioPage {
Expand Down Expand Up @@ -155,10 +157,12 @@ class PopoverListPage {

@Component({
template: `
<ion-list>
<ion-list-header>Ionic</ion-list-header>
<button ion-item *ngFor="let item of items">Item {{item}}</button>
</ion-list>
<ion-content>
<ion-list>
<ion-list-header>Ionic</ion-list-header>
<button ion-item *ngFor="let item of items">Item {{item}}</button>
</ion-list>
</ion-content>
`
})
class PopoverLongListPage {
Expand Down

0 comments on commit c801d18

Please sign in to comment.