[Cashtab] Better hamburger menu
ClosedPublic
Actions

Authored by bytesofman on Tue, Apr 16, 18:45.

Details

Reviewers

emack

Group Reviewers

Restricted Project

Commits

rABC5562be9e1887: [Cashtab] Better hamburger menu

Summary

We still want to migrate to horizontal scrolling nav buttons or mb a sidebar, but we are also still adding screens so it could be awhile.

In the meantime, the hamburger menu could use an update.

Make it bigger, get rid of empty space to the left of and between some buttons, use px instead of rem since we are not really font-size dependent for these dimensions, do not use float which does not work in display flex

Test Plan

npm test, npm start and aesthetic judgments

before

after

Diff Detail

Repository

rABC Bitcoin ABC

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

bytesofman created this revision.Tue, Apr 16, 18:45

Herald added a reviewer: Restricted Project. · View Herald TranscriptTue, Apr 16, 18:45

bytesofman published this revision for review.Tue, Apr 16, 18:46

bytesofman edited the test plan for this revision. (Show Details)

Harbormaster completed remote builds in B28602: Diff 47210.Tue, Apr 16, 18:49

Build Bitcoin ABC Diffs / Diff Testing (cashtab-tests) failed.

Failed tests logs:

====== CashTab Unit Tests: <App /> Navigation menu routes to expected components ======
Error: expect(element).toHaveStyle()

- Expected

- max-height: 100rem;
+ max-height: 100vh;
    at Object.toHaveStyle (/work/cashtab/src/components/App/__tests__/App.test.js:262:54)

Each failure log is accessible here:
CashTab Unit Tests: <App /> Navigation menu routes to expected components