From 8dcf893b890fdd9d48caa1c687037bcbd2cb3f41 Mon Sep 17 00:00:00 2001
From: Jordan Bibla
Date: Fri, 10 Aug 2018 12:15:17 -0400
Subject: [PATCH 1/6] cleaned up modal styling
---
.../renderer/components/common/TmModal.vue | 49 ++++++--------
.../components/common/TmModalHelp.vue | 15 +----
.../components/common/TmModalReceive.vue | 12 ----
.../wallet/TmModalSendConfirmation.vue | 65 +++----------------
4 files changed, 31 insertions(+), 110 deletions(-)
diff --git a/app/src/renderer/components/common/TmModal.vue b/app/src/renderer/components/common/TmModal.vue
index a8d916ae59..176857b018 100644
--- a/app/src/renderer/components/common/TmModal.vue
+++ b/app/src/renderer/components/common/TmModal.vue
@@ -47,45 +47,31 @@ export default {
align-items center
backdrop-filter blur(0.5em)
+ b
+ font-weight 500
+ color var(--bright)
+
&.tm-modal-fullscreen
display flex
- .tm-modal-container
- flex 1
- height 100vh
- max-height 100vh
- max-width 40rem + 6rem
- display flex
- flex-flow column nowrap
-
- .tm-modal-main
- flex 1
- overflow-y scroll
- display block
-
.tm-modal-container
background var(--app-fg)
box-shadow hsla(0,0,0,0.25) 0 0.25rem 1rem
-
display flex
flex-flow column nowrap
-
min-width 20rem
- min-height 20rem
-
max-width 30rem
- max-height 40rem
.tm-modal-header
display flex
flex-flow row nowrap
align-items center
flex 0 0 3rem + 0.0625rem
+ padding 2rem 1rem
background var(--app-nav)
.tm-modal-icon
height 3rem
- flex 0 0 3rem
display flex
align-items center
justify-content center
@@ -95,9 +81,9 @@ export default {
&.tm-modal-close
cursor pointer
i
- color var(--link)
+ color var(--txt)
&:hover i
- color var(--hover)
+ color var(--link)
.tm-modal-icon + .tm-modal-title
padding-left 0
@@ -105,11 +91,12 @@ export default {
.tm-modal-title
flex 1
font-size h3
- color var(--txt)
+ font-weight 500
+ color var(--bright)
padding 0 1rem
-.tm-modal-main, .tm-modal-footer
- padding 1rem
+.tm-modal-main
+ padding 2rem 1rem
.tm-modal-main + .tm-modal-footer
border-top px solid var(--bc)
@@ -125,16 +112,20 @@ export default {
.tm-modal-main p
margin-bottom 1rem
-
-.tm-modal-main > .tm-article-body
- margin -1rem
+ padding 0 1rem
+ word-wrap break-word
+ color var(--txt)
.tm-modal-footer
- flex 0 0 4rem + px
+ padding 1rem
+
+ button
+ padding-left 1rem
+
&:empty
display none
.tm-modal-footer > div
display flex
- justify-content space-between
+ justify-content flex-end
diff --git a/app/src/renderer/components/common/TmModalHelp.vue b/app/src/renderer/components/common/TmModalHelp.vue
index d5c1e2b09c..e9caba07a6 100644
--- a/app/src/renderer/components/common/TmModalHelp.vue
+++ b/app/src/renderer/components/common/TmModalHelp.vue
@@ -1,13 +1,8 @@
modal.tm-modal-help(v-if="active" :close="close")
div(slot='title') Need Help?
- p Something in the app not working correctly? Tell us what went wrong in developer chat or create an issue on our GitHub. Thanks for improving Cosmos Voyager!
- tm-list-item(
- title="Developer Chat"
- href="https://riot.im/app/#/room/#cosmos:matrix.org")
- tm-list-item(
- title="GitHub Issues"
- href="https://github.com/cosmos/voyager/issues")
+ p Something in the app not working correctly? Tell us what went wrong in our developer chat or create an issue on GitHub.
+ p Thanks for improving Cosmos Voyager!
-
-
From 43af448ba2f63d11f000d3400e7f33f583c58de1 Mon Sep 17 00:00:00 2001
From: Jordan Bibla
Date: Fri, 10 Aug 2018 12:42:47 -0400
Subject: [PATCH 2/6] deleted some unused css, button stays in its lane
---
app/src/renderer/components/common/TmPart.vue | 50 -------------------
.../renderer/components/wallet/PageSend.vue | 11 +++-
app/src/renderer/styles/app.styl | 3 --
3 files changed, 10 insertions(+), 54 deletions(-)
diff --git a/app/src/renderer/components/common/TmPart.vue b/app/src/renderer/components/common/TmPart.vue
index 7358509a27..a31ed974c9 100644
--- a/app/src/renderer/components/common/TmPart.vue
+++ b/app/src/renderer/components/common/TmPart.vue
@@ -17,53 +17,3 @@ export default {
props: ["title"]
}
-
-
diff --git a/app/src/renderer/components/wallet/PageSend.vue b/app/src/renderer/components/wallet/PageSend.vue
index 9aaa0add68..4012305148 100644
--- a/app/src/renderer/components/wallet/PageSend.vue
+++ b/app/src/renderer/components/wallet/PageSend.vue
@@ -50,7 +50,6 @@ tm-page(title='Send')
br(v-if='mockedConnector')
div(slot='footer')
- div
tm-btn(v-if='sending' value='Sending...' disabled color="primary")
tm-btn(v-else @click='onSubmit' value="Send Tokens" color="primary")
@@ -214,3 +213,13 @@ export default {
}
}
+
+
diff --git a/app/src/renderer/styles/app.styl b/app/src/renderer/styles/app.styl
index e86b0fb207..328f9ff4da 100644
--- a/app/src/renderer/styles/app.styl
+++ b/app/src/renderer/styles/app.styl
@@ -81,9 +81,6 @@ input.tm-field
height 100% !important
border-left 1px solid var(--bc, #ddd);
-.tm-part-header
- border 1px solid var(--bc-dim)
-
.tm-part-main
padding-right 0 !important
From 80a9d9aef4f2d735262ed4039a13434f67b5f28a Mon Sep 17 00:00:00 2001
From: Jordan Bibla
Date: Fri, 10 Aug 2018 12:45:29 -0400
Subject: [PATCH 3/6] update snapshots
---
.../__snapshots__/TmModalHelp.spec.js.snap | 99 +++------------
.../__snapshots__/PageSend.spec.js.snap | 114 +++++-------------
.../TmModalSendConfirmation.spec.js.snap | 107 +++++-----------
3 files changed, 69 insertions(+), 251 deletions(-)
diff --git a/test/unit/specs/components/common/__snapshots__/TmModalHelp.spec.js.snap b/test/unit/specs/components/common/__snapshots__/TmModalHelp.spec.js.snap
index 9c6ecc0985..92aceb62e5 100644
--- a/test/unit/specs/components/common/__snapshots__/TmModalHelp.spec.js.snap
+++ b/test/unit/specs/components/common/__snapshots__/TmModalHelp.spec.js.snap
@@ -32,92 +32,23 @@ exports[`TmModalHelp has the expected html structure 1`] = `
class="tm-modal-main"
>
- Something in the app not working correctly? Tell us what went wrong in developer chat or create an issue on our GitHub. Thanks for improving Cosmos Voyager!
-
-
-
-
-
-
-
-
- Developer Chat
-
-
-
-
-
-
-
- chevron_right
-
-
- my_location
-
-
-
-
-
-
-
+ GitHub
+
+ .
+
+
+ Thanks for improving Cosmos Voyager!
+