Skip to content

Remove iframe-resizer and render support page locally - #1285

Merged
danieliser merged 1 commit into
developfrom
agent/remove-iframe-resizer
Aug 6, 2026
Merged

Remove iframe-resizer and render support page locally#1285
danieliser merged 1 commit into
developfrom
agent/remove-iframe-resizer

Conversation

@danieliser

@danieliser danieliser commented Aug 6, 2026

Copy link
Copy Markdown
Member

Summary

  • replace the externally hosted Help & Support iframe with a native WordPress admin page
  • remove iframe-resizer registration, enqueueing, dependency, lockfile entries, and build copying
  • retain direct links to guides, documentation, community, extension docs, developer wiki, and support
  • coordinate with PopupMaker/Remote-Content#20, which already removed the extension's legacy dependency

Why

The previous iframe only displayed a resource page and linked users onward to the support site; it did not submit tickets. Rendering those resources locally removes an unnecessary cross-origin iframe and the iframe-resizer dependency.

Validation

  • frozen pnpm install
  • production build
  • focused Stylelint for the support page
  • PHPCS and PHP syntax checks for changed PHP files
  • repository search confirms no iframe-resizer references remain

Follow-up

A direct ticket form can be added separately once the support system endpoint/authentication contract and diagnostic-data consent requirements are defined.

Summary by CodeRabbit

  • New Features

    • Added a redesigned admin support page with localized help content.
    • Added five resource cards linking to helpful support materials.
    • Added a dedicated contact-support call to action.
  • Improvements

    • Improved support-page layout, readability, and visual hierarchy.
    • Added responsive behavior for smaller screens.
    • Added clearer hover and keyboard-focus states for support links.

@coderabbitai

coderabbitai Bot commented Aug 6, 2026

Copy link
Copy Markdown

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: 598bddb6-055a-4501-b741-e92429aaf4b9

📥 Commits

Reviewing files that changed from the base of the PR and between 0de5132 and 9d22783.

⛔ Files ignored due to path filters (1)
  • pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml
📒 Files selected for processing (8)
  • assets/js/src/admin/support-page.scss
  • classes/Admin/Assets.php
  • classes/Admin/Support.php
  • classes/Site/Assets.php
  • docs/build-processes.md
  • package.json
  • readme.md
  • webpack.old.config.js

Walkthrough

The admin support page replaces an embedded external iframe with localized resource cards and a contact-support action. Responsive styles support desktop and mobile layouts. The iframe-resizer asset, dependency, vendor copy rule, and documentation references are removed.

Changes

Support page update

Layer / File(s) Summary
Localized support page and responsive styling
classes/Admin/Support.php, assets/js/src/admin/support-page.scss
The support page renders five external resource cards and a contact-support action. New styles provide the grid, typography, interaction states, and mobile stacking.
Asset and dependency cleanup
classes/Admin/Assets.php, classes/Site/Assets.php, package.json, webpack.old.config.js
iframe-resizer registration, enqueueing, production dependency, and vendor copy configuration are removed.
Documentation cleanup
docs/build-processes.md, readme.md
The documented external dependency and Built With entries no longer include iframe-resizer.

Estimated code review effort: 3 (Moderate) | ~20 minutes

Sequence Diagram(s)

sequenceDiagram
  participant Admin as WordPress admin
  participant SupportPage as PUM_Admin_Support::page()
  participant Resource as External resource
  Admin->>SupportPage: Open support page
  SupportPage-->>Admin: Render resource cards and contact action
  Admin->>Resource: Open selected resource in a new tab
Loading
✨ Finishing Touches 💡 1
🛠️ Fix failing CI checks 💡
  • Create stacked PR
  • Commit on current branch
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch agent/remove-iframe-resizer

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@danieliser
danieliser marked this pull request as ready for review August 6, 2026 02:03
@danieliser
danieliser merged commit fe3359f into develop Aug 6, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant