Skip to content
This repository has been archived by the owner on Jun 26, 2020. It is now read-only.

t/ckeditor5/811: Fixed bold icon #64

Merged
merged 3 commits into from
Feb 13, 2018
Merged

t/ckeditor5/811: Fixed bold icon #64

merged 3 commits into from
Feb 13, 2018

Conversation

dkonopka
Copy link

@dkonopka dkonopka commented Feb 12, 2018 •
edited
Loading

Suggested merge commit message (convention)

Other: Rounded width and height of the bold icon. Closes ckeditor/ckeditor5#811.

@dkonopka dkonopka requested a review from oleq February 12, 2018 15:09
@coveralls
Copy link

coveralls commented Feb 12, 2018 •
edited
Loading

Coverage Status

Coverage remained the same at 100.0% when pulling 5f51ba2 on t/ckeditor5/811 into deb0c7c on master.

oleq
oleq suggested changes Feb 12, 2018
View reviewed changes
Copy link
Member

@oleq oleq left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I see a <use> tag in the XML. I think we don't need that.

@dkonopka
Copy link
Author

Fixed 👍 Weird thing svgo doesn't remove that.

@oleq oleq merged commit 4281eca into master Feb 13, 2018
@oleq oleq deleted the t/ckeditor5/811 branch February 13, 2018 11:24
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@oleq oleq oleq requested changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@dkonopka @coveralls @oleq