### Group
- [x] Group name font should be bold and 13px
### Header
- [x] Update icons in header view(toggle button and search button)
- [x] toggle button in collapsed state is not aligned center
- [x] toggle button color should be match with icons on toolbar in dark mode -> white
### Tabs
- [x] Spacing around tabs is a little bit bigger (8px → 4px)
- [x] Tab's width should be 36px / when in group, 32px
- [x] Pinned tabs border radius should be 8px, while unpinned tabs has 4px radius.
- [x] Pinned tabs' borderline looks too thin
- [x] Pinned tabs' drop shadow looks clipped
- [x] Active tab's BG color is too strong when it's dark mode
- [x] Divider color in dark mode is not accurate
### New tab button
- [x] Spacing and color of new tab button should be matched to those of tabs (icon padding, title etc.)
- [x] NewTabButton's hover effect should have padding just like tabs
Implements new NFT tab in the Wallet section.
The layout will use an auto-resizing grid that adapts the number of columns depending on the width of the device screen.
The logic that handles the calculation of the span count is implemented in AutoFitVerticalGridLayoutManager.
The new layout follows the Figma design provided and will default to "All networks" when multichain support will be merged.
VerticalTabStripRegionView::Layout() could be called multiple times.
Even when there's a damaged rect, the method could be called. This could
cause unnecessary layout and make hover card shaking. In order to prevent
that, check the last size and keep the dirty bit for layout.
* Add GetNftStandard to JsonRpcService
* Use JsonRpcService::GetNftStandard in BraveWalletService::AddUserAsset
* Complete support for ERC1155 tokens (add is_erc1155 property, etc)
This PR removes the uses of `WebUIDataSource::Create/Add`, which are
set to get removed in `cr113`, and replaces it with the unique call
`CreateAndAdd()`.
Chromium change:
https://chromium.googlesource.com/chromium/src/+/95ce39d5ebb5d59c4b96ac242652c6e5a08bd2f6
commit 95ce39d5ebb5d59c4b96ac242652c6e5a08bd2f6
Author: Lei Zhang <thestig@chromium.org>
Date: Sat Mar 18 06:04:22 2023 +0000
Get rid of WebUIDataSource::Create() and WebUIDataSource::Add()
- Create() has been replaced by CreateAndAdd().
- Replace the last Create() caller in WebUIDataSourceTest with a direct
WebUIDataSourceImpl allocation.
- Fold the 1 line Add() implementation into CreateAndAdd().
- Implement TestWebUIDataSource::AddDataSourceForBrowserContext().
- Update the last Add() caller, which is in a unit test, to use
TestWebUIDataSource::AddDataSourceForBrowserContext().
Bug: 1206140
* fix(wallet): update network icons
* feat: show network name and overlapping icon
* show overlapping network icon, name, and use list
* update token id for history
* add contract address check for ERC NFTs
* adjust overlapping network icon size
* Fix off-by-one-pixel in vertical tab strip
* Don't set inset when it's fullscreen
* Don't make host view wider than it's desired
* Fix not to set 1px inset when it's not vertical tab or when it's fullscreen
* Fix browser test - As the container's width already contains the inset, we shouldn't add
1 in test.
Make window.ethereum a proxy object
It uses a handler that would call things directly on the actual
JSEthereumProvider object so dApps which creates and uses their own proxy of
window.ethereum to access our provider won't throw a "Illegal invocation:
Function must be called on an object of type JSEthereumProvider" error.
Jay Harris