Skip to content
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.

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

EPS-1056: Removed deprecated class .elementor-button-wrapper #1062

Open
wants to merge 10 commits into
base: release-candidate
Choose a base branch
from

Conversation

vijetaR
Copy link
Contributor

@vijetaR vijetaR commented Jan 29, 2025

Description

[BSF-PR-SUMMARY]

Removed deprecated class .elementor-button-wrapper

Widget

  • Custom cart
  • Navigation menu - All Layout with CTA button

Screenshots

Types of changes

Bug fix (non-breaking change which fixes an issue)

How has this been tested?

  1. Drop all the mention widget i.e Cart and Navigation Menu
  2. Check the functionality with CTA button which contain the class mentioned in all the layout and menu alignment and hamburger icon alignment.
  3. The UI and functionality should remain intact on removal of the class
  4. It should look as expected.

Checklist:

  • My code is tested
  • My code passes the PHPCS tests
  • My code follows accessibility standards
  • My code has proper inline documentation
  • I've included any necessary tests
  • I've included developer documentation
  • I've added proper labels to this pull request

@@ -403,8 +403,8 @@ protected function register_general_content_controls() {
],
'selectors' => [
'{{WRAPPER}} li.menu-item a' => 'justify-content: {{VALUE}};',
'{{WRAPPER}} li .elementor-button-wrapper' => 'text-align: {{VALUE}};',
'{{WRAPPER}}.hfe-menu-item-flex-end li .elementor-button-wrapper' => 'text-align: right;',
'{{WRAPPER}} li.hfe-button-wrapper' => 'text-align: {{VALUE}};',
Copy link
Collaborator

Choose a reason for hiding this comment

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

there is a space after li
li .elementor-button-wrapper
confirm if we are replacing .hfe-button-wrapper exactly with .elementor-button-wrapper then space should be addedhere too

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, we are replacing it, checked here - https://prnt.sc/-_kSOsgmCQ8-
It doesn't affect either way, I will add space wrt older ref

readme.txt Outdated
@@ -135,6 +135,7 @@ To access the advanced features and premium widgets, you’ll need to upgrade to
- Fix: "Function map_meta_cap called incorrectly" notice appearing for logged out users when shortcode executed manually.
- Fix: Deprecated constant FILTER_SANITIZE_STRING replaced to ensure compatibility with PHP.
- Fix: Resolved PHP error with NPS library.
- Improvement: Removed 'elementor-button-wrapper' to enhance performance and optimize the DOM structure.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Improvement should be added at the top

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Updated in new dev version

Copy link
Collaborator

@sushmak02 sushmak02 left a comment

Choose a reason for hiding this comment

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

@vijetaR Added comments in the PR

@vijetaR vijetaR requested a review from sushmak02 February 3, 2025 08:33
@@ -625,7 +625,7 @@ protected function render() {
<?php
} else {
?>
<div class="hfe-menu-cart__toggle elementor-button-wrapper">
<div class="hfe-menu-cart__toggle">
<a id="hfe-menu-cart__toggle_button" href="<?php echo esc_url( wc_get_cart_url() ); ?>" class="elementor-button hfe-cart-container" aria-label="<?php esc_attr_e( 'Cart', 'header-footer-elementor' ); ?>">

Choose a reason for hiding this comment

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

Add "hfe-button-wrapper" class here, it is not showing the cart button when we click on it and go to the cart page.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

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

Successfully merging this pull request may close these issues.

3 participants