Fix preview toggle button (missing iconbutton class after click)

Aleksander Machniak

2014-06-19 f331daffe197a4a14a5e7e8dc17240f0b69a46d9

Fix preview toggle button (missing iconbutton class after click)

 skins/larry/mail.css

@@ -460,7 +460,7 @@
#mailpreviewtoggle {
    display: block;
    position: absolute;
    top: 6px;
    top: 7px;
    right: 4px;
    width: 20px;
    height: 18px;

 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' });