Updated navbar to latest design.

[Randy808]

To run locally, npm install and use the following snippets:

# Bitcoin
export API_URL=https://blockstream.info/api
export PORT=4999
source flavors/blockstream/config.env
source flavors/bitcoin-mainnet/config.env
npm run dev-server

# Liquid
export API_URL=https://blockstream.info/liquid/api
export PORT=5000
source flavors/blockstream/config.env
source flavors/liquid-mainnet/config.env
npm run dev-server

---

Bitcoin theme on Chrome (desktop):

Bitcoin theme on Chrome (mobile):

Liquid theme on Chrome (desktop):

Bitcoin theme on Firefox (desktop):

Bitcoin theme on Firefox (mobile):

[Randy808]

- [ ] Fix logo in light mode

EDIT: Product said to remove light mode so this action item is no longer applicable

[Randy808]

I'm going to move this back to draft since I want #627 to get merged first

[FedOken]

I think this could be attributed to this MR. It's up to you. It seems to me that the trigger area for opening the menu is quite large, you can try it locally, it opens unexpectedly before you actually hover over icon.

[FedOken]

For assets page, there are no gap between navbar and the page content

Are you planning to fix this in another PR?

[FedOken]

I prefer to use less nesting, I think it's easier to read something like this. At your discretion.

const hasModifier = e.metaKey || e.ctrlKey || e.altKey || e.shiftKey
if (hasModifier || e.key.toLowerCase() !== 'k') return

const searchInput = document.querySelector('.search-bar-input')
if (!searchInput) return

e.preventDefault()
searchInput.focus()

[Randy808]

Updated to be more legible

[Randy808]

@FedOken The overly large trigger area is intended. See the behavior on blockstream.info. Maybe less necessary then, but now the hamburger icon is a lot smaller

[Randy808]

Last commit rebased and added margin to the top of the assets table

[EddieHouston]

lgtm 👍