Don't render placeholder markup unless necessary
Rather than always rendering divs for the key and placeholder, render the placeholder only if the key is missing. // FREEBIE
This commit is contained in:
parent
6c7f2cf0fc
commit
05681ce35b
4 changed files with 15 additions and 19 deletions
|
@ -217,12 +217,11 @@
|
|||
</div>
|
||||
<div class='container'>
|
||||
<p> {{theirIdentity}} </p>
|
||||
<div class='key placeholder'>
|
||||
{{ #their_key_unknown }} <span>{{ their_key_unknown }}</span>{{ /their_key_unknown }}
|
||||
</div>
|
||||
{{ ^their_key }}
|
||||
<div class='placeholder'>{{ their_key_unknown }}</div>
|
||||
{{ /their_key }}
|
||||
<div class='key'>
|
||||
{{ #their_key }} <span>{{ . }}</span> {{ /their_key }}
|
||||
|
||||
</div>
|
||||
<p> {{yourIdentity}} </p>
|
||||
<div class='key'>
|
||||
|
|
|
@ -25,18 +25,14 @@
|
|||
return octets;
|
||||
},
|
||||
render_attributes: function() {
|
||||
var attributes = {
|
||||
return {
|
||||
verifyIdentity: i18n('verifyIdentity'),
|
||||
yourIdentity: i18n('yourIdentity'),
|
||||
theirIdentity: i18n('theirIdentity'),
|
||||
their_key_unknown: i18n('theirIdentityUnknown'),
|
||||
your_key: this.splitKey(this.model.your_key),
|
||||
their_key: this.splitKey(this.model.their_key)
|
||||
};
|
||||
if(this.model.their_key) {
|
||||
attributes.their_key = this.splitKey(this.model.their_key);
|
||||
} else {
|
||||
attributes.their_key_unknown = i18n('theirIdentityUnknown');
|
||||
}
|
||||
return attributes;
|
||||
}
|
||||
});
|
||||
})();
|
||||
|
|
|
@ -49,13 +49,14 @@
|
|||
p {
|
||||
padding: 1em;
|
||||
}
|
||||
.key {
|
||||
font-family: monospace;
|
||||
.key, .placeholder {
|
||||
padding: 0 1em;
|
||||
-webkit-user-select: text;
|
||||
}
|
||||
.key.placeholder {
|
||||
font-family: inherit;
|
||||
.key {
|
||||
font-family: monospace;
|
||||
}
|
||||
.placeholder {
|
||||
font-weight: bold;
|
||||
}
|
||||
.container {
|
||||
|
|
|
@ -616,12 +616,12 @@ input.search {
|
|||
background: white; }
|
||||
.key-verification p {
|
||||
padding: 1em; }
|
||||
.key-verification .key {
|
||||
font-family: monospace;
|
||||
.key-verification .key, .key-verification .placeholder {
|
||||
padding: 0 1em;
|
||||
-webkit-user-select: text; }
|
||||
.key-verification .key.placeholder {
|
||||
font-family: inherit;
|
||||
.key-verification .key {
|
||||
font-family: monospace; }
|
||||
.key-verification .placeholder {
|
||||
font-weight: bold; }
|
||||
.key-verification .container {
|
||||
height: calc(100% - (36px + 5px + 1px + 4px)); }
|
||||
|
|
Loading…
Reference in a new issue