Browse Source

Add popupVisible className setting

pull/5305/head
Jack Lukic 7 years ago
parent
commit
55508be387
2 changed files with 14 additions and 10 deletions
  1. 3
      RELEASE-NOTES.md
  2. 21
      src/definitions/modules/popup.js

3
RELEASE-NOTES.md

@ -14,6 +14,9 @@
- **Menu** - Fixed issue where `left menu` and `right menu` did not display correctly in `stackable menu` on mobile #3604 #5116 **Thanks @BleuDiamant @Traverse**
- **Dimmer** - Fixed typo causing body dimmer to add unnecessary `position: relative;` **Thanks @jinyangzhen** #4707
**Enhancements**
- **Popup** - Separated className setting for `visible` into `visible` and `popupVisible`, this way you can remove visible indiciation on activating element without modifying popup.
### Version 2.2.10 - March 28, 2017
**Critical Bugs**

21
src/definitions/modules/popup.js

@ -360,7 +360,7 @@ $.fn.popup = function(parameters) {
hideAll: function() {
$(selector.popup)
.filter('.' + className.visible)
.filter('.' + className.popupVisible)
.each(function() {
$(this)
.data(metadata.activator)
@ -1078,7 +1078,7 @@ $.fn.popup = function(parameters) {
return ($popup !== undefined && $popup.hasClass(className.fluid));
},
visible: function() {
return ($popup !== undefined && $popup.hasClass(className.visible));
return ($popup !== undefined && $popup.hasClass(className.popupVisible));
},
dropdown: function() {
return $module.hasClass(className.dropdown);
@ -1424,14 +1424,15 @@ $.fn.popup.settings = {
},
className : {
active : 'active',
animating : 'animating',
dropdown : 'dropdown',
fluid : 'fluid',
loading : 'loading',
popup : 'ui popup',
position : 'top left center bottom right',
visible : 'visible'
active : 'active',
animating : 'animating',
dropdown : 'dropdown',
fluid : 'fluid',
loading : 'loading',
popup : 'ui popup',
position : 'top left center bottom right',
visible : 'visible',
popupVisible : 'visible'
},
selector : {

Loading…
Cancel
Save