haraldwolff
/
collabora-online
mirror of https://github.com/CollaboraOnline/online.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Slide sorter: presentation toolbar: Remove ui break 

There is no need to add a visual break/divider between "Fullscreen
presentation" and the "Insert Slide" buttons:
1. They are visually very well distinctive so there seems to be not a
need for a division
2. There is not so much white space so the more we can reduce visually
the better

Signed-off-by: Pedro Pinto Silva <pedro.silva@collabora.com>
Change-Id: I27b84782e5af987d9aec3734f7557c746a03667c
pull/8999/head
Pedro Pinto Silva 2024-05-06 14:46:03 +02:00 committed by pedropintosilva
parent c54683f349
commit 45c51c42de
1 changed files with 0 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
browser/src/control/Control.PresentationBar.js
View File

@ -54,11 +54,6 @@ class PresentationBar {
text: this._getItemUnoName('presentation'),
command: 'presentation'
},
{
id: 'presentationbreak',
type: 'separator',
orientation: 'vertical',
},
{
id: 'insertpage',
type: 'customtoolitem',
@ -100,7 +95,6 @@ class PresentationBar {
if (this.map.getDocType() === 'drawing') {
this.showItem('presentation', false);
this.showItem('presentationbreak', false);
}
}
@ -144,14 +138,12 @@ class PresentationBar {
onWopiProps(e) {
if (e.HideExportOption) {
this.showItem('presentation', false);
this.showItem('presentationbreak', false);
}
}
onDocLayerInit() {
if (!this.map['wopi'].HideExportOption && this.map.getDocType() !== 'drawing') {
this.showItem('presentation', true);
this.showItem('presentationbreak', true);
}
if (!window.mode.isMobile())