From 126d909ca7ae75a7fa03626761cdd2ceb2a1c4cf Mon Sep 17 00:00:00 2001
From: tophf <tophf@gmx.com>
Date: Sun, 22 Jul 2018 19:55:27 +0300
Subject: [PATCH] use a typical OS order for #help-popup buttons

* horizontally centered in Windows
* right-aligned otherwise
---
 edit/edit.css | 13 ++++++++++---
 edit/edit.js  | 14 +++++++-------
 edit/lint.js  | 10 ++++++----
 3 files changed, 23 insertions(+), 14 deletions(-)

diff --git a/edit/edit.css b/edit/edit.css
index 6500e315..75df6501 100644
--- a/edit/edit.css
+++ b/edit/edit.css
@@ -564,10 +564,11 @@ body[data-match-highlight="selection"] .CodeMirror-selection-highlight-scrollbar
   top: .5em;
 }
 #help-popup .saved-message {
-  display: none;
+  /* display: none; */
   color: #090;
-  margin-left: 10px;
+  /* margin-left: 10px; */
   font-weight: bold;
+  position: absolute;
 }
 #help-popup .saved-message.show {
   display: inline-block;
@@ -590,13 +591,19 @@ body[data-match-highlight="selection"] .CodeMirror-selection-highlight-scrollbar
   padding-right: 0.5rem;
 }
 
+#help-popup .buttons {
+  text-align: center;
+}
+.non-windows #help-popup .buttons {
+  direction: rtl;
+  text-align: right;
+}
 #help-popup button[name^="import"] {
   line-height: 1.5rem;
   padding: 0 0.5rem;
   margin: 0.5rem 0 0 0.5rem;
   pointer-events: none;
   opacity: 0.5;
-  float: right;
 }
 #help-popup.ready button[name^="import"] {
   pointer-events: all;
diff --git a/edit/edit.js b/edit/edit.js
index db19b917..a08d2c1c 100644
--- a/edit/edit.js
+++ b/edit/edit.js
@@ -427,19 +427,19 @@ function toMozillaFormat() {
 
 function fromMozillaFormat() {
   const popup = showCodeMirrorPopup(t('styleFromMozillaFormatPrompt'),
-    $create([
-      $create('button', {
-        name: 'import-append',
-        textContent: t('importAppendLabel'),
-        title: 'Ctrl-Enter:\n' + t('importAppendTooltip'),
-        onclick: doImport,
-      }),
+    $create('.buttons', [
       $create('button', {
         name: 'import-replace',
         textContent: t('importReplaceLabel'),
         title: 'Ctrl-Shift-Enter:\n' + t('importReplaceTooltip'),
         onclick: () => doImport({replaceOldStyle: true}),
       }),
+      $create('button', {
+        name: 'import-append',
+        textContent: t('importAppendLabel'),
+        title: 'Ctrl-Enter:\n' + t('importAppendTooltip'),
+        onclick: doImport,
+      }),
     ]));
   const contents = $('.contents', popup);
   contents.insertBefore(popup.codebox.display.wrapper, contents.firstElementChild);
diff --git a/edit/lint.js b/edit/lint.js
index d6c64fc2..0409c443 100644
--- a/edit/lint.js
+++ b/edit/lint.js
@@ -448,10 +448,12 @@ function setupLinterPopup(config) {
           t('linterRulesLink')),
         linter === 'csslint' ? ' ' + t('linterCSSLintSettings') : '',
       ]),
-      $create('button.save', {onclick: save, title: 'Ctrl-Enter'}, t('styleSaveLabel')),
-      $create('button.cancel', {onclick: cancel}, t('confirmClose')),
-      $create('button.reset', {onclick: reset, title: t('linterResetMessage')}, t('genericResetLabel')),
-      $create('span.saved-message', t('genericSavedMessage')),
+      $create('.buttons', [
+        $create('button.save', {onclick: save, title: 'Ctrl-Enter'}, t('styleSaveLabel')),
+        $create('button.cancel', {onclick: cancel}, t('confirmClose')),
+        $create('button.reset', {onclick: reset, title: t('linterResetMessage')}, t('genericResetLabel')),
+        $create('span.saved-message', t('genericSavedMessage')),
+      ]),
     ]);
   }