Open
Conversation
…config (#1267) When a manual config defines series with string keys (e.g. named series), override() permanently mutates chart.settings.series to a plain object. On resize, the numeric-index conversion incorrectly processed those string keys, producing a sparse array with length=0 that caused Google Charts to crash with "can't access property color, b is null".
* fix: prevent double clipboard alert on copy action render-facade.js could be loaded more than once (registered under multiple handles when charts from different libraries appear on the same page). The copy ClipboardJS instance had no guard against double-initialization, causing the success alert to fire twice. Added vizClipboard2 guard to mirror the existing vizClipboard1 pattern already used for the shortcode copy button. * fix: copy still happening twice * chore: bump version for dev builds * chore: bump version for dev builds * chore: bump version for dev builds * chore: bump version for dev builds
Fixes the issue with scrolling in Chrome wasn't always working.
* fix: visualizer getCSV method throwing constant errors * fix: pr review changes * fix: pr review changes
…1258) On some server setups (e.g. nginx with FTP configured but no active connection), WordPress initializes its FTP filesystem driver but fails when it tries to use it, throwing a PHP fatal error that crashes the page. Wrap the filesystem calls in a try/catch so any such failure falls back to the bundled customization.js gracefully.
Move override() to after settings.plugins.tooltip is assigned and before new Chart(), so manual config is applied last. Also change $.extend to deep merge so nested plugin options merge rather than replace, preserving tooltip callbacks.
HardeepAsrani
added
the
pr-checklist-skip
Contributor
Bumps [phpstan/phpstan](https://github.com/phpstan/phpstan) from 2.1.22 to 2.1.33. - [Release notes](https://github.com/phpstan/phpstan/releases) - [Changelog](https://github.com/phpstan/phpstan/blob/2.1.x/CHANGELOG.md) - [Commits](phpstan/phpstan@2.1.22...2.1.33) --- updated-dependencies: - dependency-name: phpstan/phpstan dependency-version: 2.1.33 dependency-type: direct:development update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [phpcompatibility/phpcompatibility-wp](https://github.com/PHPCompatibility/PHPCompatibilityWP) from 2.1.5 to 2.1.8. - [Release notes](https://github.com/PHPCompatibility/PHPCompatibilityWP/releases) - [Commits](PHPCompatibility/PHPCompatibilityWP@2.1.5...2.1.8) --- updated-dependencies: - dependency-name: phpcompatibility/phpcompatibility-wp dependency-version: 2.1.8 dependency-type: direct:development update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
….3 (#1239) * chore(deps-dev): bump szepeviktor/phpstan-wordpress from 2.0.2 to 2.0.3 Bumps [szepeviktor/phpstan-wordpress](https://github.com/szepeviktor/phpstan-wordpress) from 2.0.2 to 2.0.3. - [Release notes](https://github.com/szepeviktor/phpstan-wordpress/releases) - [Commits](szepeviktor/phpstan-wordpress@v2.0.2...v2.0.3) --- updated-dependencies: - dependency-name: szepeviktor/phpstan-wordpress dependency-version: 2.0.3 dependency-type: direct:development update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> * chore: update baseline * chore: update baseline --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Hardeep Asrani <hardeepasrani@gmail.com>
…1242) * chore(deps-dev): bump wp-coding-standards/wpcs from 2.3.0 to 3.3.0 Bumps [wp-coding-standards/wpcs](https://github.com/WordPress/WordPress-Coding-Standards) from 2.3.0 to 3.3.0. - [Release notes](https://github.com/WordPress/WordPress-Coding-Standards/releases) - [Changelog](https://github.com/WordPress/WordPress-Coding-Standards/blob/develop/CHANGELOG.md) - [Commits](WordPress/WordPress-Coding-Standards@2.3.0...3.3.0) --- updated-dependencies: - dependency-name: wp-coding-standards/wpcs dependency-version: 3.3.0 dependency-type: direct:development update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com> * chore: fix phpcs * chore: generate baseline --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Hardeep Asrani <hardeepasrani@gmail.com>
…1269) * fix: guard against undefined properties on license object in Library When the license option is stored as a stdClass without `key` or `download_id` properties (e.g. on a free/unlicensed install), PHP 8 throws an undefined-property notice. Add isset() guards so both properties fall back to an empty string. * fix: fix telemetry error
The info message was visible by default. Hide it via inline style and toggle visibility via JS only when entering/leaving editor mode.
…isabled (#1271) - Delete Visualizer_Render_Sidebar_Type_DataTable_DataTable (backward compat stub no longer needed; upgrade routine handles DB migration) - Remove the now-redundant class name redirect in Module::load_chart_type() - Hide pagination-dependent fields (rows per page, pagination type) in both DataTable/Tabular and GoogleCharts/Tabular sidebars when pagination is disabled; toggle visibility via JS on checkbox change - Update phpstan baseline and add memory-limit/reportUnmatchedIgnoredErrors
…1272) Wrap gv.NumberFormat/DateFormat construction in a try/catch to prevent uncaught 'Too many percent/permill' errors from invalid ICU patterns. Show a localised error message inline below the format input field and clear it automatically when the pattern becomes valid.
* chore(dev): remove legacy block editor code * chore(dev): remove legacy block editor code * chore(dev): use wp-scripts instead of webpack * chore: update actions * chore: update baseline * chore: update baseline * chore: update baseline * chore: update baseline * chore: move deps
* feat: add Elementor widget for displaying Visualizer charts - Register Visualizer Chart widget via `elementor/widgets/register` - Widget renders chart via shortcode in editor/preview context with lazy loading disabled and action buttons suppressed - Add 6 Playwright e2e tests covering widget discovery, no-charts notice, chart selection, rendering, hide-panel regression, and mid-session add * chore: fix phpcs
Initialize CodeMirror on the manual configuration textarea with syntax highlighting, line numbers, bracket matching, and auto-close brackets Closes #509
Bumps [codeinwp/themeisle-sdk](https://github.com/Codeinwp/themeisle-sdk) from 3.3.49 to 3.3.50. - [Release notes](https://github.com/Codeinwp/themeisle-sdk/releases) - [Changelog](https://github.com/Codeinwp/themeisle-sdk/blob/v3.3.50/CHANGELOG.md) - [Commits](Codeinwp/themeisle-sdk@v3.3.49...v3.3.50) --- updated-dependencies: - dependency-name: codeinwp/themeisle-sdk dependency-version: 3.3.50 dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Add Visualizer_Source_Xlsx for local .xlsx file parsing via OpenSpout
The scroll listener for lazy charts never fired if the chart was already visible on page load, causing it to remain blank until the user scrolled. Extracted the logic into a named function with a viewport bounds check and call it once at page load in addition to on scroll.
…#1284) * feat: improve import from WordPress and database UX with stepped flow - Rename "Create Filter"/"Create Query" buttons to "1. Choose Data Source" - Add "2. Set Sync Schedule" accordion toggle (always visible, animated chevron) - Move schedule selector under step-2 toggle for cleaner stepped UX * fix: qa issues
* fix: pro upsell appearing in plugin meta even when pro is there * fix: some charts showing up as available for free * fix: rate it not always visible * fix: make License link point to the exact option on the page * fix: update docs search link * fix: e2e tests * chore: fix qa
…1286) * feat(dev): add action scheduler * chore: update skills * chore: phpstan fix
#1285) * feat: add list view toggle to chart library with user meta persistence Adds a grid/list view toggle to the Visualizer library page. List view renders charts in a WP-style table. The selected view is saved to user meta so it persists across visits without a redirect. - List view: WP table layout, no canvas rendering, tooltips on actions - Shortcode cell is clickable to copy to clipboard with visual feedback - View preference saved via update_user_meta() with allowlist validation - 10 Playwright e2e tests covering toggle, persistence, and table content * fix: qa issues * chore: fix qa * chore fix: qa
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fixes