Skip to content

Commit

Permalink
Jordan/show on ledger (#3401)
Browse files Browse the repository at this point in the history
* fixed inconsistent styling issue with "show on ledger" button

* only show the button when signed in with a ledger and not on mobile

* changelog
  • Loading branch information
jbibla authored and mariopino committed Jan 13, 2020
1 parent c7a7420 commit b03c7de
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 4 deletions.
1 change: 1 addition & 0 deletions changes/jordan_show-on-ledger
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
[Fixed] there was a styling issue with the "show on ledger" button and it was being displayed for all signed in sessions, not just ledger sessions @jbibla
10 changes: 6 additions & 4 deletions src/components/common/AppMenu.vue
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
<template>
<menu class="app-menu">
<div v-if="session.signedIn" class="user-box">
<div>
<div class="user-box-address">
<div>
<h3>Your Address</h3>
<Bech32 :address="address || ''" />
Expand All @@ -11,7 +11,7 @@
</a>
</div>
<a
v-if="session.signedIn"
v-if="!session.isMobile && session.sessionType === 'ledger'"
class="show-on-ledger"
@click="showAddressOnLedger()"
>
Expand Down Expand Up @@ -263,12 +263,14 @@ export default {
padding: 0.5rem 0.75rem;
border: 2px solid var(--bc);
border-radius: 0.25rem;
display: block;
}
.user-box > div {
.user-box-address {
width: 100%;
display: flex;
justify-content: space-between;
align-items: center;
justify-content: space-between;
}
.user-box i {
Expand Down

0 comments on commit b03c7de

Please sign in to comment.