search

whole-tale / dashboard

Whole Tale Dashboard
http://wholetale.org
MIT License
7 stars 2 forks source link

Add 'Settings' link to the mobile header/menu #586

Closed bodom0015 closed 4 years ago

bodom0015 commented 4 years ago

Problem

The mobile-only navbar/menu of the WholeTale Dashboard does not include a link to the new "Settings" view. This can also be seen by resizing a desktop browser window to be sufficiently small.

Fixes #584

Approach

Update the mobile menu to include the missing link.

Also, I went ahead and changed the Manage icon from cog to hdd, as cog seems more suitable for a settings/preferences view.

How to Test

  1. Checkout and run this branch locally, rebuild the dashboard
  2. Login to the WholeTale Dashboard 3a. On Mobile: Look at the top navbar 3b. On Desktop: Resize your browser window to about half of your screen size
    • You should see a single "stacked horizontal bars" or "hamburger" icon at the top-right
  3. Click the "hamburger" icon
    • You should see a slide-out menu appear from the right
    • You should see a link to "Settings" now appears in the list
    • You should see that the link to "Manage" now uses the hdd icon
  4. Click the "Settings" link in the menu
    • You should be brought to the new "User Settings" view
craig-willis commented 4 years ago

The "User" menu also has "Log out", if that could also be added?