0ct0pu5/ladybird
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 2

LibGUI: Tweak API for getting the selected path

Browse source 
Return a String instead of a LexicalPath. Also call it a path instead
of a file since that's what we're really returning.
This commit is contained in:
Andreas Kling 2021-05-20 20:40:32 +02:00
parent 8a6c37deef
commit 3773e72752
Notes: sideshowbarker 2024-07-18 17:42:18 +09:00
2 changed files with 5 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
Userland/Libraries/LibGUI/FilePicker.cpp
View file

@ -37,7 +37,7 @@ Optional<String> FilePicker::get_open_filepath(Window* parent_window, const Stri
picker->set_title(window_title);
if (picker->exec() == Dialog::ExecOK) {
String file_path = picker->selected_file().string();
String file_path = picker->selected_file();
if (file_path.is_null())
return {};
@ -52,7 +52,7 @@ Optional<String> FilePicker::get_save_filepath(Window* parent_window, const Stri
auto picker = FilePicker::construct(parent_window, Mode::Save, String::formatted("{}.{}", title, extension), path);
if (picker->exec() == Dialog::ExecOK) {
String file_path = picker->selected_file().string();
String file_path = picker->selected_file();
if (file_path.is_null())
return {};
@ -255,7 +255,7 @@ void FilePicker::on_file_return()
return;
}
m_selected_file = path;
m_selected_file = path.string();
done(ExecOK);
}

4
Userland/Libraries/LibGUI/FilePicker.h
View file

@ -33,7 +33,7 @@ public:
virtual ~FilePicker() override;
LexicalPath selected_file() const { return m_selected_file; }
String const& selected_file() const { return m_selected_file; }
private:
void on_file_return();
@ -66,7 +66,7 @@ private:
RefPtr<MultiView> m_view;
NonnullRefPtr<FileSystemModel> m_model;
LexicalPath m_selected_file;
String m_selected_file;
RefPtr<TextBox> m_filename_textbox;
RefPtr<TextBox> m_location_textbox;