Commit 307df129331fb2304b165db3702a5b8f05c84031
fix: add profile edit callback to update profile, clear selections
mix irving committed on 1/29/2017, 1:13:09 AMParent: e7d5a35db1a7f8894ba72038b284d13234c5d9c5
Files changed
| modules_basic/avatar/edit.js | changed |
| modules_basic/message/confirm.js | changed |
| modules_basic/avatar/edit.js | |||
|---|---|---|---|
| @@ -75,8 +75,9 @@ | |||
| 75 | 75 … | pull.drain(image => images.push(image) ) | |
| 76 | 76 … | ) | |
| 77 | 77 … | ||
| 78 | 78 … | var namesRecord = MutantObject() | |
| 79 … | + // TODO constrain query to one name per peer? | ||
| 79 | 80 … | pull( | |
| 80 | 81 … | api.sbot_links({dest: id, rel: 'about', values: true}), | |
| 81 | 82 … | pull.map(e => e.value.content.name), | |
| 82 | 83 … | pull.filter(Boolean), | |
| @@ -88,10 +89,10 @@ | |||
| 88 | 89 … | var names = dictToCollection(namesRecord) | |
| 89 | 90 … | ||
| 90 | 91 … | var lb = hyperlightbox() | |
| 91 | 92 … | ||
| 92 | - | ||
| 93 | - var description = '' //TODO load this in, make this editable | ||
| 93 … | + // TODO load this in, make this editable | ||
| 94 … | + var description = '' | ||
| 94 | 95 … | ||
| 95 | 96 … | var isPossibleUpdate = computed([name.new, avatar.new], (name, avatar) => { | |
| 96 | 97 … | return name || avatar.link | |
| 97 | 98 … | }) | |
| @@ -140,9 +141,9 @@ | |||
| 140 | 141 … | }) | |
| 141 | 142 … | ]) | |
| 142 | 143 … | ]), | |
| 143 | 144 … | when(isPossibleUpdate, h('section.action', [ | |
| 144 | - h('button.cancel', { 'ev-click': handleCancelClick }, 'cancel'), | ||
| 145 … | + h('button.cancel', { 'ev-click': clearNewSelections }, 'cancel'), | ||
| 145 | 146 … | h('button.confirm', { 'ev-click': handleUpdateClick }, 'confirm changes') | |
| 146 | 147 … | ])) | |
| 147 | 148 … | ]) | |
| 148 | 149 … | ]) | |
| @@ -172,9 +173,9 @@ | |||
| 172 | 173 … | }) | |
| 173 | 174 … | lb.show(el) | |
| 174 | 175 … | } | |
| 175 | 176 … | ||
| 176 | - function handleCancelClick () { | ||
| 177 … | + function clearNewSelections () { | ||
| 177 | 178 … | name.new.set(null) | |
| 178 | 179 … | avatar.new.set({}) | |
| 179 | 180 … | } | |
| 180 | 181 … | ||
| @@ -189,9 +190,18 @@ | |||
| 189 | 190 … | ||
| 190 | 191 … | if (newName) msg.name = newName | |
| 191 | 192 … | if (newAvatar.link) msg.image = newAvatar | |
| 192 | 193 … | ||
| 193 | - api.message_confirm(msg) | ||
| 194 … | + api.message_confirm(msg, (err, data) => { | ||
| 195 … | + if (err) return console.error(err) | ||
| 196 … | + | ||
| 197 … | + if (newName) name.original.set(newName) | ||
| 198 … | + if (newAvatar) avatar.original.set(api.blob_url(newAvatar.link)) | ||
| 199 … | + | ||
| 200 … | + clearNewSelections() | ||
| 201 … | + | ||
| 202 … | + // TODO - update aliases displayed | ||
| 203 … | + }) | ||
| 194 | 204 … | } | |
| 195 | 205 … | } | |
| 196 | 206 … | ||
| 197 | 207 … | } | |
| modules_basic/message/confirm.js | ||
|---|---|---|
| @@ -39,22 +39,22 @@ | ||
| 39 | 39 … | content: content |
| 40 | 40 … | } |
| 41 | 41 … | } |
| 42 | 42 … | |
| 43 | - var okay = h('button', { | |
| 43 … | + var okay = h('button.okay', { | |
| 44 | 44 … | 'ev-click': () => { |
| 45 | 45 … | lb.remove() |
| 46 | 46 … | api.publish(content, cb) |
| 47 | 47 … | }}, |
| 48 | 48 … | 'okay' |
| 49 | 49 … | ) |
| 50 | 50 … | |
| 51 | - var cancel = h('button', { | |
| 51 … | + var cancel = h('button.cancel', { | |
| 52 | 52 … | 'ev-click': () => { |
| 53 | 53 … | lb.remove() |
| 54 | 54 … | cb(null) |
| 55 | 55 … | }}, |
| 56 | - 'Cancel' | |
| 56 … | + 'cancel' | |
| 57 | 57 … | ) |
| 58 | 58 … | |
| 59 | 59 … | okay.addEventListener('keydown', function (ev) { |
| 60 | 60 … | if(ev.keyCode === 27) cancel.click() //escape |
| @@ -62,9 +62,9 @@ | ||
| 62 | 62 … | |
| 63 | 63 … | lb.show(h('MessageConfirm', [ |
| 64 | 64 … | h('header -preview_description', h('h1', 'Preview')), |
| 65 | 65 … | h('section -message_preview', api.message_render(msg)), |
| 66 | - h('section -actions', [okay, cancel]) | |
| 66 … | + h('section -actions', [cancel, okay]) | |
| 67 | 67 … | ] |
| 68 | 68 … | )) |
| 69 | 69 … | |
| 70 | 70 … | okay.focus() |
Built with git-ssb-web