Parcourir la source

HackStudio: Make use of existing TextEditor::on_focusin callback

Sam Atkins il y a 2 ans
Parent
commit
4c551bfad9

+ 0 - 12
Userland/DevTools/HackStudio/Editor.cpp

@@ -146,18 +146,6 @@ EditorWrapper const& Editor::wrapper() const
     return static_cast<EditorWrapper const&>(*parent());
 }
 
-void Editor::focusin_event(GUI::FocusEvent& event)
-{
-    if (on_focus)
-        on_focus();
-    GUI::TextEditor::focusin_event(event);
-}
-
-void Editor::focusout_event(GUI::FocusEvent& event)
-{
-    GUI::TextEditor::focusout_event(event);
-}
-
 Gfx::IntRect Editor::gutter_icon_rect(size_t line_number) const
 {
     return gutter_content_rect(line_number).translated(frame_thickness(), 0);

+ 0 - 3
Userland/DevTools/HackStudio/Editor.h

@@ -28,7 +28,6 @@ public:
 
     virtual ~Editor() override = default;
 
-    Function<void()> on_focus;
     Function<void(DeprecatedString)> on_open;
 
     EditorWrapper& wrapper();
@@ -64,8 +63,6 @@ public:
     void set_semantic_syntax_highlighting(bool value);
 
 private:
-    virtual void focusin_event(GUI::FocusEvent&) override;
-    virtual void focusout_event(GUI::FocusEvent&) override;
     virtual void paint_event(GUI::PaintEvent&) override;
     virtual void mousemove_event(GUI::MouseEvent&) override;
     virtual void mousedown_event(GUI::MouseEvent&) override;

+ 1 - 1
Userland/DevTools/HackStudio/EditorWrapper.cpp

@@ -30,7 +30,7 @@ EditorWrapper::EditorWrapper()
     m_editor->set_ruler_visible(true);
     m_editor->set_automatic_indentation_enabled(true);
 
-    m_editor->on_focus = [this] {
+    m_editor->on_focusin = [this] {
         set_current_editor_wrapper(this);
     };