Closed Bug 1636385 Opened 2 years ago Closed 2 years ago

Netmonitor blocking - Set context menu options to disabled_status if all options are Enabled or Disabled already

Categories

(DevTools :: Netmonitor, enhancement, P3)

Unspecified
All
enhancement

Tracking

(firefox77 wontfix, firefox78 verified)

VERIFIED FIXED
Firefox 78
Tracking Status
firefox77 --- wontfix
firefox78 --- verified

People

(Reporter: cfogel, Assigned: lelouch.cpp)

References

Details

Attachments

(2 files)

Affected versions

  • 77.0b2, 78.0a1(2020-05-07);

Affected platforms

  • Windows 10, macOS 10.15.3, Ubuntu 18.04;

Steps to reproduce

  1. Launch Firefox, DevTools, Netmonitor tab;
  2. Access any webpage, in the Netmonitor tab enable the Request Blocking panel;
  3. Add any filters to start blocking(at least 2 ;
  4. Right click over any added filter in the list;
  5. Click on the Enable all option;

Expected result / Enhancement sugestion

  • option is disabled-unclickable;

Actual result

  • option is enabled-clickable;

Regression range

  • not a regression, followup from 1588076; would make the use-flow more consistent across the browser.

Additional info

  • attached screenshot the better illustrate the issue;
  • same scenario would be for the [Disable all] option when they're disabled already;

I would like to take this up if no one is working on it already
Thank you

Assignee: nobody → lelouch.cpp

Assigned to you. Thanks

The context menu of the network blocking panel have options to
"Enable all" and "Disable all" for enabling and disbaling all the
patters in the request blocking panel at once.
This patch disables those options in case all the URLs are already
enabled or disabled. Having the buttons active all the time might
lead to confusing situtaions especially with long blocking pattern
list where it may lead the user to think there is a URL that has not
been enabled/disabled eve after clicking the menu option.

Pushed by hmanilla@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/09b7ee5e868b
Disable context menu options if blocking is enabled/disabled for all URL in Network Blocking Panel r=bomsy
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 78

Verified with 78.0a1 (2020-05-26) on macOS 10.15.3., Ubuntu 16, Windows 10.

Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.