New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Dashboard widget resizing is glitchy in Chrome on Mac #1530

Closed
schakrava opened this Issue Nov 16, 2016 · 9 comments

Comments

Projects
None yet
3 participants
@schakrava
Member

schakrava commented Nov 16, 2016

A user reported that resizing disk activity widget on dashboard causes the boundaries to overlap. This was reported on Mac 10.11.6 with latest Chrome and Firefox. Apparently the behavior is the same on WIndows 10 with Chrome.

I've confirmed it on Mac with Chrome version 53.0.2785.113. I cannot reproduce it on Linux+Chrome or Firefox.

Finally, to reproduce on Mac+Chrome all you need to do is go to the dashboard and expand Disk activity widget and perhaps another one(eg: Network) if you don't see the issue already. Below screen shot shows the issue.

dashboard_glitch

@schakrava schakrava added the bug label Nov 16, 2016

@schakrava schakrava added this to the Point Bonita milestone Nov 16, 2016

@MFlyer

This comment has been minimized.

Show comment
Hide comment
@MFlyer

MFlyer Nov 16, 2016

Member

Hi @schakrava ,
got some initial tests and these don't help us (not able to reproduce):

WIN 7 + Chrome 54.0.2840.99 (latest)-> No overlapping, all fine
WIN 7 + Firefox 50 (latest)-> No overlapping, all fine

Had some time playing removing/adding widgets, having all maximized, some maximized, some not, but it was all ok.
Any idea if user having some add-on on firefox/chrome??

Member

MFlyer commented Nov 16, 2016

Hi @schakrava ,
got some initial tests and these don't help us (not able to reproduce):

WIN 7 + Chrome 54.0.2840.99 (latest)-> No overlapping, all fine
WIN 7 + Firefox 50 (latest)-> No overlapping, all fine

Had some time playing removing/adding widgets, having all maximized, some maximized, some not, but it was all ok.
Any idea if user having some add-on on firefox/chrome??

@MFlyer

This comment has been minimized.

Show comment
Hide comment
@MFlyer

MFlyer Nov 16, 2016

Member

Had another relaxing 20mins playing with widgets, but without success.
Tomorrow going to check over a WIN10 system
Looking better to that image got a clue: bootstrap is working fine & widgets too, problem is on line chart canvas resize (going out of our widget). Till can't reproduce it, but now I know where to check 😃
M.

Member

MFlyer commented Nov 16, 2016

Had another relaxing 20mins playing with widgets, but without success.
Tomorrow going to check over a WIN10 system
Looking better to that image got a clue: bootstrap is working fine & widgets too, problem is on line chart canvas resize (going out of our widget). Till can't reproduce it, but now I know where to check 😃
M.

MFlyer added a commit to MFlyer/rockstor-core that referenced this issue Nov 16, 2016

#1530 - Removing duplicated div id disk-details-ph, both on disk_util…
…_select and disk_utilization templates

Signed-off-by: Mirko Arena <mirko.arena@gmail.com>
@MFlyer

This comment has been minimized.

Show comment
Hide comment
@MFlyer

MFlyer Nov 16, 2016

Member

Hi @schakrava , while checking templates found this -> 003fb9a

Can you test if it solves over a Mac? a wrapper div always required by canvas (no wrapper => canvas overlapping on resize), but having 2 divs with same ID probably made it crazy/content loaded out of wrapper div

Waiting your checks :)

Member

MFlyer commented Nov 16, 2016

Hi @schakrava , while checking templates found this -> 003fb9a

Can you test if it solves over a Mac? a wrapper div always required by canvas (no wrapper => canvas overlapping on resize), but having 2 divs with same ID probably made it crazy/content loaded out of wrapper div

Waiting your checks :)

MFlyer added a commit to MFlyer/rockstor-core that referenced this issue Nov 17, 2016

#1530 - Removing duplicated div id disk-details-ph, both on disk_util…
…_select and disk_utilization templates

Signed-off-by: Mirko Arena <mirko.arena@gmail.com>

MFlyer added a commit to MFlyer/rockstor-core that referenced this issue Nov 20, 2016

#1530 - Removing duplicated div id disk-details-ph, both on disk_util…
…_select and disk_utilization templates

Signed-off-by: Mirko Arena <mirko.arena@gmail.com>
@MFlyer

This comment has been minimized.

Show comment
Hide comment
@MFlyer

MFlyer Dec 17, 2016

Member

Apparently the behavior is the same on WIndows 10 with Chrome

Good UTC morning @schakrava :) , with my new laptop going to test this on WIN10 too so hopefully will have better chance to catch it.

Member

MFlyer commented Dec 17, 2016

Apparently the behavior is the same on WIndows 10 with Chrome

Good UTC morning @schakrava :) , with my new laptop going to test this on WIN10 too so hopefully will have better chance to catch it.

@MFlyer

This comment has been minimized.

Show comment
Hide comment
@MFlyer

MFlyer Jan 2, 2017

Member

Hi @schakrava,
after Xmas flu and a battle vs win10 & wifi finally got my new laptop running with dual boot and...reproduced this issue :)

Hope to solve asap, there's an error raising when resizing from small to big

M.

Member

MFlyer commented Jan 2, 2017

Hi @schakrava,
after Xmas flu and a battle vs win10 & wifi finally got my new laptop running with dual boot and...reproduced this issue :)

Hope to solve asap, there's an error raising when resizing from small to big

M.

@MFlyer

This comment has been minimized.

Show comment
Hide comment
@MFlyer

MFlyer Jan 3, 2017

Member

Got an update about this, @schakrava & @phillxnet can you check too?
Accidentally played with WIN10 resolution and FONT SCALING, WIN10 on a wide screen has 125% default...moving it to 100% (normal) Disk Widget feels ok.

Looking anyway for a solution 😉

Member

MFlyer commented Jan 3, 2017

Got an update about this, @schakrava & @phillxnet can you check too?
Accidentally played with WIN10 resolution and FONT SCALING, WIN10 on a wide screen has 125% default...moving it to 100% (normal) Disk Widget feels ok.

Looking anyway for a solution 😉

@phillxnet

This comment has been minimized.

Show comment
Hide comment
@phillxnet

phillxnet Jan 3, 2017

Member

@MFlyer Sorry I am, fortunately for me at least, without a Win10 machine (a state I intend to maintain by the way). And my OSX machine is still out on loan (apparently it's a long loan :) ) Didn't the original issue also show up in Firefox, at least with OSX?

Member

phillxnet commented Jan 3, 2017

@MFlyer Sorry I am, fortunately for me at least, without a Win10 machine (a state I intend to maintain by the way). And my OSX machine is still out on loan (apparently it's a long loan :) ) Didn't the original issue also show up in Firefox, at least with OSX?

@MFlyer

This comment has been minimized.

Show comment
Hide comment
@MFlyer

MFlyer Jan 3, 2017

Member

without a Win10 machine (a state I intend to maintain by the way)

😭
New laptop -> 7th Gen Intel -> supported only Linux and Win10 (obviously got a dual boot) -> no way with previous win sys :/

Member

MFlyer commented Jan 3, 2017

without a Win10 machine (a state I intend to maintain by the way)

😭
New laptop -> 7th Gen Intel -> supported only Linux and Win10 (obviously got a dual boot) -> no way with previous win sys :/

@MFlyer

This comment has been minimized.

Show comment
Hide comment
@MFlyer

MFlyer Jan 4, 2017

Member

Tested on a second win10 machine and can confirm - on windows side - related to screen scaling (125% -> bad dashboard widget, 100% -> ok)

M.

Member

MFlyer commented Jan 4, 2017

Tested on a second win10 machine and can confirm - on windows side - related to screen scaling (125% -> bad dashboard widget, 100% -> ok)

M.

MFlyer added a commit to MFlyer/rockstor-core that referenced this issue Jan 4, 2017

#1530 - New Dom structure to fix canvas overlapping
Signed-off-by: Mirko Arena <mirko.arena@gmail.com>

@schakrava schakrava closed this in a99c8d7 Jan 5, 2017

schakrava added a commit that referenced this issue Jan 5, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment