From fa56a428c5f5a39ebf60d4c71ed5406809502a0c Mon Sep 17 00:00:00 2001
From: Kovid Goyal <kovid@kovidgoyal.net>
Date: Wed, 4 Jul 2018 10:01:33 +0530
Subject: [PATCH] Start work on input fmt options UI

---
 src/calibre/srv/convert.py               |  2 ++
 src/pyj/book_list/conversion_widgets.pyj | 20 ++++++++++++++++
 src/pyj/book_list/convert_book.pyj       | 29 ++++++++++++++++--------
 3 files changed, 41 insertions(+), 10 deletions(-)

diff --git a/src/calibre/srv/convert.py b/src/calibre/srv/convert.py
index ef0b73fe14..c0e12b3e2a 100644
--- a/src/calibre/srv/convert.py
+++ b/src/calibre/srv/convert.py
@@ -211,6 +211,8 @@ def get_conversion_options(input_fmt, output_fmt, book_id, db):
     plumber = create_dummy_plumber(input_fmt, output_fmt)
     specifics = load_specifics(db, book_id)
     ans = {'options': {}, 'disabled': set(), 'defaults': {}, 'help': {}}
+    ans['input_plugin_name'] = plumber.input_plugin.commit_name
+    ans['output_plugin_name'] = plumber.output_plugin.commit_name
 
     def merge_group(group_name, option_names):
         if not group_name or group_name in ('debug', 'metadata'):
diff --git a/src/pyj/book_list/conversion_widgets.pyj b/src/pyj/book_list/conversion_widgets.pyj
index 399711dfd5..9b5331597f 100644
--- a/src/pyj/book_list/conversion_widgets.pyj
+++ b/src/pyj/book_list/conversion_widgets.pyj
@@ -405,6 +405,26 @@ def search_and_replace(container):
     ))
 # }}}
 
+# Comic Input {{{
+@ep
+def comic_input(container):
+    settings = v'[]'
+    def add(func, name, text, **kw):
+        g.appendChild(func(name, text, **kw))
+        settings.push(name)
+
+    add_listener('no_process', def (name):
+        disabled = get('no_process')
+        for dname in settings:
+            set_disabled(dname, disabled)
+    )
+
+    g = E.div(class_='simple-group')
+    container.appendChild(g)
+    g.appendChild(checkbox('no_process', _('&Disable comic processing')))
+    add(checkbox, 'dont_grayscale', _('Disable conversion of images to &black and white'))
+
+# }}}
 
 def restore_defaults():
     for setting in registry:
diff --git a/src/pyj/book_list/convert_book.pyj b/src/pyj/book_list/convert_book.pyj
index ab2411b6c1..dd23b7eb5e 100644
--- a/src/pyj/book_list/convert_book.pyj
+++ b/src/pyj/book_list/convert_book.pyj
@@ -165,12 +165,20 @@ def create_converting_markup():
     return ans, init
 
 
+def current_input_format():
+    return document.getElementById(overall_container_id).querySelector('select[name="input_formats"]').value.toUpperCase()
+
+
+def current_output_format():
+    return document.getElementById(overall_container_id).querySelector('select[name="output_formats"]').value.toUpperCase()
+
+
 def start_conversion():
     nonlocal current_state
     container = document.getElementById(overall_container_id)
     data = {
-        'input_fmt': container.querySelector('select[name="input_formats"]').value,
-        'output_fmt': container.querySelector('select[name="output_formats"]').value,
+        'input_fmt': current_input_format(),
+        'output_fmt': current_output_format(),
         'options': get_conversion_options(container),
         'book_id': conversion_data.book_id,
     }
@@ -240,7 +248,7 @@ def create_configuring_markup():
         ans = E.li(E.a(class_='simple-link', href='javascript: void(0)'))
         ans.dataset.group = name
         ans.firstChild.addEventListener('click', show_group)
-        if not entry_points[name]:
+        if name is not 'input_fmt' and name is not 'output_fmt' and not entry_points[name]:
             ans.style.display = 'none'
         return ans
 
@@ -282,11 +290,9 @@ def create_configuring_markup():
             if GROUP_TITLES[group_name]:
                 title = GROUP_TITLES[group_name]
             elif group_name is 'input_fmt':
-                title = container.querySelector('select[name="input_formats"]').value.toUpperCase()
-                title = _('{} input').format(title)
+                title = _('{} input').format(current_input_format(), title)
             else:
-                title = container.querySelector('select[name="output_formats"]').value.toUpperCase()
-                title = _('{} output').format(title)
+                title = _('{} output').format(current_output_format(), title)
             li.firstChild.textContent = title
 
     return ans, initialize
@@ -328,9 +334,12 @@ def create_configure_group_markup():
             _('Configuring {} settings').format(conversion_data.configuring_group_title)))
         panel = E.div()
         container.appendChild(panel)
-        create_option_group(
-            conversion_data.configuring_group, conversion_data.profiles, container,
-            get_option_value, get_option_default_value, is_option_disabled, get_option_help, on_close)
+        g = conversion_data.configuring_group
+        if g is 'input_fmt':
+            g = conversion_data.conversion_options.input_plugin_name
+        elif g is 'output_fmt':
+            g = conversion_data.conversion_options.output_plugin_name
+        create_option_group(g, conversion_data.profiles, container, get_option_value, get_option_default_value, is_option_disabled, get_option_help, on_close)
 
     return ans, init