Make protocol selection UI consistent with basic settings UI

edan-bainglass · Dec 18, 2024 · 4c54fa2 · 4c54fa2
1 parent fa65a02
commit 4c54fa2
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 8 deletions.
diff --git a/src/aiidalab_qe/app/configuration/basic/workflow.py b/src/aiidalab_qe/app/configuration/basic/workflow.py
@@ -88,6 +88,10 @@ def render(self):
                         <li>
                             If the material should be studied with spin-orbit coupling
                         </li>
+                        <li>
+                            The protocol to use for the calculation, which sets default
+                            values balancing the accuracy and speed of the calculation
+                        </li>
                     </ol>
                 </div>
             """),
@@ -118,13 +122,15 @@ def render(self):
                     self.spin_orbit,
                 ]
             ),
-            ipw.HTML("""
-                <div style="padding-top: 0px; padding-bottom: 0px">
-                    <h4>Protocol</h4>
-                </div>
-            """),
-            ipw.HTML("Select the protocol:", layout=ipw.Layout(flex="1 1 auto")),
-            self.protocol,
+            ipw.HBox(
+                children=[
+                    ipw.Label(
+                        "Protocol:",
+                        layout=ipw.Layout(justify_content="flex-start", width="120px"),
+                    ),
+                    self.protocol,
+                ]
+            ),
             ipw.HTML("""
                 <div style="line-height: 140%; padding-top: 6px; padding-bottom: 0px">
                     The "moderate" protocol represents a trade-off between accuracy and

diff --git a/src/aiidalab_qe/app/main.py b/src/aiidalab_qe/app/main.py
@@ -115,7 +115,6 @@ def __init__(self, qe_auto_setup=True):
                 self._process_loading_message,
                 self._wizard_app_widget,
                 InAppGuide(identifier="post-guide", classes=["post-guide"]),
-                InAppGuide(children=[ipw.HTML("hello")], guide_id="general/basic"),
             ]
         )