wassname/fullcalendar
1
0
Fork 0
mirror of https://github.com/wassname/fullcalendar.git synced 2026-08-12 12:00:40 +08:00
Code Issues Packages Projects Releases Wiki Activity

fix bug where hover className wouldnt be removed from disabled button. issue 2235

Browse Source
This commit is contained in:
Adam Shaw
2014-08-19 15:26:57 -07:00
parent 8e6a68ccf0
commit d8794a6a25
1 changed files with 23 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/Header.js
+23 -6
View File
@@ -69,13 +69,13 @@ function Header(calendar, options) {
isOnlyButtons = false;
}
else {
if (calendar[buttonName]) {
buttonClick = calendar[buttonName]; // calendar method
// NOTE: won't work when we move away from parasitic inheritance
if (calendar[buttonName]) { // a calendar method
buttonClick = function() {
calendar[buttonName]();
};
}
else if (fcViews[buttonName]) {
else if (fcViews[buttonName]) { // a view name
buttonClick = function() {
button.removeClass(tm + '-state-hover'); // forget why
calendar.changeView(buttonName);
};
viewsWithButtons.push(buttonName);
@@ -113,30 +113,47 @@ function Header(calendar, options) {
'</button>'
)
.click(function() {
// don't process clicks for disabled buttons
if (!button.hasClass(tm + '-state-disabled')) {
buttonClick();
// after the click action, if the button becomes the "active" tab, or disabled,
// it should never have a hover class, so remove it now.
if (
button.hasClass(tm + '-state-active') ||
button.hasClass(tm + '-state-disabled')
) {
button.removeClass(tm + '-state-hover');
}
}
})
.mousedown(function() {
// the *down* effect (mouse pressed in).
// only on buttons that are not the "active" tab, or disabled
button
.not('.' + tm + '-state-active')
.not('.' + tm + '-state-disabled')
.addClass(tm + '-state-down');
})
.mouseup(function() {
// undo the *down* effect
button.removeClass(tm + '-state-down');
})
.hover(
function() {
// the *hover* effect.
// only on buttons that are not the "active" tab, or disabled
button
.not('.' + tm + '-state-active')
.not('.' + tm + '-state-disabled')
.addClass(tm + '-state-hover');
},
function() {
// undo the *hover* effect
button
.removeClass(tm + '-state-hover')
.removeClass(tm + '-state-down');
.removeClass(tm + '-state-down'); // if mouseleave happens before mouseup
}
);