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

Commit

Permalink
Fix: Missing marker-yellow CSS class definition in tests and docs.
Browse files Browse the repository at this point in the history
  • Loading branch information
jodator committed Feb 19, 2018
1 parent 0112fd1 commit 0705945
Show file tree
Hide file tree
Showing 7 changed files with 23 additions and 15 deletions.
2 changes: 1 addition & 1 deletion docs/_snippets/features/build-highlight-source.html
@@ -1,5 +1,5 @@
<style>
.marker {
.marker-yellow {
background-color: #fdfd77;
}

Expand Down
10 changes: 7 additions & 3 deletions docs/_snippets/features/highlight-buttons.html
@@ -1,10 +1,14 @@
<div id="snippet-highlight-buttons">
<h2>Highlight feature sample.</h2>

<p>Highlight feature example.</p>
<p>
Here are some markers:
<mark class="marker">yellow one</mark>, <mark class="marker-pink">pink one</mark> and <mark class="marker-green">green one</mark>.
</p>
<ul>
<li>the <mark class="marker-yellow">yellow</mark> one,</li>
<li>the <mark class="marker-pink">pink</mark> one,</li>
<li>the <mark class="marker-green">green</mark> one,</li>
<li>the <mark class="marker-blue">blue</mark> one</li>
</ul>
<p>
Here are some pens:
<mark class="pen-red">red pen</mark> and <mark class="pen-green">green one</mark>.
Expand Down
10 changes: 7 additions & 3 deletions docs/_snippets/features/highlight.html
@@ -1,10 +1,14 @@
<div id="snippet-highlight">
<h2>Highlight feature sample.</h2>

<p>Highlight feature example.</p>
<p>
Here are some markers:
<mark class="marker">yellow one</mark>, <mark class="marker-pink">pink one</mark> and <mark class="marker-green">green one</mark>.
</p>
<ul>
<li>the <mark class="marker-yellow">yellow</mark> one,</li>
<li>the <mark class="marker-pink">pink</mark> one,</li>
<li>the <mark class="marker-green">green</mark> one,</li>
<li>the <mark class="marker-blue">blue</mark> one</li>
</ul>
<p>
Here are some pens:
<mark class="pen-red">red pen</mark> and <mark class="pen-green">green one</mark>.
Expand Down
2 changes: 1 addition & 1 deletion docs/features/highlight.md
Expand Up @@ -106,7 +106,7 @@ The {@link module:highlight/highlight~Highlight} plugin registers:
You can change the highlight of the current selection by executing the command with a desired value:

```js
editor.execute( 'highlight', { value: 'marker' } );
editor.execute( 'highlight', { value: 'yellowMarker' } );
```

The `value` corresponds to the `model` property in configuration object. For the default configuration:
Expand Down
2 changes: 1 addition & 1 deletion src/highlight.js
Expand Up @@ -87,7 +87,7 @@ export default class Highlight extends Plugin {
* The available highlighters options. The default value is:
*
* options: [
* { model: 'marker', class: 'marker', title: 'Marker', color: '#fdfd77', type: 'marker' },
* { model: 'yellowMarker', class: 'marker-yellow', title: 'Yellow marker', color: '#fdfd77', type: 'marker' },
* { model: 'greenMarker', class: 'marker-green', title: 'Green marker', color: '#63f963', type: 'marker' },
* { model: 'pinkMarker', class: 'marker-pink', title: 'Pink marker', color: '#fc7999', type: 'marker' },
* { model: 'blueMarker', class: 'marker-blue', title: 'Blue marker', color: '#72cdfd', type: 'marker' },
Expand Down
6 changes: 3 additions & 3 deletions tests/manual/highlight-buttons.html
@@ -1,5 +1,5 @@
<style>
.marker {
.marker-yellow {
background-color: #fdfd77;
}

Expand Down Expand Up @@ -32,7 +32,7 @@
Here are some markers:
</p>
<ul>
<li>the <mark class="marker">yellow</mark> one,</li>
<li>the <mark class="marker-yellow">yellow</mark> one,</li>
<li>the <mark class="marker-pink">pink</mark> one,</li>
<li>the <mark class="marker-green">green</mark> one,</li>
<li>the <mark class="marker-blue">blue</mark> one</li>
Expand All @@ -43,6 +43,6 @@
</p>
<figure class="image">
<img src="sample.jpg" alt="CKEditor logo" />
<figcaption>Some image with caption and <mark class="marker">highlighted text</mark>.</figcaption>
<figcaption>Some image with caption and <mark class="marker-yellow">highlighted text</mark>.</figcaption>
</figure>
</div>
6 changes: 3 additions & 3 deletions tests/manual/highlight.html
@@ -1,5 +1,5 @@
<style>
.marker {
.marker-yellow {
background-color: #fdfd77;
}

Expand Down Expand Up @@ -32,7 +32,7 @@
Here are some markers:
</p>
<ul>
<li>the <mark class="marker">yellow</mark> one,</li>
<li>the <mark class="marker-yellow">yellow</mark> one,</li>
<li>the <mark class="marker-pink">pink</mark> one,</li>
<li>the <mark class="marker-green">green</mark> one,</li>
<li>the <mark class="marker-blue">blue</mark> one</li>
Expand All @@ -43,6 +43,6 @@
</p>
<figure class="image">
<img src="sample.jpg" alt="CKEditor logo" />
<figcaption>Some image with caption and <mark class="marker">highlighted text</mark>.</figcaption>
<figcaption>Some image with caption and <mark class="marker-yellow">highlighted text</mark>.</figcaption>
</figure>
</div>

0 comments on commit 0705945

Please sign in to comment.