Browse Source

Default the "always on status bar" to off for sanity's sake.

Remove the option to "toggle" it from the menu, because it never did
anything when set to always on, and it'd never be useful to close it
from there as opposed to the "x", which also doesn't work with
"always on".

You'll have to deselect "always on status bar" for your already existing
services.

Closes #15
pull/3202/head
TheGoddessInari 7 years ago
parent
commit
fb4e634cb2
  1. 7
      app.js
  2. 2
      app/view/add/Add.js
  3. 9
      electron/menu.js

7
app.js

@ -130,14 +130,7 @@ ipc.on('reloadCurrentService', function(e) {
var tab = Ext.cq1('app-main').getActiveTab();
if ( tab.id !== 'ramboxTab' ) tab.reloadService();
});
// Toggle Status Bar
ipc.on('toggleStatusBar', function() {
var tab = Ext.cq1('app-main').getActiveTab();
if ( tab.id !== 'ramboxTab' ) {
tab.down('statusbar').closed ? tab.setStatusBar(tab.record.get('statusbar')) : tab.closeStatusBar();
}
});
// Focus the current service when Alt + Tab or click in webviews textfields
window.addEventListener('focus', function() {
if(Ext.cq1("app-main")) Ext.cq1("app-main").getActiveTab().down('component').el.dom.focus();

2
app/view/add/Add.js

@ -182,7 +182,7 @@ Ext.define('Rambox.view.add.Add',{
xtype: 'checkbox'
,boxLabel: 'Always display Status Bar'
,name: 'statusbar'
,checked: me.edit ? me.record.get('statusbar') : true
,checked: me.edit ? me.record.get('statusbar') : false
,uncheckedValue: false
,inputValue: true
}

9
electron/menu.js

@ -163,15 +163,6 @@ module.exports = function(config) {
{
type: 'separator'
},
{
label: '&Toggle Status Bar',
click() {
sendAction('toggleStatusBar');
}
},
{
type: 'separator'
},
{
role: 'zoomin'
},

Loading…
Cancel
Save