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

Introduced the superscript and the subscript features #80

Merged
merged 13 commits into from
Nov 13, 2018
Merged

Introduced the superscript and the subscript features #80

merged 13 commits into from
Nov 13, 2018

Conversation

ma2ciek
Copy link
Contributor

@ma2ciek ma2ciek commented Nov 2, 2018

Suggested merge commit message (convention)

Feature: Introduced the Superscript and the Subscript features. Closes ckeditor/ckeditor5#5557. Closes #74.


Additional information

This PR is a clone of the #74 (good job @idleb), so we could add commit to it (@oleq has changed the icons), which was impossible on the original PR's branch.

@ma2ciek ma2ciek requested a review from Reinmar November 2, 2018 13:17
@oleq
Copy link
Member

oleq commented Nov 2, 2018

Don't forget to add the icons to the iconset manual test in ckeditor5-theme-lark.

Copy link
Member

@Reinmar Reinmar left a comment

Choose a reason for hiding this comment

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

Missing updates in the docs/features/basic-styles.md.

@ma2ciek
Copy link
Contributor Author

ma2ciek commented Nov 13, 2018

Missing updates in the docs/features/basic-styles.md.

Fixed.

@Reinmar
Copy link
Member

Reinmar commented Nov 13, 2018

image

Ekhm :P

I'll add them there.

@Reinmar
Copy link
Member

Reinmar commented Nov 13, 2018

I was about to merge and then:

image

Always test.

BTW, can you spot what's wrong here?

image

@Reinmar Reinmar merged commit f281e99 into master Nov 13, 2018
@Reinmar Reinmar deleted the t/76 branch November 13, 2018 17:01
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
3 participants