Skip to content

Commit 212514b

Browse files
jcsparksweb-flow
authored andcommitted
Merge pull request #104 from CuBoulder/issue/103
10.5 Icon Fix
2 parents 44b9948 + 87b8315 commit 212514b

61 files changed

Lines changed: 108 additions & 58 deletions

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

CHANGELOG.md

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,14 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
99

1010
## [Unreleased]
1111

12+
- ### 10.5 Icon Fix
13+
Needed to add the core ckeditor 5 icons to our plugin as ckeditor remove them in 10.5
14+
15+
Sister PR: https://github.com/CuBoulder/ucb_ckeditor_icons/pull/1
16+
17+
Resolves #103
18+
---
19+
1220
- ### D11 version bump
1321
D11 version bump
1422
---

ckeditor5_plugins/box/src/boxconfig.js

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -7,14 +7,16 @@
77
* @property {string} className
88
*/
99

10-
import { icons } from 'ckeditor5/src/core';
1110
import boxTitleLeft from '../../../icons/box-title-left.svg';
1211
import boxTitleCenter from '../../../icons/box-title-center.svg';
1312
import boxTitleRight from '../../../icons/box-title-right.svg';
1413
import boxTitleHidden from '../../../icons/box-title-hidden.svg';
1514
import boxStyleFill from '../../../icons/box-style-fill.svg';
1615
import boxStyleOutline from '../../../icons/box-style-outline.svg';
1716
import boxStyleNone from '../../../icons/box-style-none.svg';
17+
import objectLeft from '../../../icons/object-left.svg';
18+
import objectFullWidth from '../../../icons/object-full-width.svg';
19+
import objectRight from '../../../icons/object-right.svg';
1820

1921
/** @type {Object<string, SelectableOption>} */
2022
export const titleOptions = {
@@ -46,17 +48,17 @@ export const titleDefault = 'left';
4648
export const alignmentOptions = {
4749
left: {
4850
label: 'Float left',
49-
icon: icons.objectLeft,
51+
icon: objectLeft,
5052
className: 'ucb-box-alignment-left'
5153
},
5254
none: {
5355
label: 'Fill width',
54-
icon: icons.objectFullWidth,
56+
icon: objectFullWidth,
5557
className: 'ucb-box-alignment-none'
5658
},
5759
right: {
5860
label: 'Float right',
59-
icon: icons.objectRight,
61+
icon: objectRight,
6062
className: 'ucb-box-alignment-right'
6163
}
6264
};

ckeditor5_plugins/button/src/buttonconfig.js

Lines changed: 4 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,19 +1,17 @@
1-
import { icons } from 'ckeditor5/src/core';
1+
import objectCenter from '../../../icons/object-center.svg';
2+
import objectFullWidth from '../../../icons/object-full-width.svg';
23

34
export const sizeOptions = {
45
large: {
56
label: 'Large',
6-
icon: icons.large,
77
className: 'ucb-link-button-large'
88
},
99
regular: {
1010
label: 'Regular',
11-
icon: icons.regular,
1211
className: 'ucb-link-button-regular'
1312
},
1413
small: {
1514
label: 'Small',
16-
icon: icons.small,
1715
className: 'ucb-link-button-small'
1816
},
1917
};
@@ -48,12 +46,12 @@ export const defaultColor = 'blue';
4846
export const styleOptions = {
4947
default: {
5048
label: 'Default',
51-
icon: icons.objectCenter,
49+
icon: objectCenter,
5250
className: 'ucb-link-button-default'
5351
},
5452
full: {
5553
label: 'Full',
56-
icon: icons.objectFullWidth,
54+
icon: objectFullWidth,
5755
className: 'ucb-link-button-full'
5856
},
5957
};

ckeditor5_plugins/button/src/buttonview.js

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -9,8 +9,9 @@ import {
99
addToolbarToDropdown
1010
} from 'ckeditor5/src/ui';
1111
import { FocusTracker, KeystrokeHandler } from 'ckeditor5/src/utils';
12-
import { icons } from 'ckeditor5/src/core';
1312
import colorIcon from '../../../icons/paint.svg'
13+
import checkIcon from '../../../icons/check.svg'
14+
import cancelIcon from '../../../icons/cancel.svg'
1415
import { sizeOptions, styleOptions, colorOptions, defaultColor,defaultStyle,defaultSize} from './buttonconfig';
1516

1617
export default class FormView extends View {
@@ -36,13 +37,13 @@ export default class FormView extends View {
3637
this.set('href', '')
3738

3839

39-
this.saveButtonView = this._createButton( 'Save', icons.check, 'ck-button-save' );
40+
this.saveButtonView = this._createButton( 'Save', checkIcon, 'ck-button-save' );
4041

4142
// Submit type of the button will trigger the submit event on entire form when clicked
4243
//(see submitHandler() in render() below).
4344
this.saveButtonView.type = 'submit';
4445

45-
this.cancelButtonView = this._createButton( 'Cancel', icons.cancel, 'ck-button-cancel' );
46+
this.cancelButtonView = this._createButton( 'Cancel', cancelIcon, 'ck-button-cancel' );
4647

4748
// Delegate ButtonView#execute to FormView#cancel.
4849
this.cancelButtonView.delegate( 'execute' ).to( this, 'cancel' );

ckeditor5_plugins/buttongroup/src/buttongroupconfig.js

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,19 +1,21 @@
1-
import { icons } from 'ckeditor5/src/core';
1+
import objectSizeLarge from '../../../icons/object-size-large.svg';
2+
import objectSizeMedium from '../../../icons/object-size-medium.svg';
3+
import objectSizeSmall from '../../../icons/object-size-small.svg';
24

35
export const sizeOptions = {
46
large: {
57
label: 'Large',
6-
icon: icons.objectSizeLarge,
8+
icon: objectSizeLarge,
79
className: 'ucb-link-button-large'
810
},
911
regular: {
1012
label: 'Regular',
11-
icon: icons.objectSizeMedium,
13+
icon: objectSizeMedium,
1214
className: 'ucb-link-button-regular'
1315
},
1416
small: {
1517
label: 'Small',
16-
icon: icons.objectSizeSmall,
18+
icon: objectSizeSmall,
1719
className: 'ucb-link-button-small'
1820
},
1921
};

ckeditor5_plugins/buttongroup/src/buttongroupui.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ import { WidgetToolbarRepository } from 'ckeditor5/src/widget';
44
import { ButtonView, createDropdown, addToolbarToDropdown } from 'ckeditor5/src/ui';
55
import {sizeOptions, defaultSize, colorOptions, defaultColor} from './buttongroupconfig'
66
import icon from '../../../icons/arrows-turn-right-solid.svg';
7-
import { icons } from 'ckeditor5/src/core';
7+
import plusIcon from '../../../icons/plus.svg';
88

99

1010

@@ -44,7 +44,7 @@ export default class ButtonGroupUI extends Plugin {
4444
componentFactory.add('buttonGroupColor', locale =>
4545
this._createDropdown(locale, 'Color', colorOptions[defaultColor].icon, commands.get('buttonGroupColor'), colorOptions, defaultColor));
4646
componentFactory.add('addNewButtonBG', locale =>
47-
this._createButton(locale, 'Add Button', icons.plus, commands.get('addNewButtonBG'), null))
47+
this._createButton(locale, 'Add Button', plusIcon, commands.get('addNewButtonBG'), null))
4848
}
4949

5050
/**

ckeditor5_plugins/buttongroup/src/buttongroupview.js

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -9,8 +9,9 @@ import {
99
addToolbarToDropdown
1010
} from 'ckeditor5/src/ui';
1111
import { FocusTracker, KeystrokeHandler } from 'ckeditor5/src/utils';
12-
import { icons } from 'ckeditor5/src/core';
1312
import colorIcon from '../../../icons/paint.svg'
13+
import checkIcon from '../../../icons/check.svg'
14+
import cancelIcon from '../../../icons/cancel.svg'
1415
import { sizeOptions, colorOptions, defaultColor,defaultStyle,defaultSize} from './buttongroupconfig';
1516

1617
export default class BGFormView extends View {
@@ -27,13 +28,13 @@ export default class BGFormView extends View {
2728
this.set('size', defaultSize)
2829
this.set('color', defaultColor)
2930

30-
this.saveButtonView = this._createButton( 'Save', icons.check, 'ck-button-save' );
31+
this.saveButtonView = this._createButton( 'Save', checkIcon, 'ck-button-save' );
3132

3233
// Submit type of the button will trigger the submit event on entire form when clicked
3334
//(see submitHandler() in render() below).
3435
this.saveButtonView.type = 'submit';
3536

36-
this.cancelButtonView = this._createButton( 'Cancel', icons.cancel, 'ck-button-cancel' );
37+
this.cancelButtonView = this._createButton( 'Cancel', cancelIcon, 'ck-button-cancel' );
3738

3839
// Delegate ButtonView#execute to FormView#cancel.
3940
this.cancelButtonView.delegate( 'execute' ).to( this, 'cancel' );

ckeditor5_plugins/calendar/src/calendarformview.js

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,8 @@
99

1010
import { FocusTracker, KeystrokeHandler } from 'ckeditor5/src/utils';
1111
import { ButtonView, createLabeledInputText, FocusCycler, LabeledFieldView, submitHandler, View } from 'ckeditor5/src/ui';
12-
import { icons } from 'ckeditor5/src/core';
12+
import checkIcon from '../../../icons/check.svg';
13+
import cancelIcon from '../../../icons/cancel.svg';
1314

1415
export default class CalendarFormView extends View {
1516
/**
@@ -27,8 +28,8 @@ export default class CalendarFormView extends View {
2728
this.valueInputView.fieldView.bind('value').to(this, 'value');
2829

2930
// Creates the save and cancel buttons.
30-
this.saveButtonView = this._createActionButton(locale, 'Save', icons.check, 'ck-button-save');
31-
this.cancelButtonView = this._createActionButton(locale, 'Cancel', icons.cancel, 'ck-button-cancel');
31+
this.saveButtonView = this._createActionButton(locale, 'Save', checkIcon, 'ck-button-save');
32+
this.cancelButtonView = this._createActionButton(locale, 'Cancel', cancelIcon, 'ck-button-cancel');
3233

3334
this.saveButtonView.type = 'submit'; // Set the type to 'submit', which will trigger the submit event on entire form when clicked.
3435
this.cancelButtonView.delegate('execute').to(this, 'cancel'); // Delegate ButtonView#execute to CalendarFormView#cancel.

ckeditor5_plugins/callout/src/calloutconfig.js

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -7,23 +7,20 @@
77
* @property {string} className
88
*/
99

10-
import { icons } from 'ckeditor5/src/core';
10+
1111

1212
/** @type {Object<string, SelectableOption>} */
1313
export const sizeOptions = {
1414
medium: {
1515
label: "Medium",
16-
icon: icons.small,
1716
className: "feature-layout-callout-medium",
1817
},
1918
large: {
2019
label: "Large",
21-
icon: icons.regular,
2220
className: "feature-layout-callout-large",
2321
},
2422
xlarge: {
2523
label: "Extra Large",
26-
icon: icons.large,
2724
className: "feature-layout-callout-xlarge",
2825
},
2926
};

ckeditor5_plugins/column/src/columnui.js

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,8 @@ import { ButtonView } from 'ckeditor5/src/ui';
44
import icon from '../../../icons/table-columns-solid.svg';
55
import leftIcon from '../../../icons/arrow-left-solid.svg';
66
import rightIcon from '../../../icons/arrow-right-solid.svg';
7-
import { icons } from 'ckeditor5/src/core';
7+
import plusIcon from '../../../icons/plus.svg';
8+
import eraserIcon from '../../../icons/eraser.svg';
89
import AddColumnPositionCommand from './addcolumnpositioncommand';
910

1011
export default class ColumnUI extends Plugin {
@@ -36,9 +37,9 @@ export default class ColumnUI extends Plugin {
3637
});
3738

3839
componentFactory.add('addColumn', locale =>
39-
this._createButton(locale, 'Add Column', icons.plus, commands.get('addColumn')));
40+
this._createButton(locale, 'Add Column', plusIcon, commands.get('addColumn')));
4041
componentFactory.add('removeColumn', locale =>
41-
this._createButton(locale, 'Remove Column', icons.eraser, commands.get('removeColumn')));
42+
this._createButton(locale, 'Remove Column', eraserIcon, commands.get('removeColumn')));
4243
componentFactory.add('addColumnLeft', locale =>
4344
this._createButton(locale, 'Add Column Left', leftIcon, commands.get('addColumnPosition'), 'left'));
4445
componentFactory.add('addColumnRight', locale =>

0 commit comments

Comments
 (0)