HackStudio: Set_cursor even if the file is already open

Upon opening already opened file, the cursor was previously not set to the correct line and column. With this patch, it should be correctly set. Fixes a bug where ctrl+clicking a function declaration would not jump to the line if the file containing the function is already open.
Author: https://github.com/iyush Commit: https://github.com/SerenityOS/serenity/commit/cc301f9861 Pull-request: https://github.com/SerenityOS/serenity/pull/18334
2023-04-12 21:06:46 +02:00 · 2023-04-12 21:06:46 +02:00 · cc301f9861 · 2024-07-17 08:27:05 +09:00
commit cc301f9861
parent 742f6f7e26
1 changed files with 1 additions and 0 deletions
--- a/Userland/DevTools/HackStudio/HackStudioWidget.cpp
+++ b/Userland/DevTools/HackStudio/HackStudioWidget.cpp
@ -321,6 +321,7 @@ bool HackStudioWidget::open_file(DeprecatedString const& full_filename, size_t l

    if (editor_wrapper_or_none.has_value()) {
        set_current_editor_wrapper(editor_wrapper_or_none.release_value());
+        current_editor().set_cursor(line, column);
        return true;
    } else if (active_file().is_empty() && !current_editor().document().is_modified() && !full_filename.is_empty()) {
        // Replace "Untitled" blank file since it hasn't been modified