Skip to content
This repository was archived by the owner on Jan 11, 2023. It is now read-only.
This repository was archived by the owner on Jan 11, 2023. It is now read-only.

[Shortcuts] modal polish #4213

Description

@jasonLaster

Todos

  • we should add a shortcut for launching it via cmd + / i think
  • we should move K+ to the right and soften the color (whatever the accordion headers are)
  • the section headers in the mocks looked pretty nice. Lets do some small typography cleanup.
diff --git a/src/components/App.js b/src/components/App.js
index 1dd5132..62f5b35 100644
--- a/src/components/App.js
+++ b/src/components/App.js
@@ -100,6 +100,9 @@ class App extends Component {
       L10N.getStr("symbolSearch.search.key2"),
       this.toggleSymbolModal
     );
+
+    shortcuts.on(shortcutModalKey, this.toggleShortcutsModal);
+
     shortcuts.on("Escape", this.onEscape);
   }

diff --git a/src/components/SecondaryPanes/index.js b/src/components/SecondaryPanes/index.js
index 55cd6f9..174f661 100644
--- a/src/components/SecondaryPanes/index.js
+++ b/src/components/SecondaryPanes/index.js
@@ -222,6 +222,7 @@ class SecondaryPanes extends Component {
       return;
     }

+    // we can style this in utils bar.css
     return (
       <UtilsBar
         horizontal={this.props.horizontal}
diff --git a/src/components/ShortcutsModal.css b/src/components/ShortcutsModal.css
index b3edaaa..c29b771 100644
--- a/src/components/ShortcutsModal.css
+++ b/src/components/ShortcutsModal.css
@@ -5,6 +5,7 @@
   column-width: 250px;
 }

+/* HERE */
 .shortcuts-content h2 {

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Fields

    Priority

    None yet

    Jira

    None yet

    Severity

    None yet

    Estimate

    None yet

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions