Commit 7be27d2e405a2a962e2b8b3743cb328dca84c297
Factor out getting repo name as a text node
Charles Lehner authored on 9/25/2016, 6:21:51 PMDominic Tarr committed on 10/24/2016, 8:26:19 PM
Parent: 7830521a20a57b285da7357ef05f53b3dd072424
Files changed
| modules/git.js | changed |
| modules/git.js | ||
|---|---|---|
| @@ -76,18 +76,26 @@ | ||
| 76 | 76 … | }) |
| 77 | 77 … | ) |
| 78 | 78 … | } |
| 79 | 79 … | |
| 80 | -function repoLink(id) { | |
| 81 | - var el = h('a', {href: '#'+id}, id.substr(0, 10) + '…') | |
| 80 … | +function repoText(id) { | |
| 81 … | + var text = document.createTextNode(id.substr(0, 10) + '…') | |
| 82 | 82 … | getAvatar({links: sbot_links, get: sbot_get}, self_id, id, |
| 83 | - function (err, avatar) { | |
| 83 … | + function (err, avatar) { | |
| 84 | 84 … | if(err) return console.error(err) |
| 85 | - el.textContent = avatar.name | |
| 85 … | + text.nodeValue = avatar.name | |
| 86 | 86 … | }) |
| 87 | - return el | |
| 87 … | + return text | |
| 88 | 88 … | } |
| 89 | 89 … | |
| 90 … | +function repoLink(id) { | |
| 91 … | + return h('a', {href: '#'+id}, repoText(id)) | |
| 92 … | +} | |
| 93 … | + | |
| 94 … | +function repoName(id) { | |
| 95 … | + return h('ins', repoText(id)) | |
| 96 … | +} | |
| 97 … | + | |
| 90 | 98 … | function getIssueState(id, cb) { |
| 91 | 99 … | pull( |
| 92 | 100 … | sbot_links({dest: id, rel: 'issues', values: true, reverse: true}), |
| 93 | 101 … | pull.map(function (msg) { |
| @@ -131,19 +139,8 @@ | ||
| 131 | 139 … | } |
| 132 | 140 … | return t |
| 133 | 141 … | } |
| 134 | 142 … | |
| 135 | -function repoName(id, link) { | |
| 136 | - var el = link | |
| 137 | - ? h('a', {href: '#'+id}, id.substr(0, 8) + '…') | |
| 138 | - : h('ins', id.substr(0, 8) + '…') | |
| 139 | - getAvatar({links: sbot_links}, self_id, id, function (err, avatar) { | |
| 140 | - if(err) return console.error(err) | |
| 141 | - el.textContent = avatar.name | |
| 142 | - }) | |
| 143 | - return el | |
| 144 | -} | |
| 145 | - | |
| 146 | 143 … | function renderIssueEdit(c) { |
| 147 | 144 … | var id = c.issue || c.link |
| 148 | 145 … | return [ |
| 149 | 146 … | c.title ? h('p', 'renamed issue ', message_link(id), |
| @@ -159,9 +156,9 @@ | ||
| 159 | 156 … | var branchesT, tagsT, openIssuesT, closedIssuesT, openPRsT, closedPRsT |
| 160 | 157 … | var forksT |
| 161 | 158 … | var div = h('div', |
| 162 | 159 … | h('p', 'git repo ', repoName(msg.key)), |
| 163 | - c.upstream ? h('p', 'fork of ', repoName(c.upstream, true)) : '', | |
| 160 … | + c.upstream ? h('p', 'fork of ', repoLink(c.upstream)) : '', | |
| 164 | 161 … | h('p', h('code', 'ssb://' + msg.key)), |
| 165 | 162 … | h('div.git-table-wrapper', {style: {'max-height': '12em'}}, |
| 166 | 163 … | h('table', |
| 167 | 164 … | branchesT = tableRows(h('tr', |
| @@ -247,9 +244,9 @@ | ||
| 247 | 244 … | pull( |
| 248 | 245 … | getForks(msg.key), |
| 249 | 246 … | pull.drain(function (fork) { |
| 250 | 247 … | forksT.append(h('tr', h('td', |
| 251 | - repoName(fork.id, true), | |
| 248 … | + repoLink(fork.id), | |
| 252 | 249 … | ' by ', h('a', {href: '#'+fork.author}, avatar_name(fork.author))))) |
| 253 | 250 … | }, function (err) { |
| 254 | 251 … | if (err) console.error(err) |
| 255 | 252 … | }) |
| @@ -401,9 +398,9 @@ | ||
| 401 | 398 … | pull.once({id: msg.key, author: msg.value.author}), |
| 402 | 399 … | getForks(msg.key) |
| 403 | 400 … | ]), pull.map(function (fork) { |
| 404 | 401 … | return h('option', {value: fork.id}, |
| 405 | - repoName(fork.id), ' by ', avatar_name(fork.author)) | |
| 402 … | + repoLink(fork.id), ' by ', avatar_name(fork.author)) | |
| 406 | 403 … | })) |
| 407 | 404 … | }), |
| 408 | 405 … | ':', |
| 409 | 406 … | headBranchInput, |
Built with git-ssb-web