Skip to content

fix: various issues#1282

Open
HardeepAsrani wants to merge 6 commits intodevelopmentfrom
fix/various
Open

fix: various issues#1282
HardeepAsrani wants to merge 6 commits intodevelopmentfrom
fix/various

Conversation

@HardeepAsrani
Copy link
Member

Summary

This PR addresses various small issues:

In addition, this PR was also made to add search support to new docs site:

Will affect visual aspect of the product

YES/NO

Screenshots

Test instructions

  • Check the PR to make sure each issue was been addresses.

Check before Pull Request is ready:

@HardeepAsrani HardeepAsrani requested a review from abaicus March 16, 2026 10:55
@HardeepAsrani HardeepAsrani added the pr-checklist-skip Allow this Pull Request to skip checklist. label Mar 16, 2026
@pirate-bot pirate-bot added the pr-checklist-complete The Pull Request checklist is complete. (automatic label) label Mar 16, 2026
@HardeepAsrani HardeepAsrani changed the title Fix/various fix: various issues Mar 16, 2026
@pirate-bot
Copy link
Contributor

Plugin build for 4395c7d is ready 🛎️!

Comment on lines +1210 to 1215
$is_pro_filter = apply_filters( 'visualizer_is_pro', false );

// Pro 1.9.0+: filter is hooked and returns true only with a valid license.
if ( $is_pro_filter ) {
return true;
}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am not entirely sure about the semantics of this method. Its name is proFeaturesLocked, but it seems to return true when the pro filter returns true.

As per the comment: // Pro 1.9.0+: filter is hooked and returns true only with a valid license.

So I'm not sure if it says that pro features are locked, or unlocked. The fallthrough below seems to be coherent in this regard though.

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

Labels

pr-checklist-complete The Pull Request checklist is complete. (automatic label) pr-checklist-skip Allow this Pull Request to skip checklist.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants