[Libreoffice-commits] online.git: loleaflet/css loleaflet/images loleaflet/src
Libreoffice Gerrit user
logerrit at kemper.freedesktop.org
Thu May 23 12:34:12 UTC 2019
loleaflet/css/toolbar.css | 1 +
loleaflet/images/lc_backgroundcolor.svg | 1 +
loleaflet/src/control/Control.Toolbar.js | 23 ++++++++++++++++++-----
loleaflet/src/unocommands.js | 1 +
4 files changed, 21 insertions(+), 5 deletions(-)
New commits:
commit 6167dc9c44b783cfac0062755a1f1fc73114b538
Author: Samuel Mehrbrodt <Samuel.Mehrbrodt at cib.de>
AuthorDate: Wed May 22 16:39:49 2019 +0200
Commit: Samuel Mehrbrodt <Samuel.Mehrbrodt at cib.de>
CommitDate: Thu May 23 14:33:52 2019 +0200
Disambiguate Highlight color vs background color
Highlighting is character level while background color is paragraph/cell level.
Use different uno commands/icons/texts for the two.
Change-Id: I79264fcdcd968734b68278f69450a377d6607323
Reviewed-on: https://gerrit.libreoffice.org/72786
Reviewed-by: Samuel Mehrbrodt <Samuel.Mehrbrodt at cib.de>
Tested-by: Samuel Mehrbrodt <Samuel.Mehrbrodt at cib.de>
diff --git a/loleaflet/css/toolbar.css b/loleaflet/css/toolbar.css
index 29a9aa472..5db1821ea 100644
--- a/loleaflet/css/toolbar.css
+++ b/loleaflet/css/toolbar.css
@@ -550,6 +550,7 @@ button.leaflet-control-search-next
.w2ui-icon.strikeout{ background: url('images/lc_strikeout.svg') no-repeat center !important; }
.w2ui-icon.textcolor{ background: url('images/lc_color.svg') no-repeat center !important; }
.w2ui-icon.backcolor{ background: url('images/lc_backcolor.svg') no-repeat center !important; }
+.w2ui-icon.backgroundcolor{ background: url('images/lc_backgroundcolor.svg') no-repeat center !important; }
.w2ui-icon.alignleft{ background: url('images/lc_alignleft.svg') no-repeat center !important; }
.w2ui-icon.alignhorizontal{ background: url('images/lc_alignhorizontalcenter.svg') no-repeat center !important; }
.w2ui-icon.alignright{ background: url('images/lc_alignright.svg') no-repeat center !important; }
diff --git a/loleaflet/images/lc_backgroundcolor.svg b/loleaflet/images/lc_backgroundcolor.svg
new file mode 100644
index 000000000..ebd7aac9d
--- /dev/null
+++ b/loleaflet/images/lc_backgroundcolor.svg
@@ -0,0 +1 @@
+<svg viewBox="0 0 24 24" xmlns="http://www.w3.org/2000/svg"><path d="m11.307716 20.638247 9.438078-5.638247-5.708126-10.3752563-9.9336041 5.8220523z" fill="#fff" fill-rule="evenodd"/><ellipse cx="12" cy="19.725481" fill="none" rx="10" ry="2.274613"/><path d="m9 8h-5c-1 0-1 .217546-1 2.276478v5.723522c0 1 2 1 2 0v-6z" fill="#4d82b8" fill-rule="evenodd"/><g fill="none" stroke="#808080"><ellipse cx="4.962672" cy="16.187546" rx="1.752345" ry="1.741017" stroke-linecap="round" stroke-width=".95097" transform="matrix(.86821741 -.49618397 .50382625 .86380502 0 0)"/><path d="m12.488244 9.8063594v-5.2263907c0-2.8972803 5.493132-2.8080105 5.493132.1206297v4.9056967" stroke-linecap="round" stroke-width=".909468"/><path d="m5.1040639 10.446796 5.3390121 9.387521c.410089.65506 1.069393.906864 1.704955.572463l7.748423-4.541322c.589382-.407352.86271-1.172428.486324-1.842477l-5.34511-9.3982373z" stroke-width=".896453"/></g></svg>
\ No newline at end of file
diff --git a/loleaflet/src/control/Control.Toolbar.js b/loleaflet/src/control/Control.Toolbar.js
index 7161f800f..b8028da40 100644
--- a/loleaflet/src/control/Control.Toolbar.js
+++ b/loleaflet/src/control/Control.Toolbar.js
@@ -230,6 +230,9 @@ function onClick(e, id, item, subItem) {
else if (id === 'backcolor' && typeof e.color !== 'undefined') {
onColorPick(id, e.color)
}
+ else if (id === 'backgroundcolor' && typeof e.color !== 'undefined') {
+ onColorPick(id, e.color)
+ }
else if (id === 'sum') {
map.sendUnoCommand('.uno:AutoSum');
}
@@ -637,15 +640,24 @@ function onColorPick(id, color) {
command[fontcolor].value = color;
var uno = '.uno:' + fontcolor;
}
+ // "backcolor" can be used in Writer and Impress and translates to "Highlighting" while
+ // "backgroundcolor" can be used in Writer and Calc and translates to "Background color".
else if (id === 'backcolor') {
backcolor = {'text': 'BackColor',
- 'spreadsheet': 'BackgroundColor',
'presentation': 'CharBackColor'}[map.getDocType()];
command[backcolor] = {};
command[backcolor].type = 'long';
command[backcolor].value = color;
uno = '.uno:' + backcolor;
}
+ else if (id === 'backgroundcolor') {
+ backcolor = {'text': 'BackgroundColor',
+ 'spreadsheet': 'BackgroundColor'}[map.getDocType()];
+ command[backcolor] = {};
+ command[backcolor].type = 'long';
+ command[backcolor].value = color;
+ uno = '.uno:' + backcolor;
+ }
map.sendUnoCommand(uno, command);
map.focus();
}
@@ -746,7 +758,8 @@ function initNormalToolbar() {
{type: 'button', id: 'strikeout', img: 'strikeout', hint: _UNO('.uno:Strikeout'), uno: 'Strikeout'},
{type: 'break', id: 'breakformatting'},
{type: 'text-color', id: 'fontcolor', img: 'textcolor', hint: _UNO('.uno:FontColor')},
- {type: 'color', id: 'backcolor', img: 'backcolor', hint: _UNO('.uno:BackgroundColor')},
+ {type: 'color', id: 'backcolor', img: 'backcolor', hint: _UNO('.uno:BackColor'), hidden: true},
+ {type: 'color', id: 'backgroundcolor', img: 'backgroundcolor', hint: _UNO('.uno:BackgroundColor'), hidden: true},
{type: 'break' , id: 'breakcolor', mobile:false},
{type: 'button', id: 'leftpara', img: 'alignleft', hint: _UNO('.uno:LeftPara', '', true), uno: 'LeftPara', hidden: true, unosheet: 'AlignLeft', disabled: true},
{type: 'button', id: 'centerpara', img: 'alignhorizontal', hint: _UNO('.uno:CenterPara', '', true), uno: 'CenterPara', hidden: true, unosheet: 'AlignHorizontalCenter', disabled: true},
@@ -1395,7 +1408,7 @@ function onDocLayerInit() {
toolbarUp.show('textalign', 'wraptext', 'breakspacing', 'insertannotation', 'conditionalformaticonset',
'numberformatcurrency', 'numberformatpercent',
'numberformatincdecimals', 'numberformatdecdecimals', 'break-number', 'togglemergecells', 'breakmergecells',
- 'setborderstyle', 'sortascending', 'sortdescending', 'breaksorting');
+ 'setborderstyle', 'sortascending', 'sortdescending', 'breaksorting', 'backgroundcolor');
toolbarUp.remove('styles');
statusbar.remove('prev', 'next', 'prevnextbreak');
@@ -1462,7 +1475,7 @@ function onDocLayerInit() {
case 'text':
toolbarUp.show('leftpara', 'centerpara', 'rightpara', 'justifypara', 'breakpara', 'linespacing',
'breakspacing', 'defaultbullet', 'defaultnumbering', 'breakbullet', 'incrementindent', 'decrementindent',
- 'breakindent', 'inserttable', 'insertannotation');
+ 'breakindent', 'inserttable', 'insertannotation', 'backcolor');
if (!_inMobileMode()) {
statusbar.insert('left', [
@@ -1520,7 +1533,7 @@ function onDocLayerInit() {
// FALLTHROUGH intended
case 'drawing':
toolbarUp.show('leftpara', 'centerpara', 'rightpara', 'justifypara', 'breakpara', 'linespacing',
- 'breakspacing', 'defaultbullet', 'defaultnumbering', 'breakbullet', 'inserttable');
+ 'breakspacing', 'defaultbullet', 'defaultnumbering', 'breakbullet', 'inserttable', 'backcolor');
statusbar.show('prev', 'next');
$('#presentation-toolbar').show();
diff --git a/loleaflet/src/unocommands.js b/loleaflet/src/unocommands.js
index cb6c0c1d9..a0e65bffb 100644
--- a/loleaflet/src/unocommands.js
+++ b/loleaflet/src/unocommands.js
@@ -11,6 +11,7 @@ var unoCommandsArray = {
ArrangeFrameMenu:{text:{menu:_('Arrange'),},},
ArrangeMenu:{global:{menu:_('A~rrange'),},presentation:{menu:_('~Arrange'),},},
AutoFormatMenu:{text:{menu:_('AutoCorr~ect'),},},
+ BackColor:{text:{menu:_('Highlight Color'),},},
BackgroundColor:{global:{menu:_('Background Color'),},},
Bold:{global:{menu:_('Bold'),},},
BringToFront:{global:{menu:_('~Bring to Front'),},},
More information about the Libreoffice-commits
mailing list