Enhance meta-review UI

[li-boxuan]

Related to https://github.com/coala/gh-board/issues/84

[li-boxuan]

Regarding enhancement 2 mentioned in https://github.com/coala/gh-board/issues/84#issuecomment-406809509,

this would be handled later in a separate pr (ability to do meta-reviews directly on gh-board)

[li-boxuan]

Preview:

[hemangsk]

1. Some space between the emoji button row and the "Import igitt data" div

   

2. emoji size needs to grow a bit more

   

[li-boxuan]

Preview after update:

[hemangsk]

ack 83fc25d477fefaeba6d1293ccb71bbd21df0e444

[jayvdb]

@gitmate-bot rebase

[gitmate-bot]

Hey! I'm GitMate.io! This pull request is being rebased automatically. Please DO NOT push while rebase is in progress or your changes would be lost permanently :warning:

[gitmate-bot]

Automated rebase with GitMate.io was successful! :tada:

[TravisBuddy]

Travis tests have failed

Hey @li-boxuan, Please read the following log in order to understand the failure reason. It'll be awesome if you fix what's wrong and commit the changes.

Node.js: undefined

npm run test

``` > gh-board@0.0.0 test /home/travis/build/coala/gh-board > ./script/run-test.sh +script/verify-files.sh +files=("issues.json" "recent-issues.json") +for file in '"${files[@]}"' +[[ -f issues.json ]] +echo 'File issues.json exists.' File issues.json exists. +for file in '"${files[@]}"' +[[ -f recent-issues.json ]] +echo 'File recent-issues.json exists.' File recent-issues.json exists. ++npm bin +/home/travis/build/coala/gh-board/node_modules/.bin/jest FAIL test/src/components/reactions.test.js ✕ does not display number zero (32ms) ● does not display number zero expect(value).toMatchSnapshot() Received value does not match stored snapshot "does not display number zero 1". - Snapshot + Received @@ -1,48 +1,48 @@ Array [ 👍 , 👎 , 😄 , 🎉 , 😕 , ❤️ 14 | 15 | ); > 16 | expect(rendered.toJSON()).toMatchSnapshot(); | ^ 17 | }); 18 | at Object. (test/src/components/reactions.test.js:16:29) › 1 snapshot failed. console.error node_modules/fbjs/lib/warning.js:33 Warning: Each child in an array or iterator should have a unique "key" prop. See https://fb.me/react-warning-keys for more information. in Reactions -------------------------|----------|----------|----------|----------|-------------------| File | % Stmts | % Branch | % Funcs | % Lines | Uncovered Line #s | -------------------------|----------|----------|----------|----------|-------------------| All files | 0.07 | 0.73 | 0.35 | 0.08 | | script | 0 | 0 | 0 | 0 | | fetch-issues.js | 0 | 0 | 0 | 0 |... 58,159,162,166 | src | 0 | 0 | 0 | 0 | | bipartite-graph.js | 0 | 0 | 0 | 0 |... 15,17,18,38,41 | card-model.js | 0 | 0 | 0 | 0 |... 41,243,244,247 | database.js | 0 | 0 | 0 | 0 |... 42,344,345,349 | filter-store.js | 0 | 0 | 0 | 0 |... 79,180,182,184 | gantt-chart.js | 0 | 0 | 0 | 0 |... 24,325,326,329 | gfm-dom.js | 0 | 0 | 0 | 0 |... 35,136,137,138 | github-client.js | 0 | 0 | 0 | 0 |... 50,251,253,255 | github-graphql.js | 0 | 0 | 0 | 0 |... 87,593,598,601 | helpers.js | 0 | 0 | 0 | 0 |... 84,85,86,88,89 | history.js | 0 | 100 | 100 | 0 | 1 | index.js | 0 | 100 | 100 | 0 | 1,2,3,4,6,7,9 | initial-data.js | 0 | 0 | 0 | 0 | 1,2,4,5,6,9,10,17 | issue-store.js | 0 | 0 | 0 | 0 |... 93,594,596,598 | new-version-checker.js | 0 | 0 | 0 | 0 |... 32,34,35,36,39 | progress.js | 0 | 0 | 0 | 0 |... 26,29,30,33,34 | route-utils.js | 0 | 0 | 0 | 0 |... 02,403,404,405 | router.jsx | 0 | 100 | 0 | 0 |... 81,90,94,95,96 | settings-store.js | 0 | 0 | 0 | 0 |... 99,100,102,105 | user-store.js | 0 | 0 | 0 | 0 |... 38,40,41,42,47 | src/components | 0.16 | 1.96 | 0.9 | 0.17 | | anonymous-modal.jsx | 0 | 0 | 100 | 0 |... 16,17,18,20,23 | async-button.jsx | 0 | 0 | 100 | 0 |... 77,79,80,82,84 | batch-labels.jsx | 0 | 0 | 0 | 0 |... 22,324,325,328 | board.jsx | 0 | 0 | 0 | 0 |... 35,136,139,142 | burnup.jsx | 0 | 0 | 0 | 0 |... 06,232,239,241 | by-milestone-view.jsx | 0 | 0 | 0 | 0 |... 34,135,139,151 | by-user-view.jsx | 0 | 0 | 0 | 0 |... 08,109,112,124 | chart.jsx | 0 | 0 | 100 | 0 |... 70,71,72,74,76 | colored-icon.jsx | 0 | 0 | 0 | 0 |... 15,17,18,19,22 | dashboard.jsx | 0 | 0 | 0 | 0 |... 65,366,371,380 | diff-envs.jsx | 0 | 0 | 0 | 0 |... 62,63,65,68,71 | etherpad-file.jsx | 0 | 0 | 0 | 0 |... 29,30,31,35,36 | etherpad-issue.jsx | 0 | 0 | 0 | 0 |... 27,28,31,32,36 | etherpad-modal.jsx | 0 | 0 | 0 | 0 |... 48,51,58,73,83 | etherpad.jsx | 0 | 0 | 0 | 0 |... 63,164,165,167 | gantt-view.jsx | 0 | 0 | 0 | 0 |... 20,221,226,228 | gfm.jsx | 0 | 0 | 0 | 0 |... 70,272,276,279 | issue-blurb.jsx | 0 | 0 | 0 | 0 |... 43,144,146,149 | issue-list.jsx | 0 | 0 | 0 | 0 |... 53,154,160,163 | issue.jsx | 0 | 0 | 0 | 0 |... 59,560,562,567 | label-badge.jsx | 0 | 0 | 0 | 0 |... 32,35,36,48,64 | loadable.jsx | 0 | 0 | 0 | 0 |... 83,84,85,86,88 | login-modal.jsx | 0 | 0 | 0 | 0 |... 46,52,59,70,95 | merged-since.jsx | 0 | 0 | 0 | 0 |... 85,186,190,194 | move-modal.jsx | 0 | 0 | 0 | 0 |... 67,169,172,193 | not-found.jsx | 0 | 100 | 0 | 0 | 1,4 | reactions.jsx | 100 | 100 | 100 | 100 | | repo-kanban.jsx | 0 | 0 | 0 | 0 |... 03,204,206,217 | review-blurb.jsx | 0 | 100 | 0 | 0 |... 20,21,22,24,27 | review-list.jsx | 0 | 0 | 100 | 0 |... 56,57,58,64,67 | review.jsx | 0 | 0 | 0 | 0 |... ,78,81,119,120 | time.jsx | 0 | 0 | 0 | 0 |... 73,76,77,78,79 | src/components/app | 0 | 0 | 0 | 0 | | filter-dropdown.jsx | 0 | 0 | 0 | 0 |... 68,369,370,373 | footer.jsx | 0 | 0 | 0 | 0 |... 78,79,80,81,84 | index.jsx | 0 | 0 | 100 | 0 |... 37,42,46,47,49 | nav.jsx | 0 | 0 | 0 | 0 |... 86,187,192,282 | saved-filters.jsx | 0 | 0 | 0 | 0 |... 47,49,50,55,57 | src/lib | 0 | 100 | 0 | 0 | | columns.js | 0 | 100 | 0 | 0 | 1,2,4,9,12,16,17 | -------------------------|----------|----------|----------|----------|-------------------| Snapshot Summary › 1 snapshot failed from 1 test suite. Inspect your code changes or re-run jest with `-u` to update them. Test Suites: 1 failed, 1 total Tests: 1 failed, 1 total Snapshots: 1 failed, 1 total Time: 29.72s Ran all test suites. npm ERR! code ELIFECYCLE npm ERR! errno 1 npm ERR! gh-board@0.0.0 test: `./script/run-test.sh` npm ERR! Exit status 1 npm ERR! npm ERR! Failed at the gh-board@0.0.0 test script. npm ERR! This is probably not a problem with npm. There is likely additional logging output above. npm ERR! A complete log of this run can be found in: npm ERR! /home/travis/.npm/_logs/2018-08-03T03_10_55_210Z-debug.log ```

[jayvdb]

ack https://github.com/coala/gh-board/pull/128/commits/d1a7ea3ea66c7af7a51a9717f67fc635fc35cf41

[jayvdb]

@gitmate-bot ff

[gitmate-bot]

Hey! I'm GitMate.io! This pull request is being fastforwarded automatically. Please DO NOT push while fastforward is in progress or your changes would be lost permanently :warning:

[jayvdb]

@gitmate-bot ff

[gitmate-bot]

Hey! I'm GitMate.io! This pull request is being fastforwarded automatically. Please DO NOT push while fastforward is in progress or your changes would be lost permanently :warning:

[gitmate-bot]

Automated fastforward with GitMate.io was successful! :tada: