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

Display ad blocker message at the top of the screen instead of at the bottom #10137

Closed
mattab opened this issue May 10, 2016 · 4 comments
Closed
Labels
c: Usability For issues that let users achieve a defined goal more effectively or efficiently. wontfix If you can reproduce this issue, please reopen the issue or create a new one describing it.

Comments

@mattab
Copy link
Member

mattab commented May 10, 2016

Recently in the forums an issue was caused by the ad blocker AdBlockPlus blocking assets from loading. The reason that user did not notice the problem is because our useful ad blocker message (introduced in #8949 #5094) is displayed at the bottom of the page and not at the top. It takes user to scroll down to view the message.

Would it be possible to display the message at the top instead?

monday may 9 2016 - web analytics reports - 2016-05-10 14-19-01

@mattab mattab added the c: Usability For issues that let users achieve a defined goal more effectively or efficiently. label May 10, 2016
@tsteur
Copy link
Member

tsteur commented May 10, 2016

One of the reasons it is shown at the bottom of the screen is that we cannot detect ad blockers 100% correct and there could be other reasons for it such as problems with internet etc.

Are you sure what the user posted is related to ad blocker and related to this screen? It doesn't look like it

@mattab
Copy link
Member Author

mattab commented May 10, 2016

It was not related to ad blocker actually and since it's not possible to display at the top easily as you explain, i'm closing as "wont fix".

@mattab mattab closed this as completed May 10, 2016
@mattab mattab added the wontfix If you can reproduce this issue, please reopen the issue or create a new one describing it. label May 10, 2016
@sandeepjain2015
Copy link

thanks from bottom of heart @mattab after remove adblock it is working

@Findus23
Copy link
Member

will be fixed by #12493

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
c: Usability For issues that let users achieve a defined goal more effectively or efficiently. wontfix If you can reproduce this issue, please reopen the issue or create a new one describing it.
Projects
None yet
Development

No branches or pull requests

4 participants