Skip to content

fix: progress indicator#5557

Merged
alexander-akait merged 1 commit into
webpack:masterfrom
calebdwilliams:fix-progress-pointer-events
Aug 9, 2025
Merged

fix: progress indicator#5557
alexander-akait merged 1 commit into
webpack:masterfrom
calebdwilliams:fix-progress-pointer-events

Conversation

@calebdwilliams

Copy link
Copy Markdown
Contributor
  • This is a bugfix
  • This is a feature
  • This is a code refactor
  • This is a test update
  • This is a docs update
  • This is a metadata update

For Bugs and Features; did you add new tests?

No, this is only a CSS addition to the progress indicator setting pointer-events: none; this prevents the element from sitting on top of the DOM and blocking users from being able to click on that region of their screen.

Motivation / Use-Case

With the progress indicator enabled, the HTMLElement persists in the DOM and is overlaying a large portion preventing users from clicking that region even if the element is invisible.

Breaking Changes

N/A

Additional Info

sets the pointer events to none on the progress indicator
@linux-foundation-easycla

linux-foundation-easycla Bot commented Aug 6, 2025

Copy link
Copy Markdown

CLA Signed

  • ✅login: calebdwilliams / (ad31a39)

The committers listed above are authorized under a signed CLA.

@codecov

codecov Bot commented Aug 9, 2025

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 83.41%. Comparing base (64e4afe) to head (ad31a39).
⚠️ Report is 2 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #5557      +/-   ##
==========================================
+ Coverage   83.31%   83.41%   +0.09%     
==========================================
  Files          13       13              
  Lines        2038     2038              
  Branches      750      750              
==========================================
+ Hits         1698     1700       +2     
+ Misses        306      304       -2     
  Partials       34       34              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.
@alexander-akait alexander-akait merged commit 41a53a1 into webpack:master Aug 9, 2025
52 of 53 checks passed
@prichey

prichey commented Dec 10, 2025

Copy link
Copy Markdown

👋 @alexander-akait Is there any chance you could cut a new release with this change included? Thanks!

@alexander-akait

Copy link
Copy Markdown
Member

@prichey Yeah, we will need to do it, ping me at the end of week, I want to finish a couple more things

@prichey

prichey commented Dec 10, 2025

Copy link
Copy Markdown

Great, thanks!

@prichey

prichey commented Dec 15, 2025

Copy link
Copy Markdown

@alexander-akait bump! 🙂

@calebdwilliams

Copy link
Copy Markdown
Contributor Author

Could we get this released?

@calebdwilliams

Copy link
Copy Markdown
Contributor Author

Bump

@alexander-akait

Copy link
Copy Markdown
Member
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants