mirror of
https://github.com/LadybirdBrowser/ladybird.git
synced 2025-07-19 07:22:21 +00:00
TextEditor: Allow starting with a file argument that doesn't exist
If TextEditor is started with an argument for a file that doesn't exist, we now allow editing it. The file will be created once it is saved.
This commit is contained in:
parent
36bfc912fc
commit
8241a6c8eb
Notes:
sideshowbarker
2024-07-18 08:29:27 +09:00
Author: https://github.com/itamar8910
Commit: 8241a6c8eb
Pull-request: https://github.com/SerenityOS/serenity/pull/8953
Reviewed-by: https://github.com/alimpfard
3 changed files with 28 additions and 14 deletions
|
@ -693,6 +693,13 @@ bool MainWidget::read_file_and_close(int fd, String const& path)
|
||||||
return true;
|
return true;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
void MainWidget::open_nonexistent_file(String const& path)
|
||||||
|
{
|
||||||
|
m_editor->set_text({});
|
||||||
|
set_path(path);
|
||||||
|
m_editor->set_focus(true);
|
||||||
|
}
|
||||||
|
|
||||||
bool MainWidget::request_close()
|
bool MainWidget::request_close()
|
||||||
{
|
{
|
||||||
if (!editor().document().is_modified())
|
if (!editor().document().is_modified())
|
||||||
|
|
|
@ -25,6 +25,7 @@ class MainWidget final : public GUI::Widget {
|
||||||
public:
|
public:
|
||||||
virtual ~MainWidget() override;
|
virtual ~MainWidget() override;
|
||||||
bool read_file_and_close(int fd, String const& path);
|
bool read_file_and_close(int fd, String const& path);
|
||||||
|
void open_nonexistent_file(String const& path);
|
||||||
bool request_close();
|
bool request_close();
|
||||||
|
|
||||||
GUI::TextEditor& editor() { return *m_editor; }
|
GUI::TextEditor& editor() { return *m_editor; }
|
||||||
|
|
|
@ -36,12 +36,14 @@ int main(int argc, char** argv)
|
||||||
if (file_to_edit) {
|
if (file_to_edit) {
|
||||||
FileArgument parsed_argument(file_to_edit);
|
FileArgument parsed_argument(file_to_edit);
|
||||||
|
|
||||||
file_to_edit_full_path = Core::File::real_path_for(parsed_argument.filename());
|
file_to_edit_full_path = Core::File::absolute_path(parsed_argument.filename());
|
||||||
VERIFY(!file_to_edit_full_path.is_empty());
|
VERIFY(!file_to_edit_full_path.is_empty());
|
||||||
dbgln("unveil for: {}", file_to_edit_full_path);
|
if (Core::File::exists(parsed_argument.filename())) {
|
||||||
if (unveil(file_to_edit_full_path.characters(), "r") < 0) {
|
dbgln("unveil for: {}", file_to_edit_full_path);
|
||||||
perror("unveil");
|
if (unveil(file_to_edit_full_path.characters(), "r") < 0) {
|
||||||
return 1;
|
perror("unveil");
|
||||||
|
return 1;
|
||||||
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -107,17 +109,21 @@ int main(int argc, char** argv)
|
||||||
if (file_to_edit) {
|
if (file_to_edit) {
|
||||||
// A file name was passed, parse any possible line and column numbers included.
|
// A file name was passed, parse any possible line and column numbers included.
|
||||||
FileArgument parsed_argument(file_to_edit);
|
FileArgument parsed_argument(file_to_edit);
|
||||||
auto file = Core::File::open(file_to_edit_full_path, Core::OpenMode::ReadOnly);
|
if (Core::File::exists(file_to_edit_full_path)) {
|
||||||
|
auto file = Core::File::open(file_to_edit_full_path, Core::OpenMode::ReadOnly);
|
||||||
|
|
||||||
if (file.is_error()) {
|
if (file.is_error()) {
|
||||||
GUI::MessageBox::show_error(window, String::formatted("Opening \"{}\" failed: {}", file_to_edit_full_path, file.error()));
|
GUI::MessageBox::show_error(window, String::formatted("Opening \"{}\" failed: {}", file_to_edit_full_path, file.error()));
|
||||||
return 1;
|
return 1;
|
||||||
|
}
|
||||||
|
|
||||||
|
if (!text_widget.read_file_and_close(file.value()->leak_fd(), file_to_edit_full_path))
|
||||||
|
return 1;
|
||||||
|
|
||||||
|
text_widget.editor().set_cursor_and_focus_line(parsed_argument.line().value_or(1) - 1, parsed_argument.column().value_or(0));
|
||||||
|
} else {
|
||||||
|
text_widget.open_nonexistent_file(file_to_edit_full_path);
|
||||||
}
|
}
|
||||||
|
|
||||||
if (!text_widget.read_file_and_close(file.value()->leak_fd(), file_to_edit_full_path))
|
|
||||||
return 1;
|
|
||||||
|
|
||||||
text_widget.editor().set_cursor_and_focus_line(parsed_argument.line().value_or(1) - 1, parsed_argument.column().value_or(0));
|
|
||||||
}
|
}
|
||||||
text_widget.update_title();
|
text_widget.update_title();
|
||||||
|
|
||||||
|
|
Loading…
Add table
Add a link
Reference in a new issue