Skip to content

Commit

Permalink
Signpost Advanced settings info better
Browse files Browse the repository at this point in the history
  • Loading branch information
NoelLH committed May 10, 2020
1 parent 3d637d3 commit f5740ed
Show file tree
Hide file tree
Showing 2 changed files with 12 additions and 3 deletions.
6 changes: 3 additions & 3 deletions src/app/settings/settings.page.html
Original file line number Diff line number Diff line change
Expand Up @@ -150,9 +150,9 @@
</ion-button>
<p class="help">
<ion-icon name="information-circle-outline"></ion-icon>
Advanced mode enables multiple profiles and some extra output settings.
You can't change back to the simple mode if you switch.
<a (click)="openHelp()" target="_system">Learn more</a>
Advanced mode enables multiple profiles and extra output settings.
You can't return to the simpler mode if you switch.
<a (click)="openAdvancedInfo()" target="_system">Learn more</a>
</p>
</div>
</form>
Expand Down
9 changes: 9 additions & 0 deletions src/app/settings/settings.page.ts
Original file line number Diff line number Diff line change
Expand Up @@ -103,6 +103,11 @@ export class SettingsPage {
icon: 'cog',
handler: () => this.addFirstProfile(),
}, {
text: 'Learn more first',
icon: 'help',
handler: () => this.openAdvancedInfo(),
}
, {
text: 'Cancel',
icon: 'close',
role: 'cancel',
Expand Down Expand Up @@ -150,6 +155,10 @@ export class SettingsPage {
this.iab.create('https://passwordmaker.webful.uk/#settings', '_system');
}

openAdvancedInfo() {
this.iab.create('https://passwordmaker.webful.uk/#advanced', '_system');
}

private update() {
this.settingsService.getCurrentSettings()
.then(settings => {
Expand Down

0 comments on commit f5740ed

Please sign in to comment.