tor-browser

The Tor Browser
git clone https://git.dasho.dev/tor-browser.git
Log | Files | Refs | README | LICENSE

commit 424a165e85f9ad6589c928772f9fe4093223208b
parent 7a30b24c52bf46f431d4fe0798497948ba29f2d8
Author: Thomas Wisniewski <twisniewski@mozilla.com>
Date:   Wed,  8 Oct 2025 13:38:20 +0000

Bug 1890762 - remove our webcompat intervention for urvaerket.dk; r=denschub,webcompat-reviewers

Differential Revision: https://phabricator.services.mozilla.com/D267801

Diffstat:
Mbrowser/extensions/webcompat/data/interventions.json | 17-----------------
Dbrowser/extensions/webcompat/injections/css/bug1890762-urvaerket.dk-fix-mobile-filter-scrolling.css | 18------------------
Dtesting/webcompat/interventions/tests/test_1890762_urvaerket_dk.py | 43-------------------------------------------
3 files changed, 0 insertions(+), 78 deletions(-)

diff --git a/browser/extensions/webcompat/data/interventions.json b/browser/extensions/webcompat/data/interventions.json @@ -1868,23 +1868,6 @@ } ] }, - "1890762": { - "label": "urvaerket.dk", - "bugs": { - "1890762": { - "issue": "broken-scrolling", - "matches": ["*://*.urvaerket.dk/*"] - } - }, - "interventions": [ - { - "platforms": ["android"], - "content_scripts": { - "css": ["bug1890762-urvaerket.dk-fix-mobile-filter-scrolling.css"] - } - } - ] - }, "1892073": { "label": "snapchat.com", "bugs": { diff --git a/browser/extensions/webcompat/injections/css/bug1890762-urvaerket.dk-fix-mobile-filter-scrolling.css b/browser/extensions/webcompat/injections/css/bug1890762-urvaerket.dk-fix-mobile-filter-scrolling.css @@ -1,18 +0,0 @@ -/* This Source Code Form is subject to the terms of the Mozilla Public - * License, v. 2.0. If a copy of the MPL was not distributed with this - * file, You can obtain one at http://mozilla.org/MPL/2.0/. */ - -/** - * salling.dk - filters cannot be scrolled on the mobile layout - * Bug #1890762 - https://bugzilla.mozilla.org/show_bug.cgi?id=1890762 - * WebCompat issue #75895 - https://github.com/webcompat/web-bugs/issues/75895 - * - * They are relying on containers with display:inline-table and a definite height - * set with JS to propagate that value to relative-height children, which breaks - * the scrolling on those children. This CSS works around the problem. - */ -@media (max-width: 767px) { - .filter.open { - display: block; - } -} diff --git a/testing/webcompat/interventions/tests/test_1890762_urvaerket_dk.py b/testing/webcompat/interventions/tests/test_1890762_urvaerket_dk.py @@ -1,43 +0,0 @@ -import asyncio - -import pytest - -URL = "https://www.urvaerket.dk/herreure-c-5097.html" -COOKIES_CSS = "button[class*=accept]" -COOKIES_OVERLAY_CSS = "#coiOverlay" -FILTER_CSS = ".filter_products .fp" -FILTER_CONTENT_CSS = ".filter-content" -MAERKE_CSS = ".filter-content .fpbx" - - -async def can_scroll_filters(client): - await client.navigate(URL, wait="none") - client.await_css(COOKIES_CSS, is_displayed=True).click() - client.await_element_hidden(client.css(COOKIES_OVERLAY_CSS)) - client.await_css(FILTER_CSS, is_displayed=True).click() - client.await_css(MAERKE_CSS, is_displayed=True).click() - content = client.await_css(FILTER_CONTENT_CSS, is_displayed=True) - await asyncio.sleep(1) - top = client.execute_script("return arguments[0].scrollTop", content) - for i in range(20): - await asyncio.sleep(0.1) - client.apz_scroll(content, dy=100) - if top != client.execute_script("return arguments[0].scrollTop", content): - return True - return False - - -@pytest.mark.only_platforms("android") -@pytest.mark.actual_platform_required -@pytest.mark.asyncio -@pytest.mark.with_interventions -async def test_enabled(client): - assert await can_scroll_filters(client) - - -@pytest.mark.only_platforms("android") -@pytest.mark.actual_platform_required -@pytest.mark.asyncio -@pytest.mark.without_interventions -async def test_disabled(client): - assert not await can_scroll_filters(client)