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.

Sign up for GitHub

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

Jump to bottom

remove code for dealing with twitter mobile noscript redirect #5439

Closed
diracdeltas opened this issue Jul 29, 2019 · 1 comment
Closed

remove code for dealing with twitter mobile noscript redirect #5439

diracdeltas opened this issue Jul 29, 2019 · 1 comment
Assignees
@iefremov
Labels
feature/shields/!scripts Blocking JavaScript with Shields priority/P4 Planned work. We expect to get to it "soon".
Milestone
Closed / Dupe / I...

Comments

@diracdeltas
Copy link
Member

brave/browser-laptop#2884 doesn't seem to be relevant anymore because if you block scripts on twitter.com, the site doesn't auto redirect to mobile.twitter.com

cc @bridiver
@bridiver bridiver assigned iefremov and unassigned iefremov Jul 29, 2019
@kjozwiak kjozwiak added the feature/shields/!scripts Blocking JavaScript with Shields label Jul 30, 2019
@tildelowengrimm tildelowengrimm added the priority/P4 Planned work. We expect to get to it "soon". label Jul 30, 2019
@iefremov
Copy link
Contributor

Fixed after migration to NetworkService

@bbondy bbondy added this to the Closed / Invalid milestone Jun 3, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@iefremov iefremov

Labels
feature/shields/!scripts Blocking JavaScript with Shields priority/P4 Planned work. We expect to get to it "soon".
Projects
None yet
Milestone
Closed / Dupe / Invalid
Development

No branches or pull requests
5 participants
@diracdeltas @tildelowengrimm @bbondy @kjozwiak @iefremov