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

Jump to bottom

✅ 🐞 Bug Report: Border hover transition duration slider not available until 'NORMAL' state border is set to a type other than 'None' #11723

Closed
5 tasks done
manyfew opened this issue Jun 23, 2020 · 1 comment
Closed
5 tasks done

✅ 🐞 Bug Report: Border hover transition duration slider not available until 'NORMAL' state border is set to a type other than 'None' #11723

manyfew opened this issue Jun 23, 2020 · 1 comment
Labels
bug Indicates a bug with one or multiple components. mod* solved Indicates that an Issue has been Solved, or a Feature Request has been Released. status/merged Indicates when a Pull Request has been merged to a Release. type/styles Indicates when a topic is related to the styles or design of a component. type/ui-ux Indicates when a topic is related to the User Interface (UI), or User Experience (UX).
Milestone
3.9.0

Comments

@manyfew
Copy link

manyfew commented Jun 23, 2020 •
edited

Prerequisites

  • I have searched for similar issues in both open and closed tickets and cannot find a duplicate.
  • The issue still exists against the latest stable version of Elementor.

Description

The problem is that the border transition duration slider is not shown for Section and Column elements unless the 'Normal' border is set to a type other than 'None'.

This is problematic when you want a border on hover but none when not on hover. Or if you want a box shadow on hover only.

Steps to reproduce

  1. Add a section
  2. Add a widget e.g. a Image
  3. Edit Section > Style > Border > Normal, set type to 'None'
  4. Edit Section > Style > Border > Hover (type not important but you can set it to 'Solid')

Expected behaviour: Transition duration slider is available
Observed behaviour: Transition duration slider is NOT available

Isolating the problem

  • This bug happens with only Elementor plugin active (and Elementor Pro).
  • This bug happens with a default WordPress theme active.
  • I can reproduce this bug consistently using the steps above.

Environment

System Info ```

== Server Environment ==
Operating System: Linux
Software: Apache/2.4.43 (cPanel) OpenSSL/1.1.1g mod_bwlimited/1.4 Phusion_Passenger/5.3.7
MySQL version: MySQL Community Server (GPL) v5.7.30
PHP Version: 7.2.31
PHP Max Input Vars: 10000
PHP Max Post Size: 512M
GD Installed: Yes
ZIP Installed: Yes
Write Permissions: All right
Elementor Library: Connected

== WordPress Environment ==
Version: 5.4.2
Site URL: https://maxforming.com
Home URL: https://maxforming.com
WP Multisite: No
Max Upload Size: 512 MB
Memory limit: 512M
Permalink Structure: /%year%/%monthnum%/%day%/%postname%/
Language: en-US
Timezone: 0
Debug Mode: Inactive

== Theme ==
Name: Hello Elementor
Version: 2.3.0
Author: Elementor Team
Child Theme: No

== User ==
Role: administrator
WP Profile lang: en_US
User Agent: Mozilla/5.0 (Windows NT 10.0; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/81.0.4044.138 Safari/537.36

== Active Plugins ==
Elementor
Version: 2.9.13
Author: Elementor.com

== Elements Usage ==

== Log ==

Log: showing 1 of 12020-06-23 22:48:35 [info] Elementor data updater process has been completed. [array (
'plugin' => 'Elementor',
'from' => '2.9.12',
'to' => '2.9.13',
)]

</details>
@rami-elementor rami-elementor mentioned this issue Oct 26, 2022
Merged
12 tasks
@rami-elementor rami-elementor added type/ui-ux Indicates when a topic is related to the User Interface (UI), or User Experience (UX). status/has-pr Indicates that an Issue, or Discussion has a companion Pull Request awaiting to be merged. labels Oct 26, 2022
KingYes pushed a commit that referenced this issue Oct 27, 2022
@ramiy
Fix: Separate "Default" and "None" values in "Border Type" [ED-8511] (#…
411e865 
…20145)

## PR Checklist
<!-- 
Please check if your PR fulfills the following requirements:
**Filling out the template is required.** Any pull request that does not
include enough information to be reviewed in a timely manner may be
closed at the maintainers' discretion.
 -->
- [x] The commit message follows our guidelines:
https://github.com/elementor/elementor/blob/master/.github/CONTRIBUTING.md


## PR Type
What kind of change does this PR introduce?
<!-- Please check the one that applies to this PR using "x" with no
spaces eg: [x]. -->
- [x] Bugfix
- [ ] Feature
- [ ] Code style update (formatting, local variables)
- [ ] Refactoring (no functional changes, no api changes)
- [ ] Build related changes
- [ ] CI related changes
- [ ] Documentation content changes
- [ ] Other... Please describe:

## Summary

This PR can be summarized in the following changelog entry:

* Fix: Separate "Default" and "None" values in "Border Type"

## Description
An explanation of what is done in this PR

* This PR fixes several bug. It changes the initial label to "Default".
In addition it adds the ability to set border none.

## Test instructions
This PR can be tested by following these steps:

*

## Quality assurance

- [x] I have tested this code to the best of my abilities
- [ ] I have added unittests to verify the code works as intended
- [ ] Docs have been added / updated (for bug fixes / features)

Fixes: #11565, #13328, #11723
@rami-elementor rami-elementor added this to the 3.9.0 milestone Oct 27, 2022
@rami-elementor rami-elementor added solved Indicates that an Issue has been Solved, or a Feature Request has been Released. status/merged Indicates when a Pull Request has been merged to a Release. and removed status/has-pr Indicates that an Issue, or Discussion has a companion Pull Request awaiting to be merged. solved Indicates that an Issue has been Solved, or a Feature Request has been Released. labels Oct 27, 2022
@nicholaszein nicholaszein changed the title Border hover transition duration slider not available until 'NORMAL' state border is set to a type other than 'None' ✔️ 🐞 Bug Report: Border hover transition duration slider not available until 'NORMAL' state border is set to a type other than 'None' Dec 7, 2022
@nicholaszein
Copy link
Contributor

Hello there @manyfew!

We have great news! 🙌

📢 We're happy to announce that the issue you raised was resolved in Elementor v3.9.0! 🥳

Thank you so much for reporting this bug, and for being so patient! 🙏

✔️ Please feel free to check it out and update your plugin to the new version!

Cheers 🥂

@nicholaszein nicholaszein added bug Indicates a bug with one or multiple components. solved Indicates that an Issue has been Solved, or a Feature Request has been Released. component/design and removed status/merged Indicates when a Pull Request has been merged to a Release. labels Dec 7, 2022
@nicholaszein nicholaszein changed the title ✔️ 🐞 Bug Report: Border hover transition duration slider not available until 'NORMAL' state border is set to a type other than 'None' ✅ 🐞 Bug Report: Border hover transition duration slider not available until 'NORMAL' state border is set to a type other than 'None' May 24, 2023
@elementor elementor locked and limited conversation to collaborators May 24, 2023
@nicholaszein nicholaszein added the status/merged Indicates when a Pull Request has been merged to a Release. label May 24, 2023
@nicholaszein nicholaszein added the type/styles Indicates when a topic is related to the styles or design of a component. label Jan 11, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
bug Indicates a bug with one or multiple components. mod* solved Indicates that an Issue has been Solved, or a Feature Request has been Released. status/merged Indicates when a Pull Request has been merged to a Release. type/styles Indicates when a topic is related to the styles or design of a component. type/ui-ux Indicates when a topic is related to the User Interface (UI), or User Experience (UX).
Projects
None yet
Milestone
3.9.0
Development

No branches or pull requests
3 participants
@manyfew @rami-elementor @nicholaszein