strepsirrhini/libreoffice-online
3
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

NB: collpasing tabs: hide container instead of child

Browse source 
We can hid the whole container and avoid the background of that
container to be visible (when collpased)
 - This was useful in the before when the tabs were visually "connected"
 to the content
 - Also it was done like that before to give a "clue" to the user that
 something is there and can be expanded but probably by now it's not
 needed and if the user actively collpased them then he/she would know
 how to expand them

Signed-off-by: Pedro Pinto Silva <pedro.silva@collabora.com>
Change-Id: Ib586f4aa3e83e650276b25c2ac13d45f475816e3
This commit is contained in:
Pedro Pinto Silva 2022-02-28 12:04:43 +01:00 committed by pedropintosilva
parent 725c0dc582
commit 30e6538453
1 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
browser/src/control/Control.UIManager.js
View file

@ -496,7 +496,7 @@ L.Control.UIManager = L.Control.extend({
return;
this.moveObjectVertically($('#formulabar'), -1);
$('#toolbar-up').css('display', 'none');
$('#toolbar-wrapper').css('display', 'none');
$('#document-container').addClass('tabs-collapsed');
@ -508,7 +508,7 @@ L.Control.UIManager = L.Control.extend({
return;
this.moveObjectVertically($('#formulabar'), 1);
$('#toolbar-up').css('display', '');
$('#toolbar-wrapper').css('display', '');
$('#document-container').removeClass('tabs-collapsed');