Update sync-v5 with 1.9.0 code.

Author: Rémy HUBSCHER

README.md

@@ -2,6 +2,7 @@
 
 [![CircleCI](https://circleci.com/gh/mozilla/notes/tree/master.svg?style=svg)](https://circleci.com/gh/mozilla/notes/tree/master)
 [![Available on Test Pilot](https://img.shields.io/badge/available_on-Test_Pilot-0996F8.svg)](https://testpilot.firefox.com/experiments/notes)
+[![Waffle.io - Bug Triage](https://badge.waffle.io/mozilla/notes.svg?columns=all)](https://waffle.io/mozilla/notes)
 
 ## What does it aim for
 
@@ -43,4 +44,4 @@ Firefox Notes localization is managed via [Pontoon](https://pontoon.mozilla.org/
 
 Design for reference https://mozilla.invisionapp.com/share/6VBUYHMRB#/235284916_Desktop_Sidebar
 
-![Notes Screenshot](https://i.imgur.com/kHwBN2f.png)
+![Notes Screenshot](https://i.imgur.com/lQM0tpr.png)

ckeditor-build/src/ckeditor.js

@@ -5,12 +5,12 @@
 
 import ClassicEditorBase from '@ckeditor/ckeditor5-editor-classic/src/classiceditor';
 import EssentialsPlugin from '@ckeditor/ckeditor5-essentials/src/essentials';
-import MarkdownPlugin from './markdown';
 import AutoformatPlugin from '@ckeditor/ckeditor5-autoformat/src/autoformat';
 import BoldPlugin from '@ckeditor/ckeditor5-basic-styles/src/bold';
 import ItalicPlugin from '@ckeditor/ckeditor5-basic-styles/src/italic';
 import StrikePlugin from './strike';
 import CodePlugin from '@ckeditor/ckeditor5-basic-styles/src/code';
+import UnderlinePlugin from '@ckeditor/ckeditor5-basic-styles/src/underline';
 import HeadingPlugin from '@ckeditor/ckeditor5-heading/src/heading';
 import ListPlugin from '@ckeditor/ckeditor5-list/src/list';
 import ParagraphPlugin from '@ckeditor/ckeditor5-paragraph/src/paragraph';
@@ -21,12 +21,12 @@ export default class ClassicEditor extends ClassicEditorBase {}
 ClassicEditor.build = {
   plugins: [
     EssentialsPlugin,
-    MarkdownPlugin,
     AutoformatPlugin,
     BoldPlugin,
     ItalicPlugin,
     StrikePlugin,
     CodePlugin,
+    UnderlinePlugin,
     HeadingPlugin,
     ListPlugin,
     ParagraphPlugin

ckeditor-build/src/markdown.js

@@ -1,6 +1,8 @@
 welcomeTitle2=¡Hola!
 welcomeText2=Bienvenido a este bloc de notas de una página integrado en Firefox. Navegar la web. Tomar notas. Así de simple.
 
+migratedEditor=Reemplazamos el editor de texto en Notes para mejor velocidad y funcionalidad pero tuvimos que eliminar el formato de las notas existentes. Disculpas por el inconveniente, apreciamos tu participación.
+
 emptyPlaceHolder=Tomar nota…
 
 giveFeedback=Clic aquí para darnos tu opinión

locales/es-AR/notes.properties

@@ -1,6 +1,8 @@
 welcomeTitle2=¡Hola!
 welcomeText2=Bienvenido a este bloc de notas integrado en Firefox. Navega por la web y anota. Es así de sencillo.
 
+migratedEditor=Hemos mejorado el editor de texto de Notes con más velocidad y funcionalidades, pero tuvimos que eliminar el formato de las notas que ya tenías. Lo sentimos, pero te agradecemos que lo uses.
+
 emptyPlaceHolder=Tomar notas…
 
 giveFeedback=Haz clic aquí para darnos tu opinión

locales/es-ES/notes.properties

@@ -1,6 +1,8 @@
 welcomeTitle2=¡Hola!
 welcomeText2=Bienvenido a este bloc de notas integrado en Firefox. Navega por la web y anota. Así de sencillo.
 
+migratedEditor=Reemplazamos el editor de texto en Notas para una mayor velocidad y funcionalidad, pero tuvimos que eliminar el formato de tus notas existentes. ¡Lo sentimos, pero agradecemos tu participación!
+
 emptyPlaceHolder=Toma nota…
 
 giveFeedback=Haz clic aquí para hacernos algunos comentarios

locales/es-MX/notes.properties

@@ -1,6 +1,8 @@
 welcomeTitle2=Hei!
 welcomeText2=Velkomen til denne sida med innebygd «enkel tekst» i Firefox. Surf på nettet. Skribl ned notat. Så enkelt er det.
 
+migratedEditor=Vi erstatta tekstredigeringsprogrammet i Notes for betre fart og funksjonalitet, men måtte fjerne formateringa av eksisterande notat. Beklagar, men vi set pris på deltakinga di!
+
 emptyPlaceHolder=Skriv eit notat…
 
 giveFeedback=Klikk her for å gje oss tilbakemelding

locales/nn-NO/notes.properties

@@ -1,6 +1,8 @@
 welcomeTitle2=Здраво!
 welcomeText2=Добродошли на једностранични нотес уграђен у Firefox. Сурфујте вебом. Запишите белешке. Једноставно је.
 
+migratedEditor=Заменили смо едитор текста у Notes-у ради веће брзине и боље функционисање, али смо морали да уклонимо форматирање ваших постојећих нота. Жао нам је због тога, али ценимо ваше учествовање!
+
 emptyPlaceHolder=Забележи ноту…
 
 giveFeedback=Кликните овде за повратну информацију

locales/sr/notes.properties

@@ -1,6 +1,8 @@
 welcomeTitle2=Hej!
 welcomeText2=Välkommen till den här sidan med ett inbyggt anteckningsblock i Firefox. Surfa på webben. Krafsa ner anteckningar. Så enkelt är det.
 
+migratedEditor=Vi ersatte textredigeraren i Notes för högre hastighet och funktionalitet, men måste ta bort formateringen av dina befintliga anteckningar. Ledsen för det, men vi uppskattar ditt deltagande!
+
 emptyPlaceHolder=Gör en anteckning…
 
 giveFeedback=Klicka här för att ge oss lite återkoppling

locales/sv-SE/notes.properties

@@ -8,7 +8,7 @@
     "url": "https://github.com/mozilla/notes/issues"
   },
   "dependencies": {
-    "@ckeditor/ckeditor5-autoformat": "^1.0.0-alpha.2",
+    "@ckeditor/ckeditor5-autoformat": "vladikoff/ckeditor5-autoformat#fix-numbered-list",
     "@ckeditor/ckeditor5-basic-styles": "^1.0.0-alpha.2",
     "@ckeditor/ckeditor5-build-classic": "^1.0.0-alpha.2",
     "@ckeditor/ckeditor5-dev-utils": "^5.0.0",
@@ -17,7 +17,6 @@
     "@ckeditor/ckeditor5-essentials": "^1.0.0-alpha.2",
     "@ckeditor/ckeditor5-heading": "vladikoff/ckeditor5-heading#short-heading",
     "@ckeditor/ckeditor5-list": "^1.0.0-alpha.2",
-    "@ckeditor/ckeditor5-markdown-gfm": "^1.0.0-alpha.2",
     "@ckeditor/ckeditor5-paragraph": "^1.0.0-alpha.2",
     "babel-minify-webpack-plugin": "^0.2.0",
     "css-loader": "^0.28.7",

package.json

@@ -33,23 +33,29 @@ ClassicEditor.create(document.querySelector('#editor'), {
     .then(() => {
       let ignoreNextLoadEvent = false;
 
-      editor.document.on('change', () => {
-        const content = editor.getData();
-
-        if (!ignoreNextLoadEvent) {
-          // Debounce this second event
-          chrome.runtime.sendMessage({
-            action: 'kinto-save',
-            content
+      editor.document.on('change', (eventInfo, name) => {
+        const isFocused = document.querySelector('.ck-editor__editable').classList.contains('ck-focused');
+        // Only use the focused editor or handle 'rename' events to set the data into storage.
+        if (isFocused || name === 'rename') {
+          const content = editor.getData();
+          browser.storage.local.set({ notes2: content }).then(() => {
+            // Notify other sidebars
+            if (!ignoreNextLoadEvent) {
+              chrome.runtime.sendMessage('[email protected]', {
+                action: 'kinto-save',
+                content
+              });
+
+              updateSavingIndicator();
+              // Debounce this second event
+              chrome.runtime.sendMessage({
+                action: 'metrics-changed',
+                context: getPadStats(editor)
+              });
+            } else {
+              ignoreNextLoadEvent = false;
+            }
           });
-
-          // Debounce this second event
-          chrome.runtime.sendMessage({
-            action: 'metrics-changed',
-            context: getPadStats(editor)
-          });
-        } else {
-          ignoreNextLoadEvent = false;
         }
       });
 
@@ -78,6 +84,14 @@ ClassicEditor.create(document.querySelector('#editor'), {
         });
       });
 
+      // Fixes an issue with CKEditor and keeping multiple Firefox windows in sync
+      // Ref: https://github.com/mozilla/notes/issues/424
+      document.querySelectorAll('.ck-heading-dropdown .ck-list__item').forEach((btn) => {
+        btn.addEventListener('click', () => {
+          editor.fire('changesDone');
+        });
+      });
+
       localizeEditorButtons();
     });
 
@@ -89,7 +103,7 @@ ClassicEditor.create(document.querySelector('#editor'), {
 let ignoreNextLoadEvent = false;
 function handleLocalContent(editor, data) {
   if (!data.hasOwnProperty('notes2')) {
-    editor.setData(`## ${browser.i18n.getMessage('welcomeTitle2')}\n\n${browser.i18n.getMessage('welcomeText2')}`);
+    editor.setData(`<h2>${browser.i18n.getMessage('welcomeTitle2')}</h2><p>${browser.i18n.getMessage('welcomeText2')}</p>`);
   } else {
     if (JSON.stringify(editor.getData()) !== JSON.stringify(data.notes2)) {
       editor.setData(data.notes2);
@@ -403,8 +417,8 @@ function localizeEditorButtons () {
     bold = document.querySelector('button.ck-button:nth-child(2)'),
     italic = document.querySelector('button.ck-button:nth-child(3)'),
     strike = document.querySelector('button.ck-button:nth-child(4)'),
-    ordered = document.querySelector('button.ck-button:nth-child(5)'),
-    bullet = document.querySelector('button.ck-button:nth-child(6)');
+    bullet = document.querySelector('button.ck-button:nth-child(5)'),
+    ordered = document.querySelector('button.ck-button:nth-child(6)');
 
 // Setting button titles in place of tooltips
   size.title = browser.i18n.getMessage('fontSizeTitle');

src/sidebar/panel.js

@@ -47,7 +47,7 @@ body .ck-editor .ck-editor__editable {
 }
 
 .ck-editor__editable ol, .ck-editor__editable ul {
-  padding-left: 10px;
+  padding-left: 20px;
 }
 
 /**
@@ -366,18 +366,23 @@ CKEditor overrides
 
 #notes .ck-editor__editable p {
   margin: 5px 0;
+  font-size: 14px;
 }
 
 #notes .ck-editor__editable h1 {
-  margin: 10px 0;
+  margin: 2px 0;
+  font-size: 24px;
+  padding: 0;
 }
 
 #notes .ck-editor__editable h2 {
   margin: 7px 0;
+  font-size: 20px;
 }
 
 #notes .ck-editor__editable h3 {
   margin: 5px 0;
+  font-size: 16px;
 }
 
 .ck-heading-dropdown button {