Skip to content
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

Theme: Page jumps to top when server-page refreshes (on Safari) #551

Closed
applejack1923 opened this issue Sep 26, 2016 · 9 comments
Closed
Labels

Comments

@applejack1923
Copy link

When on a page which updates itself during the process such as software update or a verbose cron job the page keeps jumping very quickly back to the top and only stops when the process has finished.

I'm not sure which version this started with as it has been going on for I would say at least a couple of months. I have only tested this in Safari so next time I will try Chrome and the server has been re-booted since it first began.

Authentic 18.10, Virtualmin 5.04.gpl, Webmin 1.810, CSF 9.22, CentOS 6, Safari 9, Mac

@iliajie
Copy link
Collaborator

iliajie commented Sep 26, 2016

Alright, I will try to reproduce it in Safari on MacBook.

@iliajie iliajie changed the title Page jumps to top on data refresh Theme: Page jumps to top when server-page refreshes (on Safari) Sep 26, 2016
@iliajie
Copy link
Collaborator

iliajie commented Oct 23, 2016

Confirmed and will be fixed in 18.20

@iliajie iliajie closed this as completed Oct 23, 2016
@iliajie
Copy link
Collaborator

iliajie commented Oct 29, 2016

Version 18.20 is out, please upgrade.

1 similar comment
@iliajie
Copy link
Collaborator

iliajie commented Oct 29, 2016

Version 18.20 is out, please upgrade.

@applejack1923
Copy link
Author

Good job thanks for sorting and also the CSF issue. One point though I see you have added arrows in the drop down menu of the virtual servers under Virtualmin and which arrow depends on whether it is an alias or sub server. Frankly this is somewhat annoying as it now looks too cluttered and is unnecessary as these have different levels of indent anyhow.

@iliajie
Copy link
Collaborator

iliajie commented Oct 30, 2016

It was done in response to:

#466

@applejack1923
Copy link
Author

applejack1923 commented Oct 30, 2016

I have always been able to view the different padding. There is no mention in that thread about which browser they are using in not being able to see this. Rather than adding the arrows as content surely it would have been better to use the pseudo class before on the li tag then at least one could edit it in the css or better still just make the padding work in the browsers where the it is currently not showing and get rid of the arrows.

@iliajie
Copy link
Collaborator

iliajie commented Oct 30, 2016

I'll try making it optional in 18.30

@iliajie
Copy link
Collaborator

iliajie commented Nov 1, 2016

@applejack1923 #569

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants