Problems with the new UI overview page

Magickbase / neuron-public-issues

Neuron Issues

6 stars 3 forks source link

Problems with the new UI overview page #159

Closed FrederLu closed 10 months ago

FrederLu commented 1 year ago

[ ] 1、The lack of a mask layer effect is an overall problem, and other pages also need to have it.

[ ] 2、The default style of the day/night switch button does not need to display the hover state.

[ ] 3、The style of the mouse hover day/night button (or synced) needs to be changed to a small hand, which is now displayed as a mouse pointer.

[ ] 4、When the synchronization is abnormal (red font), the small red dot is missing before the copy.

[ ] 5、The center of the i icon needs to be consistent with the text, and the back X is smaller than the UI draft.

[ ] 6、When creating or importing a wallet, there is a lack of screenshot prompts.

[ ] 7、Looking for preset trusted blocks is displayed abnormally.

devchenyan commented 1 year ago

https://github.com/nervosnetwork/neuron/pull/2708

FrederLu commented 1 year ago

Problems 1 and 3 still exist.

Problem 6 has been improved, but the prompt time is too short, and there is no need to prompt when switching wallets at the same time, and the information is only prompted at the initial entry.

https://github.com/Magickbase/neuron-public-issues/assets/25676576/68bbc652-01da-4f33-a314-885b3778ce34

devchenyan commented 1 year ago

Problems 1 : The mask layer and the whole background are the same background color. so it's not a problem @FrederLu

FrederLu commented 1 year ago

Problems 1 : The mask layer and the whole background are the same background color. so it's not a problem @FrederLu

@devchenyan @Kirl70 Now the function is not implemented.

Kirl70 commented 1 year ago

Problems 1 : The mask layer and the whole background are the same background color. so it's not a problem @FrederLu

@devchenyan @Kirl70 Now the function is not implemented.

The purpose of the mask layer is to ensure that the information on top of the mask layer remains visible at the forefront, regardless of page scrolling. The header mask layer should be present on all operational pages. Additionally, on this particular page, there should also be an mask layer for the bottom buttons.

devchenyan commented 1 year ago

Packaging for test is done in 5408143725 for commit https://github.com/nervosnetwork/neuron/commit/3edf8bb334ed62ee37476e470fa7c45a6507398c .

FrederLu commented 1 year ago

Packaging for test is done in 5408143725 for commit nervosnetwork/neuron@3edf8bb .

After the top is sucked, each page remains fixed when sliding. Now it is found that the top of the "Transaction History" and "Settings" pages will move when sliding.

https://github.com/Magickbase/neuron-public-issues/assets/25676576/2c8ef931-0483-4dfc-a2ad-1627af0ce28f

devchenyan commented 1 year ago

Packaging for test is done in 5461287659

FrederLu commented 1 year ago

Verified.