subquery / network-app

dApp for Node Operators, consumers and delegators interact with SubQuery Network
https://app.subquery.network
GNU General Public License v3.0
14 stars 8 forks source link

Fix/responsive mobile #723

Closed duchuytb9x closed 3 months ago

duchuytb9x commented 3 months ago

Description

Change some Pages to meet Responsive Mobile

Test cases:

Please describe tested cases.

Type of change

Please delete options that are not relevant.

UI Changes

Add before and after changes for UI if needed.

before after
1 1
3 4
5 6
7 8
9 10
11 12
13 14
16 15
HuberTRoy commented 3 months ago

Thanks for your contribute, a little more need to change:

  1. The align of project detail looks not same as before, can you please fix this?
image image
  1. substr is a deprecated method, can you please change a method to do?
duchuytb9x commented 3 months ago
  1. I will edit the project details so that the layout does not change
  2. Can I change the DeploymentId to aaa...bbb? Because on mobile the text is too long
HuberTRoy commented 3 months ago

@duchuytb9x Yes, shrink the deploymentId is good. Just don't use substr, use slice or something would better than a deprecated method. Also if can add more type asset would more better: e.g: if (!isString(value)) return value

duchuytb9x commented 3 months ago

There is a table in the project's Indexer List that requires additional scrolling. I see you are ToDo something

duchuytb9x commented 3 months ago

I have just changed the code according to the review

duchuytb9x commented 3 months ago

@HuberTRoy Almost every necessary page has been changed to be responsive for mobile

HuberTRoy commented 3 months ago

Made some tiny changes. Very thanks for the great work~.