yaooqinn opened a new pull request, #54613:
URL: https://github.com/apache/spark/pull/54613
### What changes were proposed in this pull request?
Replaces jQuery `.show()`, `.hide()`, `.toggle()`, `.css("display", ...)`,
and inline `style="display:none"` with Bootstrap 5 `d-none` utility class
across the Spark Web UI.
**JS changes (5 files):**
- `environmentpage.js`: `.show()`/`.hide()` →
`.removeClass("d-none")`/`.addClass("d-none")`
- `stagepage.js`: 6 `.show()`/`.hide()` calls for speculation summary and
accumulator table
- `executorspage.js`: `.hide()`/`.show()` for active-process-container and
flamegraph toggle
- `streaming-page.js`: `.toggle("collapsed")` → `.toggleClass("d-none")`
- `log-view.js`: `.css("display", ...)` → `d-none` class add/remove
**Scala changes (6 files):**
- 4 LogPage variants: `style="display: none;"` → `class="d-none"`
- `UIUtils.scala`: dag-viz-metadata div
- `StreamingPage.scala`: inputs-table row
### Why are the changes needed?
Using `d-none` consistently across the codebase:
- Aligns with Bootstrap 5 conventions (no inline styles)
- Makes visibility state inspectable via class (easier debugging)
- Removes reliance on jQuery `.show()`/`.hide()` which manipulate inline
styles
Part of SPARK-55760 (Spark Web UI Modernization).
### Does this PR introduce _any_ user-facing change?
No. Visibility behavior is identical.
### How was this patch tested?
- `lint-js` passes
- `scalastyle` passes
- Manual UI verification
### Was this patch authored or co-authored using generative AI tooling?
Yes, co-authored with GitHub Copilot.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
