From 26aee5845344f64e516d83fbe8640bedc388d0ff Mon Sep 17 00:00:00 2001 From: Kovid Goyal Date: Thu, 22 Mar 2018 12:15:52 +0530 Subject: [PATCH] Use three toolbars for comments editor so they dont wrap even on small screens --- imgsrc/srv/select-all.svg | 6 ++++-- src/pyj/book_list/comments_editor.pyj | 10 +++++++--- 2 files changed, 11 insertions(+), 5 deletions(-) diff --git a/imgsrc/srv/select-all.svg b/imgsrc/srv/select-all.svg index 0506a50d60..afd4b77978 100644 --- a/imgsrc/srv/select-all.svg +++ b/imgsrc/srv/select-all.svg @@ -1,5 +1,7 @@ - + - + + diff --git a/src/pyj/book_list/comments_editor.pyj b/src/pyj/book_list/comments_editor.pyj index c3614e4546..767aa04ac8 100644 --- a/src/pyj/book_list/comments_editor.pyj +++ b/src/pyj/book_list/comments_editor.pyj @@ -447,7 +447,8 @@ def create_comments_editor(container): toolbars = E.div(style='flex-grow: 0') toolbar1 = E.div(class_=TOOLBAR_CLASS) toolbar2 = E.div(class_=TOOLBAR_CLASS) - toolbars.appendChild(toolbar1), toolbars.appendChild(toolbar2) + toolbar3 = E.div(class_=TOOLBAR_CLASS) + toolbars.appendChild(toolbar1), toolbars.appendChild(toolbar2), toolbars.appendChild(toolbar3) acmap = all_editor_actions() def add(toolbar, ac_name): @@ -457,12 +458,15 @@ def create_comments_editor(container): else: toolbar.appendChild(E.div(class_='sep')) - for ac_name in 'undo redo select-all remove-format | bold italic underline strikethrough | hr superscript subscript format-block'.split(' '): + for ac_name in 'undo redo select-all remove-format bold italic underline strikethrough'.split(' '): add(toolbar1, ac_name) - for ac_name in 'ul ol indent outdent | justify-left justify-center justify-right justify-full | insert-link insert-image fg bg'.split(' '): + for ac_name in 'hr superscript subscript format-block ul ol indent outdent'.split(' '): add(toolbar2, ac_name) + for ac_name in 'justify-left justify-center justify-right justify-full insert-link insert-image fg bg'.split(' '): + add(toolbar3, ac_name) + container.setAttribute('style', (container.getAttribute('style') or '') + ';display: flex; flex-direction: column; align-items: stretch') container.appendChild(toolbars) container.appendChild(iframe)