Skip to content

EPS-1483: New Post duplicator feature #1401

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
ShubhamGupta05 wants to merge 4 commits into
base: release-candidate
Choose a base branch
from
+439 −0
Open

EPS-1483: New Post duplicator feature #1401

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
c08e124
Added post duplicator functionality
ShubhamGupta05 Jul 11, 2025
b936794
Merge branch 'release-candidate' into post-duplicator
ShubhamGupta05 Dec 12, 2025
066acb2
Fixed the original post not editing in elementor when duplicate post …
ShubhamGupta05 Dec 17, 2025
08b0806
Added white label support when pro plugin is active
ShubhamGupta05 Dec 19, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
27 changes: 27 additions & 0 deletions assets/css/admin-post-duplicator.css
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,27 @@
/**
Copy link

@github-actions github-actions bot Dec 17, 2025

Choose a reason for hiding this comment

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

What: Consider specifying a version for the CSS file if applicable.

Why: Versioning helps in managing assets and ensures that users receive the correct style updates. It aids in cache management.

How: Add a comment at the top of the CSS file indicating the version, e.g., /* Version 1.0.0 */. This practice can be integrated with your build process or asset management approach.

* Admin styles for Post Duplicator.
*
Copy link

@github-actions github-actions bot Dec 17, 2025

Choose a reason for hiding this comment

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

What: Consider adding more context to the comment above the .hfe-post-duplicator-notice class.

Why: While the comment generally states it's for the Post Duplicator, including what the notice signifies (e.g., success or error messages) can improve maintainability and understanding for future developers.

How: Change the comment to something like: /* Styles for success/error notices related to the Post Duplicator feature. */.

* @package header-footer-elementor
Copy link

@github-actions github-actions bot Dec 17, 2025

Choose a reason for hiding this comment

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

What: The color definitions used here may benefit from being stored as CSS variables for easier theming or future changes.

Why: Utilizing CSS variables enhances maintainability and makes it easier to apply changes if the color scheme requires updates later on.

How: Define CSS variables in a root selector, like :root { --primary-color: #0073aa; --hover-color: #00a0d2; }, then use these variables in your styles, e.g., color: var(--primary-color);.

*/

.hfe-post-duplicator-notice {
padding: 10px 15px;
background-color: #fff;
border-left: 4px solid #00a0d2;
margin: 5px 0 15px;
box-shadow: 0 1px 1px rgba(0, 0, 0, 0.04);
}

.hfe-post-duplicator-notice p {
margin: 0.5em 0;
padding: 2px;
}

/* Style for the UA Duplicate link in row actions */
.row-actions .hfe_duplicate a {
color: #0073aa;
}

.row-actions .hfe_duplicate a:hover {
color: #00a0d2;
}
1 change: 1 addition & 0 deletions inc/class-header-footer-elementor.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -635,6 +635,7 @@ public function includes() {

require_once HFE_DIR . 'inc/hfe-functions.php';
Copy link

@github-actions github-actions bot Dec 17, 2025

Choose a reason for hiding this comment

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

What: The addition of the Post Duplicator class appears to be correctly included, but I recommend considering the implications of loading this class for all users who access the admin interface. If this class introduces heavy operations or data processing, it could impact performance.

Why: Loading unnecessary classes can lead to increased memory usage and slower performance, particularly for users who do not utilize the duplication feature. This is crucial for maintaining a responsive admin experience, especially if many users will be accessing the admin interface.

How: To improve performance, consider utilizing a conditional loading mechanism. For example, load the class-hfe-post-duplicator.php file only if the user is accessing a specific admin page or if the feature is enabled in the settings. This can be implemented using a check like if (isset($_GET['page']) && $_GET['page'] === 'desired_page_name'). This reduces unnecessary overhead for users who do not need the duplication functionality.

require_once HFE_DIR . 'inc/class-hfe-rollback.php';
require_once HFE_DIR . 'inc/class-hfe-post-duplicator.php';

// Load Elementor Canvas Compatibility.
require_once HFE_DIR . 'inc/class-hfe-elementor-canvas-compat.php';
Expand Down
Loading