Merge lp:~daker/webbrowser-app/fix.airbnb-smartbanner into lp:webbrowser-app
Proposed by
Adnane Belmadiaf
Status: | Needs review |
---|---|
Proposed branch: | lp:~daker/webbrowser-app/fix.airbnb-smartbanner |
Merge into: | lp:webbrowser-app |
Diff against target: |
54 lines (+39/-0) 2 files modified
src/Ubuntu/Web/UbuntuWebContext.qml (+6/-0) src/Ubuntu/Web/airbnb-smartbanner.js (+33/-0) |
To merge this branch: | bzr merge lp:~daker/webbrowser-app/fix.airbnb-smartbanner |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Olivier Tilloy | Needs Fixing | ||
Review via email: mp+230855@code.launchpad.net |
Commit message
Added a userscript to hide the airbnb smartbanner
To post a comment you must log in.
Unmerged revisions
- 669. By Adnane Belmadiaf
-
Fixes
- 668. By Adnane Belmadiaf
-
Added a userscript to hide the airbnb smartbanner
The metadata of the user script should probably have a "@include" directive to limit the execution of the script to airbnb only.
Also, I’m not seeing this work (testing on desktop but forcing the formFactor to mobile, with your changes I’m still seeing the banner show up).