NebraLtd / hm-diag

Helium Miner Diagnostics
https://nebra.io/hnt
MIT License
21 stars 24 forks source link

Remaining layout issues June #593

Closed shawaj closed 1 year ago

shawaj commented 1 year ago

Ip Addresses not obviously separated...

Screenshot_20230605_004631_Chrome

Footer still missing on small screen / mobile...

Screenshot_20230605_004228_Chrome

Side nav auto closes if you scroll down...

https://github.com/NebraLtd/hm-diag/assets/3359418/937a639d-0b5b-4856-9dae-7eb4653d80db

shawaj commented 1 year ago

Meta theme colour removed since hyper update...

Fixed by #594

shawaj commented 1 year ago

Large amount of empty space between content and footer on small screens... Screenshot_20230605_004631_Chrome Screenshot_20230605_004348_Chrome Screenshot_20230605_004246_Chrome

shawaj commented 1 year ago

@KevinWassermann94 @WayneNebra @robputt crack any layout issues you find in the new diagnostics in here for @Louisreed

shawaj commented 1 year ago

Should add dark theme as per #475

shawaj commented 1 year ago

helium serial spills over edge of container...

image
shawaj commented 1 year ago

We should also fix #573 as part of this too...

Screenshot 2023-06-07 at 18 11 38
shawaj commented 1 year ago

Also on upgrade page, the button is an odd colour and the whole thing is not responsive and does not fit smallish windows...

image
Louisreed commented 1 year ago

IP addresses are being displayed through {{ device_info.ip_address }}, no available option to filter

Louisreed commented 1 year ago

Unable to reproduce 'Side nav auto closes if you scroll down' error. Will require further testing on more devices. @shawaj is this just with chrome on Android?

shawaj commented 1 year ago

@Louisreed I've only tried on Chrome on Android on s23 ultra.

I can try on a different browser

Not sure if anyone has an iPhone to test with

WayneNebra commented 1 year ago

Same for me on an Android Oppo.

https://github.com/NebraLtd/hm-diag/assets/90446757/644149af-e8e4-4e38-bf77-08886bfad13b

Louisreed commented 1 year ago

Strange! I'll have to try with other devices. I have an iPad so will test on there as well.

shawaj commented 1 year ago

Same issue in iPhone 13 pro safari @Louisreed https://nebraltd.slack.com/archives/C024BNQ1Y6T/p1686242502586949

shawaj commented 1 year ago

IP addresses are being displayed through {{ device_info.ip_address }}, no available option to filter

@Louisreed can they be split out using JS or something? As presumably they have got spaces between them?

Louisreed commented 1 year ago

Issue resolved!

Louisreed commented 1 year ago

IP addresses are being displayed through {{ device_info.ip_address }}, no available option to filter

@Louisreed can they be split out using JS or something? As presumably they have got spaces between them?

Yeah I thought about using split() but would be better if I could directly access each IP. I'll see what I can do.

shawaj commented 1 year ago

@Louisreed ping @robputt on slack - he might know a better way to split these out in python and then just pull them in as an array or list or whatever

shawaj commented 1 year ago

Closed by #608 #615 #616