From f331daffe197a4a14a5e7e8dc17240f0b69a46d9 Mon Sep 17 00:00:00 2001
From: Aleksander Machniak <alec@alec.pl>
Date: Thu, 19 Jun 2014 06:29:01 -0400
Subject: [PATCH] Fix preview toggle button (missing iconbutton class after click)
---
skins/larry/mail.css | 2 +-
skins/larry/ui.js | 10 ++++++----
2 files changed, 7 insertions(+), 5 deletions(-)
diff --git a/skins/larry/mail.css b/skins/larry/mail.css
index 85da974..5e2677f 100644
--- a/skins/larry/mail.css
+++ b/skins/larry/mail.css
@@ -460,7 +460,7 @@
#mailpreviewtoggle {
display: block;
position: absolute;
- top: 6px;
+ top: 7px;
right: 4px;
width: 20px;
height: 18px;
diff --git a/skins/larry/ui.js b/skins/larry/ui.js
index c843750..eb9569f 100644
--- a/skins/larry/ui.js
+++ b/skins/larry/ui.js
@@ -218,9 +218,11 @@
}
else if (rcmail.env.action == 'list' || !rcmail.env.action) {
var previewframe = $('#mailpreviewframe').is(':visible');
- $('#mailpreviewtoggle').addClass(previewframe ? 'enabled' : 'closed').attr('aria-expanded', previewframe ? 'true' : 'false').click(function(e){ toggle_preview_pane(e); return false });
- $('#maillistmode').addClass(rcmail.env.threading ? '' : 'selected').click(function(e){ switch_view_mode('list'); return false });
- $('#mailthreadmode').addClass(rcmail.env.threading ? 'selected' : '').click(function(e){ switch_view_mode('thread'); return false });
+
+ $('#mailpreviewtoggle').addClass(previewframe ? 'enabled' : 'closed').attr('aria-expanded', previewframe ? 'true' : 'false')
+ .click(function(e) { toggle_preview_pane(e); return false; });
+ $('#maillistmode').addClass(rcmail.env.threading ? '' : 'selected').click(function(e) { switch_view_mode('list'); return false; });
+ $('#mailthreadmode').addClass(rcmail.env.threading ? 'selected' : '').click(function(e) { switch_view_mode('thread'); return false; });
mailviewsplit = new rcube_splitter({ id:'mailviewsplitter', p1:'#mailview-top', p2:'#mailview-bottom',
orientation:'h', relative:true, start:310, min:150, size:12, offset:4 });
@@ -595,7 +597,7 @@
topstyles, bottomstyles, uid;
frame.toggle();
- button.removeClass().toggleClass('enabled closed').attr('aria-expanded', visible ? 'true' : 'false');
+ button.toggleClass('enabled closed').attr('aria-expanded', visible ? 'true' : 'false');
if (visible) {
$('#mailview-top').removeClass('fullheight').css({ bottom:'auto' });
--
Gitblit v1.9.1