Skip to content
GitLab
  • Menu
Projects Groups Snippets
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
    • Switch to GitLab Next
  • Sign in / Register
  • adblockplusui adblockplusui
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
    • Locked Files
  • Issues 247
    • Issues 247
    • List
    • Boards
    • Service Desk
    • Milestones
    • Iterations
    • Requirements
  • Merge requests 10
    • Merge requests 10
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
    • Test Cases
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages & Registries
    • Packages & Registries
    • Container Registry
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Code review
    • Insights
    • Issue
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • Adblock inc
  • Extensions
  • Adblock Plus
  • adblockplusuiadblockplusui
  • Issues
  • #1003
Closed
Open
Created Jul 23, 2021 by Thomas Greiner@ThomasGreinerOwner

Stop applying filter updates to developer tools panel

Background / User story

We're currently using Core's defaultMatcher.match() function to update existing records in the developer tools panel in response to filter changes (e.g. filter got removed). This functionality is not going to be available to us after #972 (closed). Furthermore, the records shouldn't show filters that could've matched but filters that did match, so updating existing records probably doesn't make sense anyway.

Therefore we should stop updating existing records in the developer tools panel.

What to change

  • Design: N/A
  • Research: N/A
  • Spec: eyeo/specs/spec!391
  • Legal: TBD
  • Development: In developer tools panel:
    • Stop updating existing records.
    • Restore visual feedback for action buttons.
    • If an existing request is encountered again, but with a different matching filter, a new record should be added, rather than an existing one updated. (we stopped replacing filters on existing records in #909 (closed))

Hints for testers

  • The developer tools panel no longer updates in response to filter changes elsewhere in the extension.
  • We don't know whether a filter actually matches any of the requests shown in the developer tools panel, so what we do is:
    • Add filter to one row, even if it could apply to some other row as well.
    • Remove filter from one or more rows.
  • Removing an allowlisting filter that was added while the developer tools panel is open should show the blocking filter that was there before.
  • The footer remains visible, even after undoing all the changes since the opening of the developer tools panel (i.e. removing all allowlisting filters that the user added).

Hints for translators

N/A

Edited Feb 17, 2022 by Thomas Greiner
Assignee
Assign to
Time tracking