From efd1a6c2b82473e975e479a801c41b13fe0317f6 Mon Sep 17 00:00:00 2001 From: eidheim Date: Mon, 22 Jun 2015 19:28:45 +0200 Subject: [PATCH 01/10] Can now open files and directories using the command line, for instance: juci . source.cc notebook.* --- juci/juci.cc | 46 ++++++++++++++++++++++++++++++++++++++-------- juci/juci.h | 15 +++++++++++++++ 2 files changed, 53 insertions(+), 8 deletions(-) create mode 100644 juci/juci.h diff --git a/juci/juci.cc b/juci/juci.cc index cedf99d..5402b10 100644 --- a/juci/juci.cc +++ b/juci/juci.cc @@ -1,5 +1,4 @@ -#include "window.h" -#include "logging.h" +#include "juci.h" void init_logging() { add_common_attributes(); @@ -8,12 +7,43 @@ void init_logging() { INFO("Logging initalized"); } +int Juci::on_command_line(const Glib::RefPtr &cmd) { + Glib::set_prgname("juci"); + Glib::OptionContext ctx("[PATH ...]"); + Glib::OptionGroup gtk_group(gtk_get_option_group(true)); + ctx.add_group(gtk_group); + int argc; + char **argv = cmd->get_arguments(argc); + ctx.parse(argc, argv); + if(argc>=2) { + for(size_t c=1;c(new Window()); + add_window(*window); + window->show(); + if(directory!="") { + //TODO: use the following instead, window->notebook_.open_directory(directory); + window->notebook_.project_path=directory; + window->notebook_.directories().open_folder(directory); + } + for(auto &f: files) + window->notebook_.OnOpenFile(f); +} + int main(int argc, char *argv[]) { - Glib::RefPtr app = Gtk::Application::create( - argc, - argv, - "no.sout.juci"); init_logging(); - Window window; - return app->run(window); + return Juci().run(argc, argv); } diff --git a/juci/juci.h b/juci/juci.h new file mode 100644 index 0000000..9d55502 --- /dev/null +++ b/juci/juci.h @@ -0,0 +1,15 @@ +#include "window.h" +#include "logging.h" + +class Juci : public Gtk::Application { +public: + Juci(): Gtk::Application("no.sout.juci", Gio::APPLICATION_HANDLES_COMMAND_LINE) {} + + int on_command_line(const Glib::RefPtr &cmd); + void on_activate(); + +private: + std::unique_ptr window; + std::string directory; + std::vector files; +}; From e7b6e425a08c6ebee7486489b0479a85430f7c94 Mon Sep 17 00:00:00 2001 From: eidheim Date: Tue, 23 Jun 2015 14:00:13 +0200 Subject: [PATCH 02/10] Small change in setting and using Source::View::project_path. --- juci/source.cc | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/juci/source.cc b/juci/source.cc index 4b0b423..a623c40 100644 --- a/juci/source.cc +++ b/juci/source.cc @@ -181,7 +181,7 @@ get_autocomplete_suggestions(int line_number, int column) { std::vector Source::ClangView:: get_compilation_commands() { - clang::CompilationDatabase db(project_path+"/"); + clang::CompilationDatabase db(project_path); clang::CompileCommands commands(file_path, &db); std::vector cmds = commands.get_commands(); std::vector arguments; @@ -408,9 +408,7 @@ Source::Controller::Controller(const Source::Config &config, const std::string& file_path, std::string project_path) : config(config) { if(project_path=="") { - size_t last_of=file_path.find_last_of("\\/"); - if(last_of!=std::string::npos) - project_path=file_path.substr(0, last_of); + project_path=boost::filesystem::path(file_path).parent_path().string(); } if (config.legal_extension(file_path.substr(file_path.find_last_of(".") + 1))) view=std::unique_ptr(new ClangView(config, file_path, project_path)); From ecc56b66df656023ff8311a0ce4d2d7af2407e93 Mon Sep 17 00:00:00 2001 From: eidheim Date: Tue, 23 Jun 2015 14:15:53 +0200 Subject: [PATCH 03/10] Fixed annoying indentation issue when entering enter after a bracket. --- juci/source.cc | 12 ++++++++++++ 1 file changed, 12 insertions(+) diff --git a/juci/source.cc b/juci/source.cc index a623c40..8377728 100644 --- a/juci/source.cc +++ b/juci/source.cc @@ -288,6 +288,18 @@ bool Source::ClangView::on_key_press(GdkEventKey* key) { string line(get_line_before_insert()); std::smatch sm; if(std::regex_match(line, sm, bracket_regex)) { + size_t line_nr=get_source_buffer()->get_insert()->get_iter().get_line(); + if((line_nr+1)get_line_count()) { + string next_line=get_line(line_nr+1); + std::smatch sm2; + if(std::regex_match(next_line, sm2, spaces_regex)) { + if(sm2[1].str()==sm[1].str()+config.tab) { + get_source_buffer()->insert_at_cursor("\n"+sm[1].str()+config.tab); + scroll_to(get_source_buffer()->get_insert()); + return true; + } + } + } get_source_buffer()->insert_at_cursor("\n"+sm[1].str()+config.tab+"\n"+sm[1].str()+"}"); auto insert_it = get_source_buffer()->get_insert()->get_iter(); for(size_t c=0;c Date: Tue, 23 Jun 2015 17:47:47 +0200 Subject: [PATCH 04/10] Moved completiondialog-code to seperate file: selectiondialog.h/cc. The selectiondialog is now started from Source::ClangView. Some cleanup. See source.cc lines 340-345 for example use of SelectionDialog. --- juci/selectiondialog.cc | 85 +++++++++++++++++++++++++++++++++++++++++ juci/selectiondialog.h | 19 +++++++++ 2 files changed, 104 insertions(+) create mode 100644 juci/selectiondialog.cc create mode 100644 juci/selectiondialog.h diff --git a/juci/selectiondialog.cc b/juci/selectiondialog.cc new file mode 100644 index 0000000..b618c12 --- /dev/null +++ b/juci/selectiondialog.cc @@ -0,0 +1,85 @@ +#include "selectiondialog.h" + +SelectionDialog::SelectionDialog(Source::View& view): Gtk::Dialog(), view(view), + list_view_text(1, false, Gtk::SelectionMode::SELECTION_SINGLE) { + scrolled_window.set_policy(Gtk::PolicyType::POLICY_NEVER, Gtk::PolicyType::POLICY_NEVER); + list_view_text.set_enable_search(true); + list_view_text.set_headers_visible(false); + list_view_text.set_hscroll_policy(Gtk::ScrollablePolicy::SCROLL_NATURAL); + list_view_text.set_activate_on_single_click(true); +} + +void SelectionDialog::show(const std::map& rows) { + for (auto &i : rows) { + list_view_text.append(i.first); + } + scrolled_window.add(list_view_text); + get_vbox()->pack_start(scrolled_window); + set_transient_for((Gtk::Window&)(*view.get_toplevel())); + show_all(); + int popup_x = get_width(); + int popup_y = rows.size() * 20; + adjust(popup_x, popup_y); + + list_view_text.signal_row_activated().connect([this](const Gtk::TreeModel::Path& path, Gtk::TreeViewColumn*) { + if(on_select) + on_select(list_view_text); + response(Gtk::RESPONSE_DELETE_EVENT); + }); + + signal_focus_out_event().connect(sigc::mem_fun(*this, &SelectionDialog::close), false); + + run(); +} + +bool SelectionDialog::close(GdkEventFocus*) { + response(Gtk::RESPONSE_DELETE_EVENT); + return true; +} + +void SelectionDialog::adjust(int current_x, int current_y) { + INFO("SelectionDialog set size"); + int view_x = view.get_width(); + int view_y = 150; + bool is_never_scroll_x = true; + bool is_never_scroll_y = true; + if (current_x > view_x) { + current_x = view_x; + is_never_scroll_x = false; + } + if (current_y > view_y) { + current_y = view_y; + is_never_scroll_y = false; + } + scrolled_window.set_size_request(current_x, current_y); + if (!is_never_scroll_x && !is_never_scroll_y) { + scrolled_window.set_policy(Gtk::PolicyType::POLICY_AUTOMATIC, Gtk::PolicyType::POLICY_AUTOMATIC); + } else if (!is_never_scroll_x && is_never_scroll_y) { + scrolled_window.set_policy(Gtk::PolicyType::POLICY_AUTOMATIC, Gtk::PolicyType::POLICY_NEVER); + } else if (is_never_scroll_x && !is_never_scroll_y) { + scrolled_window.set_policy(Gtk::PolicyType::POLICY_NEVER, Gtk::PolicyType::POLICY_AUTOMATIC); + } + + INFO("SelectionDialog set position"); + Gdk::Rectangle temp1, temp2; + view.get_cursor_locations(view.get_source_buffer()->get_insert()->get_iter(), temp1, temp2); + int view_edge_x = 0; + int view_edge_y = 0; + int x, y; + view.buffer_to_window_coords(Gtk::TextWindowType::TEXT_WINDOW_WIDGET, + temp1.get_x(), temp1.get_y(), x, y); + Glib::RefPtr gdkw = view.get_window(Gtk::TextWindowType::TEXT_WINDOW_WIDGET); + gdkw->get_origin(view_edge_x, view_edge_y); + + x += view_edge_x; + y += view_edge_y; + if ((view_edge_x-x)*-1 > view.get_width()-current_x) { + x -= current_x; + if (x < view_edge_x) x = view_edge_x; + } + if ((view_edge_y-y)*-1 > view.get_height()-current_y) { + y -= (current_y+14) + 15; + if (x < view_edge_y) y = view_edge_y +15; + } + move(x, y+15); +} \ No newline at end of file diff --git a/juci/selectiondialog.h b/juci/selectiondialog.h new file mode 100644 index 0000000..6ea11c4 --- /dev/null +++ b/juci/selectiondialog.h @@ -0,0 +1,19 @@ +#include "gtkmm.h" +#include "logging.h" +#include "source.h" + +class SelectionDialog : public Gtk::Dialog { +public: + SelectionDialog(Source::View& view); + void show(const std::map& rows); + bool close(GdkEventFocus*); + + std::function on_select; + +private: + void adjust(int current_x, int current_y); + + Source::View& view; + Gtk::ScrolledWindow scrolled_window; + Gtk::ListViewText list_view_text; +}; \ No newline at end of file From d33df03dd7fa97360d79c93b4887739c31e1b711 Mon Sep 17 00:00:00 2001 From: eidheim Date: Tue, 23 Jun 2015 17:49:16 +0200 Subject: [PATCH 05/10] Moved completiondialog-code to seperate file: selectiondialog.h/cc. The selectiondialog is now started from Source::ClangView. Some cleanup. See source.cc lines 340-345 for example use of SelectionDialog. --- juci/CMakeLists.txt | 3 + juci/notebook.cc | 195 -------------------------------------------- juci/notebook.h | 17 ---- juci/source.cc | 72 ++++++++++++++++ juci/source.h | 1 + juci/window.cc | 7 -- juci/window.h | 1 - 7 files changed, 76 insertions(+), 220 deletions(-) diff --git a/juci/CMakeLists.txt b/juci/CMakeLists.txt index b7f4dfa..4dcf244 100644 --- a/juci/CMakeLists.txt +++ b/juci/CMakeLists.txt @@ -99,6 +99,7 @@ endif() # name of the executable on Windows will be example.exe add_executable(${project_name} #list of every needed file to create the executable + juci.h juci.cc keybindings.h keybindings.cc @@ -106,6 +107,8 @@ add_executable(${project_name} menu.cc source.h source.cc + selectiondialog.h + selectiondialog.cc config.h config.cc sourcefile.h diff --git a/juci/notebook.cc b/juci/notebook.cc index 177f104..b2fb0b8 100644 --- a/juci/notebook.cc +++ b/juci/notebook.cc @@ -22,7 +22,6 @@ Notebook::Controller::Controller(Gtk::Window* window, INFO("Create notebook"); window_ = window; refClipboard_ = Gtk::Clipboard::get(); - ispopup = false; view().pack1(directories_.widget(), true, true); CreateKeybindings(keybindings); INFO("Notebook Controller Success"); @@ -176,114 +175,6 @@ void Notebook::Controller::CreateKeybindings(Keybindings::Controller INFO("Notebook signal handlers sucsess"); } -bool Notebook::Controller:: OnMouseRelease(GdkEventButton* button) { - if (button->button == 1 && ispopup) { - popup_.response(Gtk::RESPONSE_DELETE_EVENT); - return true; - } - return false; -} - -bool Notebook::Controller::OnKeyRelease(GdkEventKey* key) { - return GeneratePopup(key->keyval); -} - -bool Notebook::Controller::GeneratePopup(int key_id) { - INFO("Notebook genereate popup, getting iters"); - std::string path = text_vec_.at(CurrentPage())->view->file_path; - if (!source_config().legal_extension(path.substr(path.find_last_of(".") + 1))) return false; - // Get function to fill popup with suggests item vector under is for testing - Gtk::TextIter beg = CurrentTextView().get_buffer()->get_insert()->get_iter(); - Gtk::TextIter end = CurrentTextView().get_buffer()->get_insert()->get_iter(); - Gtk::TextIter tmp = CurrentTextView().get_buffer()->get_insert()->get_iter(); - Gtk::TextIter tmp1 = CurrentTextView().get_buffer()->get_insert()->get_iter(); - Gtk::TextIter line = - CurrentTextView().get_buffer()->get_iter_at_line(tmp.get_line()); - if (end.backward_char() && end.backward_char()) { - bool illegal_chars = - end.backward_search("\"", Gtk::TEXT_SEARCH_VISIBLE_ONLY, tmp, tmp1, line) - || - end.backward_search("//", Gtk::TEXT_SEARCH_VISIBLE_ONLY, tmp, tmp1, line); - INFO("Notebook genereate popup, checking key_id"); - if (illegal_chars) { - return false; - } - std::string c = text_vec_[CurrentPage()]->buffer()->get_text(end, beg); - switch (key_id) { - case 46: - break; - case 58: - if (c != "::") return false; - break; - case 60: - if (c != "->") return false; - break; - case 62: - if (c != "->") return false; - break; - default: - return false; - } - } else { - return false; - } - INFO("Notebook genereate popup, getting autocompletions"); - std::vector acdata=text_vec_.at(CurrentPage())->view-> - get_autocomplete_suggestions(beg.get_line()+1, - beg.get_line_offset()+2); - std::map items; - for (auto &data : acdata) { - std::stringstream ss; - std::string return_value; - for (auto &chunk : data.chunks) { - switch (chunk.kind) { - case clang::CompletionChunk_ResultType: - return_value = chunk.chunk; - break; - case clang::CompletionChunk_Informative: break; - default: ss << chunk.chunk; break; - } - } - if (ss.str().length() > 0) { // if length is 0 the result is empty - items[ss.str() + " --> " + return_value] = ss.str(); - } - } - Gtk::ScrolledWindow popup_scroll_; - Gtk::ListViewText listview_(1, false, Gtk::SelectionMode::SELECTION_SINGLE); - popup_scroll_.set_policy(Gtk::PolicyType::POLICY_NEVER, - Gtk::PolicyType::POLICY_NEVER); - listview_.set_enable_search(true); - listview_.set_headers_visible(false); - listview_.set_hscroll_policy(Gtk::ScrollablePolicy::SCROLL_NATURAL); - listview_.set_activate_on_single_click(true); - if (items.empty()) { - items["No suggestions found..."] = ""; - } - for (auto &i : items) { - listview_.append(i.first); - } - popup_scroll_.add(listview_); - popup_.get_vbox()->pack_start(popup_scroll_); - popup_.set_transient_for(*window_); - popup_.show_all(); - INFO("Notebook genereate popup, moving popup"); - int popup_x = popup_.get_width(); - int popup_y = items.size() * 20; - PopupSetSize(popup_scroll_, popup_x, popup_y); - int x, y; - FindPopupPosition(CurrentTextView(), popup_x, popup_y, x, y); - popup_.move(x, y+15); - INFO("Notebook genereate popup, create handler"); - PopupSelectHandler(popup_, listview_, &items); - ispopup = true; - INFO("Notebook genereate popup, run popup"); - popup_.run(); - INFO("Notebook genereate popup, hide popup"); - popup_.hide(); - ispopup = false; - return true; -} - Notebook::Controller::~Controller() { INFO("Notebook destructor"); for (auto &i : editor_vec_) delete i; @@ -467,12 +358,6 @@ void Notebook::Controller::BufferChangeHandler(Glib::RefPtr } void Notebook::Controller::set_source_handlers(Source::Controller& controller) { - controller.view->signal_button_release_event(). - connect(sigc::mem_fun(*this, &Notebook::Controller::OnMouseRelease), false); - - controller.view->signal_key_release_event(). - connect(sigc::mem_fun(*this, &Notebook::Controller::OnKeyRelease), false); - //Add star on tab label when the page is not saved: controller.buffer()->signal_changed().connect([this]() { if(text_vec_.at(CurrentPage())->is_saved) { @@ -487,86 +372,6 @@ void Notebook::Controller::set_source_handlers(Source::Controller& controller) { }); } -void Notebook::Controller::PopupSelectHandler(Gtk::Dialog &popup, - Gtk::ListViewText &listview, - std::map - *items) { - listview.signal_row_activated(). - connect([this, &listview, &popup, items](const Gtk::TreeModel::Path& path, - Gtk::TreeViewColumn*) { - std::string selected = items-> - at(listview.get_text(listview.get_selected()[0])); - CurrentTextView().get_buffer()->insert_at_cursor(selected); - popup.response(Gtk::RESPONSE_DELETE_EVENT); - }); -} -void Notebook::Controller::PopupSetSize(Gtk::ScrolledWindow &scroll, - int ¤t_x, - int ¤t_y) { - INFO("Notebook popup set size"); - int textview_x = CurrentTextView().get_width(); - int textview_y = 150; - bool is_never_scroll_x = true; - bool is_never_scroll_y = true; - if (current_x > textview_x) { - current_x = textview_x; - is_never_scroll_x = false; - } - if (current_y > textview_y) { - current_y = textview_y; - is_never_scroll_y = false; - } - scroll.set_size_request(current_x, current_y); - if (!is_never_scroll_x && !is_never_scroll_y) { - scroll.set_policy(Gtk::PolicyType::POLICY_AUTOMATIC, - Gtk::PolicyType::POLICY_AUTOMATIC); - } else if (!is_never_scroll_x && is_never_scroll_y) { - scroll.set_policy(Gtk::PolicyType::POLICY_AUTOMATIC, - Gtk::PolicyType::POLICY_NEVER); - } else if (is_never_scroll_x && !is_never_scroll_y) { - scroll.set_policy(Gtk::PolicyType::POLICY_NEVER, - Gtk::PolicyType::POLICY_AUTOMATIC); - } -} - -std::string Notebook::Controller::CurrentPagePath(){ - return text_vec_.at(CurrentPage())->view->file_path; -} - -void Notebook::Controller::FindPopupPosition(Gtk::TextView& textview, - int popup_x, - int popup_y, - int &x, - int &y) { - INFO("Notebook popup find position"); - Gdk::Rectangle temp1, temp2; - textview.get_cursor_locations( - CurrentTextView(). - get_buffer()->get_insert()-> - get_iter(), temp1, temp2); - int textview_edge_x = 0; - int textview_edge_y = 0; - textview.buffer_to_window_coords( - Gtk::TextWindowType::TEXT_WINDOW_WIDGET, - temp1.get_x(), - temp1.get_y(), - x, y); - Glib::RefPtr gdkw = - CurrentTextView().get_window(Gtk::TextWindowType::TEXT_WINDOW_WIDGET); - gdkw->get_origin(textview_edge_x, textview_edge_y); - - x += textview_edge_x; - y += textview_edge_y; - if ((textview_edge_x-x)*-1 > textview.get_width()-popup_x) { - x -= popup_x; - if (x < textview_edge_x) x = textview_edge_x; - } - if ((textview_edge_y-y)*-1 > textview.get_height()-popup_y) { - y -= (popup_y+14) + 15; - if (x < textview_edge_y) y = textview_edge_y +15; - } -} - bool Notebook::Controller:: OnSaveFile() { std::string path=text_vec_.at(CurrentPage())->view->file_path; return OnSaveFile(path); diff --git a/juci/notebook.h b/juci/notebook.h index 0ffff31..21791ba 100644 --- a/juci/notebook.h +++ b/juci/notebook.h @@ -63,30 +63,15 @@ namespace Notebook { int Pages(); Directories::Controller& directories() { return directories_; } Gtk::Paned& view(); - bool GeneratePopup(int key); void Search(bool forward); Source::Config& source_config() { return source_config_; } - bool OnMouseRelease(GdkEventButton* button); - bool OnKeyRelease(GdkEventKey* key); std::string OnSaveFileAs(); bool LegalExtension(std::string extension); std::string project_path; protected: void set_source_handlers(Source::Controller& controller); - void PopupSelectHandler(Gtk::Dialog &popup, - Gtk::ListViewText &listview, - std::map - *items); private: void CreateKeybindings(Keybindings::Controller& keybindings); - void FindPopupPosition(Gtk::TextView& textview, - int popup_x, - int popup_y, - int &x, - int &y); - void PopupSetSize(Gtk::ScrolledWindow& scroll, - int ¤t_x, - int ¤t_y); void AskToSaveDialog(); Glib::RefPtr m_refBuilder; Glib::RefPtr refActionGroup; @@ -104,8 +89,6 @@ namespace Notebook { Gtk::TextIter search_match_end_; Gtk::TextIter search_match_start_; Glib::RefPtr refClipboard_; - bool ispopup; - Gtk::Dialog popup_; Gtk::Window* window_; }; // class controller } // namespace Notebook diff --git a/juci/source.cc b/juci/source.cc index 8377728..0a5ad6c 100644 --- a/juci/source.cc +++ b/juci/source.cc @@ -6,6 +6,7 @@ #include "logging.h" #include #include +#include "selectiondialog.h" bool Source::Config::legal_extension(std::string e) const { std::transform(e.begin(), e.end(),e.begin(), ::tolower); @@ -119,6 +120,7 @@ parse_thread_go(true), parse_thread_mapped(false), parse_thread_stop(false) { }); signal_key_press_event().connect(sigc::mem_fun(*this, &Source::ClangView::on_key_press), false); + signal_key_release_event().connect(sigc::mem_fun(*this, &Source::ClangView::on_key_release), false); } Source::ClangView::~ClangView() { @@ -275,6 +277,76 @@ highlight_token(clang::Token *token, end_offset), token_kind); } +bool Source::ClangView::on_key_release(GdkEventKey* key) { + INFO("Source::ClangView::on_key_release getting iters"); + // Get function to fill popup with suggests item vector under is for testing + Gtk::TextIter beg = get_source_buffer()->get_insert()->get_iter(); + Gtk::TextIter end = get_source_buffer()->get_insert()->get_iter(); + Gtk::TextIter tmp = get_source_buffer()->get_insert()->get_iter(); + Gtk::TextIter tmp1 = get_source_buffer()->get_insert()->get_iter(); + Gtk::TextIter line = get_source_buffer()->get_iter_at_line(tmp.get_line()); + if (end.backward_char() && end.backward_char()) { + bool illegal_chars = + end.backward_search("\"", Gtk::TEXT_SEARCH_VISIBLE_ONLY, tmp, tmp1, line) + || + end.backward_search("//", Gtk::TEXT_SEARCH_VISIBLE_ONLY, tmp, tmp1, line); + INFO("Source::ClangView::on_key_release checking key->keyval"); + if (illegal_chars) { + return false; + } + std::string c = get_source_buffer()->get_text(end, beg); + switch (key->keyval) { + case 46: + break; + case 58: + if (c != "::") return false; + break; + case 60: + if (c != "->") return false; + break; + case 62: + if (c != "->") return false; + break; + default: + return false; + } + } else { + return false; + } + INFO("Source::ClangView::on_key_release getting autocompletions"); + std::vector acdata=get_autocomplete_suggestions(beg.get_line()+1, + beg.get_line_offset()+2); + std::map rows; + for (auto &data : acdata) { + std::stringstream ss; + std::string return_value; + for (auto &chunk : data.chunks) { + switch (chunk.kind) { + case clang::CompletionChunk_ResultType: + return_value = chunk.chunk; + break; + case clang::CompletionChunk_Informative: break; + default: ss << chunk.chunk; break; + } + } + if (ss.str().length() > 0) { // if length is 0 the result is empty + rows[ss.str() + " --> " + return_value] = ss.str(); + } + } + if (rows.empty()) { + rows["No suggestions found..."] = ""; + } + + SelectionDialog selection_dialog(*this); + selection_dialog.on_select=[this, &rows](Gtk::ListViewText& list_view_text){ + std::string selected = rows.at(list_view_text.get_text(list_view_text.get_selected()[0])); + get_source_buffer()->insert_at_cursor(selected); + }; + selection_dialog.show(rows); + + return true; +} + //TODO: replace indentation methods with a better implementation or //maybe use libclang bool Source::ClangView::on_key_press(GdkEventKey* key) { diff --git a/juci/source.h b/juci/source.h index 1deba5c..7c7b2a2 100644 --- a/juci/source.h +++ b/juci/source.h @@ -103,6 +103,7 @@ namespace Source { std::vector *source_ranges); std::vector get_compilation_commands(); bool on_key_press(GdkEventKey* key); + bool on_key_release(GdkEventKey* key); Glib::Dispatcher parse_done; Glib::Dispatcher parse_start; diff --git a/juci/window.cc b/juci/window.cc index e139437..e3c116a 100644 --- a/juci/window.cc +++ b/juci/window.cc @@ -104,10 +104,6 @@ Window::Window() : execute.detach(); } }); - this->signal_button_release_event(). - connect(sigc::mem_fun(*this,&Window::OnMouseRelease),false); - terminal_.Terminal().signal_button_release_event(). - connect(sigc::mem_fun(*this,&Window::OnMouseRelease),false); add_accel_group(keybindings_.ui_manager_menu()->get_accel_group()); add_accel_group(keybindings_.ui_manager_hidden()->get_accel_group()); @@ -203,9 +199,6 @@ void Window::OnOpenFile() { } } } -bool Window::OnMouseRelease(GdkEventButton *button){ - return notebook_.OnMouseRelease(button); -} bool Window::SaveFile() { if(notebook_.OnSaveFile()) { diff --git a/juci/window.h b/juci/window.h index d60dfdb..344a51e 100644 --- a/juci/window.h +++ b/juci/window.h @@ -32,7 +32,6 @@ public: void OnWindowHide(); void OnOpenFile(); void OnFileOpenFolder(); - bool OnMouseRelease(GdkEventButton* button); bool SaveFile(); bool SaveFileAs(); }; From 27cff8be26cff7da7381a1bcb07250f98e35c326 Mon Sep 17 00:00:00 2001 From: eidheim Date: Tue, 23 Jun 2015 18:00:11 +0200 Subject: [PATCH 06/10] Did not intend to delete this function in last commit. --- juci/notebook.cc | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/juci/notebook.cc b/juci/notebook.cc index b2fb0b8..1042816 100644 --- a/juci/notebook.cc +++ b/juci/notebook.cc @@ -372,6 +372,10 @@ void Notebook::Controller::set_source_handlers(Source::Controller& controller) { }); } +std::string Notebook::Controller::CurrentPagePath(){ + return text_vec_.at(CurrentPage())->parser.file_path; +} + bool Notebook::Controller:: OnSaveFile() { std::string path=text_vec_.at(CurrentPage())->view->file_path; return OnSaveFile(path); From 45706a3905cc634709e67f8b0721d6a7f8d49331 Mon Sep 17 00:00:00 2001 From: eidheim Date: Tue, 23 Jun 2015 18:02:01 +0200 Subject: [PATCH 07/10] Small fix, copied old code. --- juci/notebook.cc | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/juci/notebook.cc b/juci/notebook.cc index 1042816..a813162 100644 --- a/juci/notebook.cc +++ b/juci/notebook.cc @@ -373,7 +373,7 @@ void Notebook::Controller::set_source_handlers(Source::Controller& controller) { } std::string Notebook::Controller::CurrentPagePath(){ - return text_vec_.at(CurrentPage())->parser.file_path; + return text_vec_.at(CurrentPage())->view->file_path; } bool Notebook::Controller:: OnSaveFile() { From 832ef8881cf14ff4af28394bba0cbadd9c81a4de Mon Sep 17 00:00:00 2001 From: eidheim Date: Tue, 23 Jun 2015 23:07:33 +0200 Subject: [PATCH 08/10] Made SelectionDialog more general, now receiving Gtk::TextView& instead. Minor cleanup in source.*. --- juci/selectiondialog.cc | 16 ++++---- juci/selectiondialog.h | 4 +- juci/source.cc | 86 +++++++++++++++++++++-------------------- juci/source.h | 23 ++++++----- 4 files changed, 68 insertions(+), 61 deletions(-) diff --git a/juci/selectiondialog.cc b/juci/selectiondialog.cc index b618c12..8eca51d 100644 --- a/juci/selectiondialog.cc +++ b/juci/selectiondialog.cc @@ -1,6 +1,6 @@ #include "selectiondialog.h" -SelectionDialog::SelectionDialog(Source::View& view): Gtk::Dialog(), view(view), +SelectionDialog::SelectionDialog(Gtk::TextView& text_view): Gtk::Dialog(), text_view(text_view), list_view_text(1, false, Gtk::SelectionMode::SELECTION_SINGLE) { scrolled_window.set_policy(Gtk::PolicyType::POLICY_NEVER, Gtk::PolicyType::POLICY_NEVER); list_view_text.set_enable_search(true); @@ -15,7 +15,7 @@ void SelectionDialog::show(const std::map& rows) { } scrolled_window.add(list_view_text); get_vbox()->pack_start(scrolled_window); - set_transient_for((Gtk::Window&)(*view.get_toplevel())); + set_transient_for((Gtk::Window&)(*text_view.get_toplevel())); show_all(); int popup_x = get_width(); int popup_y = rows.size() * 20; @@ -39,7 +39,7 @@ bool SelectionDialog::close(GdkEventFocus*) { void SelectionDialog::adjust(int current_x, int current_y) { INFO("SelectionDialog set size"); - int view_x = view.get_width(); + int view_x = text_view.get_width(); int view_y = 150; bool is_never_scroll_x = true; bool is_never_scroll_y = true; @@ -62,22 +62,22 @@ void SelectionDialog::adjust(int current_x, int current_y) { INFO("SelectionDialog set position"); Gdk::Rectangle temp1, temp2; - view.get_cursor_locations(view.get_source_buffer()->get_insert()->get_iter(), temp1, temp2); + text_view.get_cursor_locations(text_view.get_buffer()->get_insert()->get_iter(), temp1, temp2); int view_edge_x = 0; int view_edge_y = 0; int x, y; - view.buffer_to_window_coords(Gtk::TextWindowType::TEXT_WINDOW_WIDGET, + text_view.buffer_to_window_coords(Gtk::TextWindowType::TEXT_WINDOW_WIDGET, temp1.get_x(), temp1.get_y(), x, y); - Glib::RefPtr gdkw = view.get_window(Gtk::TextWindowType::TEXT_WINDOW_WIDGET); + Glib::RefPtr gdkw = text_view.get_window(Gtk::TextWindowType::TEXT_WINDOW_WIDGET); gdkw->get_origin(view_edge_x, view_edge_y); x += view_edge_x; y += view_edge_y; - if ((view_edge_x-x)*-1 > view.get_width()-current_x) { + if ((view_edge_x-x)*-1 > text_view.get_width()-current_x) { x -= current_x; if (x < view_edge_x) x = view_edge_x; } - if ((view_edge_y-y)*-1 > view.get_height()-current_y) { + if ((view_edge_y-y)*-1 > text_view.get_height()-current_y) { y -= (current_y+14) + 15; if (x < view_edge_y) y = view_edge_y +15; } diff --git a/juci/selectiondialog.h b/juci/selectiondialog.h index 6ea11c4..70e16de 100644 --- a/juci/selectiondialog.h +++ b/juci/selectiondialog.h @@ -4,7 +4,7 @@ class SelectionDialog : public Gtk::Dialog { public: - SelectionDialog(Source::View& view); + SelectionDialog(Gtk::TextView& text_view); void show(const std::map& rows); bool close(GdkEventFocus*); @@ -13,7 +13,7 @@ public: private: void adjust(int current_x, int current_y); - Source::View& view; + Gtk::TextView& text_view; Gtk::ScrolledWindow scrolled_window; Gtk::ListViewText list_view_text; }; \ No newline at end of file diff --git a/juci/source.cc b/juci/source.cc index 0a5ad6c..b7d794d 100644 --- a/juci/source.cc +++ b/juci/source.cc @@ -21,7 +21,7 @@ bool Source::Config::legal_extension(std::string e) const { ////////////// //// View //// ////////////// -Source::View::View(const Config& config, const std::string& file_path, const std::string& project_path): +Source::View::View(const Source::Config& config, const std::string& file_path, const std::string& project_path): config(config), file_path(file_path), project_path(project_path) { Gsv::init(); set_smart_home_end(Gsv::SMART_HOME_END_BEFORE); @@ -50,13 +50,52 @@ string Source::View::get_line_before_insert() { return line; } +bool Source::View::on_key_press(GdkEventKey* key) { + //Indent right when clicking tab, no matter where in the line the cursor is. Also works on selected text. + if(key->keyval==GDK_KEY_Tab && key->state==0) { + Gtk::TextIter selection_start, selection_end; + get_source_buffer()->get_selection_bounds(selection_start, selection_end); + int line_start=selection_start.get_line(); + int line_end=selection_end.get_line(); + for(int line=line_start;line<=line_end;line++) { + Gtk::TextIter line_it = get_source_buffer()->get_iter_at_line(line); + get_source_buffer()->insert(line_it, config.tab); + } + return true; + } + //Indent left when clicking shift-tab, no matter where in the line the cursor is. Also works on selected text. + else if((key->keyval==GDK_KEY_ISO_Left_Tab || key->keyval==GDK_KEY_Tab) && key->state==GDK_SHIFT_MASK) { + Gtk::TextIter selection_start, selection_end; + get_source_buffer()->get_selection_bounds(selection_start, selection_end); + int line_start=selection_start.get_line(); + int line_end=selection_end.get_line(); + + for(int line_nr=line_start;line_nr<=line_end;line_nr++) { + string line=get_line(line_nr); + if(!(line.size()>=config.tab_size && line.substr(0, config.tab_size)==config.tab)) + return true; + } + + for(int line_nr=line_start;line_nr<=line_end;line_nr++) { + Gtk::TextIter line_it = get_source_buffer()->get_iter_at_line(line_nr); + Gtk::TextIter line_plus_it=line_it; + + for(unsigned c=0;cerase(line_it, line_plus_it); + } + return true; + } + return false; +} + ////////////////// //// ClangView /// ////////////////// clang::Index Source::ClangView::clang_index(0, 1); -Source::ClangView::ClangView(const Config& config, const std::string& file_path, const std::string& project_path): -Source::View::View(config, file_path, project_path), +Source::ClangView::ClangView(const Source::Config& config, const std::string& file_path, const std::string& project_path): +Source::View(config, file_path, project_path), parse_thread_go(true), parse_thread_mapped(false), parse_thread_stop(false) { override_font(Pango::FontDescription(config.font)); override_background_color(Gdk::RGBA(config.background)); @@ -410,41 +449,6 @@ bool Source::ClangView::on_key_press(GdkEventKey* key) { } return true; } - //Indent right when clicking tab, no matter where in the line the cursor is. Also works on selected text. - if(key->keyval==GDK_KEY_Tab && key->state==0) { - Gtk::TextIter selection_start, selection_end; - get_source_buffer()->get_selection_bounds(selection_start, selection_end); - int line_start=selection_start.get_line(); - int line_end=selection_end.get_line(); - for(int line=line_start;line<=line_end;line++) { - Gtk::TextIter line_it = get_source_buffer()->get_iter_at_line(line); - get_source_buffer()->insert(line_it, config.tab); - } - return true; - } - //Indent left when clicking shift-tab, no matter where in the line the cursor is. Also works on selected text. - else if((key->keyval==GDK_KEY_ISO_Left_Tab || key->keyval==GDK_KEY_Tab) && key->state==GDK_SHIFT_MASK) { - Gtk::TextIter selection_start, selection_end; - get_source_buffer()->get_selection_bounds(selection_start, selection_end); - int line_start=selection_start.get_line(); - int line_end=selection_end.get_line(); - - for(int line_nr=line_start;line_nr<=line_end;line_nr++) { - string line=get_line(line_nr); - if(!(line.size()>=config.tab_size && line.substr(0, config.tab_size)==config.tab)) - return true; - } - - for(int line_nr=line_start;line_nr<=line_end;line_nr++) { - Gtk::TextIter line_it = get_source_buffer()->get_iter_at_line(line_nr); - Gtk::TextIter line_plus_it=line_it; - - for(unsigned c=0;cerase(line_it, line_plus_it); - } - return true; - } //Indent left when writing } on a new line else if(key->keyval==GDK_KEY_braceright) { string line=get_line_before_insert(); @@ -479,7 +483,8 @@ bool Source::ClangView::on_key_press(GdkEventKey* key) { } } } - return false; + + return Source::View::on_key_press(key); } //////////////////// @@ -489,8 +494,7 @@ bool Source::ClangView::on_key_press(GdkEventKey* key) { // Source::Controller::Controller() // Constructor for Controller Source::Controller::Controller(const Source::Config &config, - const std::string& file_path, std::string project_path) : - config(config) { + const std::string& file_path, std::string project_path) { if(project_path=="") { project_path=boost::filesystem::path(file_path).parent_path().string(); } diff --git a/juci/source.h b/juci/source.h index 7c7b2a2..e779c35 100644 --- a/juci/source.h +++ b/juci/source.h @@ -60,25 +60,31 @@ namespace Source { class View : public Gsv::View { public: - View(const Config& config, const std::string& file_path, const std::string& project_path); + View(const Source::Config& config, const std::string& file_path, const std::string& project_path); std::string get_line(size_t line_number); std::string get_line_before_insert(); - virtual std::vector get_autocomplete_suggestions(int line_number, int column) {return std::vector();} std::string file_path; std::string project_path; protected: - const Config& config; + const Source::Config& config; + bool on_key_press(GdkEventKey* key); }; // class View class GenericView : public View { public: - GenericView(const Config& config, const std::string& file_path, const std::string& project_path): - View(config, file_path, project_path) {} + GenericView(const Source::Config& config, const std::string& file_path, const std::string& project_path): + View(config, file_path, project_path) { + signal_key_press_event().connect(sigc::mem_fun(*this, &Source::GenericView::on_key_press), false); + } + private: + bool on_key_press(GdkEventKey* key) { + return Source::View::on_key_press(key); + } }; class ClangView : public View { public: - ClangView(const Config& config, const std::string& file_path, const std::string& project_path); + ClangView(const Source::Config& config, const std::string& file_path, const std::string& project_path); ~ClangView(); // inits the syntax highligthing on file open void init_syntax_highlighting(const std::map @@ -123,10 +129,7 @@ namespace Source { bool is_saved = true; - std::unique_ptr view; - - private: - const Config& config; + std::unique_ptr view; }; // class Controller } // namespace Source #endif // JUCI_SOURCE_H_ From 92e2417787e5f02d0513bb93e52d60aef7ab1c73 Mon Sep 17 00:00:00 2001 From: eidheim Date: Wed, 24 Jun 2015 11:14:15 +0200 Subject: [PATCH 09/10] Moved basic indentation to Source::View, Clang specific indentation still in Source::ClangView. Some fixes, improved indentation for non-clang files, also added tab_char to Source::Config. --- juci/source.cc | 80 ++++++++++++++++++++++++++++++++------------------ juci/source.h | 1 + 2 files changed, 53 insertions(+), 28 deletions(-) diff --git a/juci/source.cc b/juci/source.cc index b7d794d..1993db2 100644 --- a/juci/source.cc +++ b/juci/source.cc @@ -50,9 +50,33 @@ string Source::View::get_line_before_insert() { return line; } +//Basic indentation bool Source::View::on_key_press(GdkEventKey* key) { + const std::regex spaces_regex(std::string("^(")+config.tab_char+"*).*$"); + //Indent as in next or previous line + if(key->keyval==GDK_KEY_Return && key->state==0) { + int line_nr=get_source_buffer()->get_insert()->get_iter().get_line(); + string line(get_line_before_insert()); + std::smatch sm; + if(std::regex_match(line, sm, spaces_regex)) { + if((line_nr+1)get_line_count()) { + string next_line=get_line(line_nr+1); + std::smatch sm2; + if(std::regex_match(next_line, sm2, spaces_regex)) { + if(sm2[1].str().size()>sm[1].str().size()) { + get_source_buffer()->insert_at_cursor("\n"+sm2[1].str()); + scroll_to(get_source_buffer()->get_insert()); + return true; + } + } + } + get_source_buffer()->insert_at_cursor("\n"+sm[1].str()); + scroll_to(get_source_buffer()->get_insert()); + return true; + } + } //Indent right when clicking tab, no matter where in the line the cursor is. Also works on selected text. - if(key->keyval==GDK_KEY_Tab && key->state==0) { + else if(key->keyval==GDK_KEY_Tab && key->state==0) { Gtk::TextIter selection_start, selection_end; get_source_buffer()->get_selection_bounds(selection_start, selection_end); int line_start=selection_start.get_line(); @@ -86,6 +110,22 @@ bool Source::View::on_key_press(GdkEventKey* key) { } return true; } + //"Smart" backspace key + else if(key->keyval==GDK_KEY_BackSpace) { + Gtk::TextIter insert_it=get_source_buffer()->get_insert()->get_iter(); + int line_nr=insert_it.get_line(); + if(line_nr>0) { + string line=get_line(line_nr); + string previous_line=get_line(line_nr-1); + smatch sm; + if(std::regex_match(previous_line, sm, spaces_regex)) { + if(line==sm[1] || line==(std::string(sm[1])+config.tab) || (line+config.tab==sm[1])) { + Gtk::TextIter line_it = get_source_buffer()->get_iter_at_line(line_nr); + get_source_buffer()->erase(line_it, insert_it); + } + } + } + } return false; } @@ -386,15 +426,15 @@ bool Source::ClangView::on_key_release(GdkEventKey* key) { return true; } -//TODO: replace indentation methods with a better implementation or -//maybe use libclang +//Clang indentation +//TODO: replace indentation methods with a better implementation or maybe use libclang bool Source::ClangView::on_key_press(GdkEventKey* key) { - const std::regex bracket_regex("^( *).*\\{ *$"); - const std::regex no_bracket_statement_regex("^( *)(if|for|else if|catch|while) *\\(.*[^;}] *$"); - const std::regex no_bracket_no_para_statement_regex("^( *)(else|try|do) *$"); - const std::regex spaces_regex("^( *).*$"); + const std::regex bracket_regex(std::string("^(")+config.tab_char+"*).*\\{ *$"); + const std::regex no_bracket_statement_regex(std::string("^(")+config.tab_char+"*)(if|for|else if|catch|while) *\\(.*[^;}] *$"); + const std::regex no_bracket_no_para_statement_regex(std::string("^(")+config.tab_char+"*)(else|try|do) *$"); + const std::regex spaces_regex(std::string("^(")+config.tab_char+"*).*$"); - //Indent as in previous line, and indent right after if/else/etc + //Indent depending on if/else/etc and brackets if(key->keyval==GDK_KEY_Return && key->state==0) { string line(get_line_before_insert()); std::smatch sm; @@ -417,14 +457,17 @@ bool Source::ClangView::on_key_press(GdkEventKey* key) { insert_it--; scroll_to(get_source_buffer()->get_insert()); get_source_buffer()->place_cursor(insert_it); + return true; } else if(std::regex_match(line, sm, no_bracket_statement_regex)) { get_source_buffer()->insert_at_cursor("\n"+sm[1].str()+config.tab); scroll_to(get_source_buffer()->get_insert()); + return true; } else if(std::regex_match(line, sm, no_bracket_no_para_statement_regex)) { get_source_buffer()->insert_at_cursor("\n"+sm[1].str()+config.tab); scroll_to(get_source_buffer()->get_insert()); + return true; } else if(std::regex_match(line, sm, spaces_regex)) { std::smatch sm2; @@ -444,17 +487,14 @@ bool Source::ClangView::on_key_press(GdkEventKey* key) { } } } - get_source_buffer()->insert_at_cursor("\n"+sm[1].str()); - scroll_to(get_source_buffer()->get_insert()); } - return true; } //Indent left when writing } on a new line else if(key->keyval==GDK_KEY_braceright) { string line=get_line_before_insert(); if(line.size()>=config.tab_size) { for(auto c: line) { - if(c!=' ') + if(c!=config.tab_char) return false; } Gtk::TextIter insert_it = get_source_buffer()->get_insert()->get_iter(); @@ -467,22 +507,6 @@ bool Source::ClangView::on_key_press(GdkEventKey* key) { } return false; } - //"Smart" backspace key - else if(key->keyval==GDK_KEY_BackSpace) { - Gtk::TextIter insert_it=get_source_buffer()->get_insert()->get_iter(); - int line_nr=insert_it.get_line(); - if(line_nr>0) { - string line=get_line(line_nr); - string previous_line=get_line(line_nr-1); - smatch sm; - if(std::regex_match(previous_line, sm, spaces_regex)) { - if(line==sm[1]) { - Gtk::TextIter line_it = get_source_buffer()->get_iter_at_line(line_nr); - get_source_buffer()->erase(line_it, insert_it); - } - } - } - } return Source::View::on_key_press(key); } diff --git a/juci/source.h b/juci/source.h index e779c35..52d754f 100644 --- a/juci/source.h +++ b/juci/source.h @@ -22,6 +22,7 @@ namespace Source { unsigned tab_size; bool show_line_numbers, highlight_current_line; std::string tab, background, font; + char tab_char=' '; std::vector extensions; std::unordered_map tags, types; }; // class Config From 749e28a25942421620ac32734f024a9687897dc1 Mon Sep 17 00:00:00 2001 From: eidheim Date: Wed, 24 Jun 2015 13:01:06 +0200 Subject: [PATCH 10/10] Started cleaning up notebook.*. Fixed copy of Notebook::Controller::source_config. --- juci/juci.cc | 2 +- juci/juci.h | 5 +++++ juci/notebook.cc | 50 +++++++++++++++++++----------------------- juci/notebook.h | 11 +++------- juci/selectiondialog.h | 7 +++++- juci/source.h | 4 ---- juci/terminal.cc | 2 -- juci/window.cc | 4 ++-- 8 files changed, 40 insertions(+), 45 deletions(-) diff --git a/juci/juci.cc b/juci/juci.cc index 5402b10..1a6ec46 100644 --- a/juci/juci.cc +++ b/juci/juci.cc @@ -37,7 +37,7 @@ void Juci::on_activate() { if(directory!="") { //TODO: use the following instead, window->notebook_.open_directory(directory); window->notebook_.project_path=directory; - window->notebook_.directories().open_folder(directory); + window->notebook_.directories.open_folder(directory); } for(auto &f: files) window->notebook_.OnOpenFile(f); diff --git a/juci/juci.h b/juci/juci.h index 9d55502..4ce2cb2 100644 --- a/juci/juci.h +++ b/juci/juci.h @@ -1,3 +1,6 @@ +#ifndef JUCI_JUCI_H_ +#define JUCI_JUCI_H_ + #include "window.h" #include "logging.h" @@ -13,3 +16,5 @@ private: std::string directory; std::vector files; }; + +#endif // JUCI_JUCI_H_ \ No newline at end of file diff --git a/juci/notebook.cc b/juci/notebook.cc index a813162..d0ba733 100644 --- a/juci/notebook.cc +++ b/juci/notebook.cc @@ -17,12 +17,12 @@ Notebook::Controller::Controller(Gtk::Window* window, Keybindings::Controller& keybindings, Source::Config& source_cfg, Directories::Config& dir_cfg) : - directories_(dir_cfg), - source_config_(source_cfg) { + directories(dir_cfg), + source_config(source_cfg) { INFO("Create notebook"); window_ = window; refClipboard_ = Gtk::Clipboard::get(); - view().pack1(directories_.widget(), true, true); + view().pack1(directories.widget(), true, true); CreateKeybindings(keybindings); INFO("Notebook Controller Success"); } // Constructor @@ -31,7 +31,7 @@ Notebook::Controller::Controller(Gtk::Window* window, void Notebook::Controller::CreateKeybindings(Keybindings::Controller &keybindings) { INFO("Notebook create signal handlers"); - directories().m_TreeView.signal_row_activated() + directories.m_TreeView.signal_row_activated() .connect(sigc::mem_fun(*this, &Notebook::Controller::OnDirectoryNavigation)); @@ -191,7 +191,7 @@ Gtk::Box& Notebook::Controller::entry_view() { void Notebook::Controller::OnOpenFile(std::string path) { INFO("Notebook open file"); INFO("Notebook create page"); - text_vec_.emplace_back(new Source::Controller(source_config(), path, project_path)); + text_vec_.emplace_back(new Source::Controller(source_config, path, project_path)); scrolledtext_vec_.push_back(new Gtk::ScrolledWindow()); editor_vec_.push_back(new Gtk::HBox()); scrolledtext_vec_.back()->add(*text_vec_.back()->view); @@ -204,7 +204,18 @@ void Notebook::Controller::OnOpenFile(std::string path) { Notebook().show_all_children(); Notebook().set_current_page(Pages()-1); Notebook().set_focus_child(*text_vec_.back()->view); - set_source_handlers(*text_vec_.back()); + //Add star on tab label when the page is not saved: + text_vec_.back()->buffer()->signal_changed().connect([this]() { + if(text_vec_.at(CurrentPage())->is_saved) { + std::string path=text_vec_.at(CurrentPage())->view->file_path; + size_t pos = path.find_last_of("/\\"); + std::string filename=path; + if(pos!=std::string::npos) + filename=path.substr(pos+1); + Notebook().set_tab_label_text(*(Notebook().get_nth_page(CurrentPage())), filename+"*"); + } + text_vec_.at(CurrentPage())->is_saved=false; + }); } void Notebook::Controller::OnCloseCurrentPage() { @@ -310,18 +321,18 @@ void Notebook::Controller ::OnDirectoryNavigation(const Gtk::TreeModel::Path& path, Gtk::TreeViewColumn* column) { INFO("Notebook directory navigation"); - Gtk::TreeModel::iterator iter = directories().m_refTreeModel->get_iter(path); + Gtk::TreeModel::iterator iter = directories.m_refTreeModel->get_iter(path); if (iter) { Gtk::TreeModel::Row row = *iter; - std::string upath = Glib::ustring(row[directories().view().m_col_path]); + std::string upath = Glib::ustring(row[directories.view().m_col_path]); boost::filesystem::path fs_path(upath); if (boost::filesystem::is_directory(fs_path)) { - directories().m_TreeView.row_expanded(path) ? - directories().m_TreeView.collapse_row(path) : - directories().m_TreeView.expand_row(path, false); + directories.m_TreeView.row_expanded(path) ? + directories.m_TreeView.collapse_row(path) : + directories.m_TreeView.expand_row(path, false); } else { std::stringstream sstm; - sstm << row[directories().view().m_col_path]; + sstm << row[directories.view().m_col_path]; std::string file = sstm.str(); OnOpenFile(file); } @@ -357,21 +368,6 @@ void Notebook::Controller::BufferChangeHandler(Glib::RefPtr }); } -void Notebook::Controller::set_source_handlers(Source::Controller& controller) { - //Add star on tab label when the page is not saved: - controller.buffer()->signal_changed().connect([this]() { - if(text_vec_.at(CurrentPage())->is_saved) { - std::string path=text_vec_.at(CurrentPage())->view->file_path; - size_t pos = path.find_last_of("/\\"); - std::string filename=path; - if(pos!=std::string::npos) - filename=path.substr(pos+1); - Notebook().set_tab_label_text(*(Notebook().get_nth_page(CurrentPage())), filename+"*"); - } - text_vec_.at(CurrentPage())->is_saved=false; - }); -} - std::string Notebook::Controller::CurrentPagePath(){ return text_vec_.at(CurrentPage())->view->file_path; } diff --git a/juci/notebook.h b/juci/notebook.h index 21791ba..98af702 100644 --- a/juci/notebook.h +++ b/juci/notebook.h @@ -61,25 +61,20 @@ namespace Notebook { void OnOpenFile(std::string filename); bool ScrollEventCallback(GdkEventScroll* scroll_event); int Pages(); - Directories::Controller& directories() { return directories_; } Gtk::Paned& view(); void Search(bool forward); - Source::Config& source_config() { return source_config_; } std::string OnSaveFileAs(); - bool LegalExtension(std::string extension); std::string project_path; - protected: - void set_source_handlers(Source::Controller& controller); + Directories::Controller directories; private: void CreateKeybindings(Keybindings::Controller& keybindings); void AskToSaveDialog(); Glib::RefPtr m_refBuilder; Glib::RefPtr refActionGroup; - Source::Config source_config_; - Directories::Controller directories_; + Source::Config& source_config; View view_; Model model_; - bool is_new_file_; + bool is_new_file_; //TODO: Remove this Entry::Controller entry_; std::vector > text_vec_; diff --git a/juci/selectiondialog.h b/juci/selectiondialog.h index 70e16de..f8479ce 100644 --- a/juci/selectiondialog.h +++ b/juci/selectiondialog.h @@ -1,3 +1,6 @@ +#ifndef JUCI_SELECTIONDIALOG_H_ +#define JUCI_SELECTIONDIALOG_H_ + #include "gtkmm.h" #include "logging.h" #include "source.h" @@ -16,4 +19,6 @@ private: Gtk::TextView& text_view; Gtk::ScrolledWindow scrolled_window; Gtk::ListViewText list_view_text; -}; \ No newline at end of file +}; + +#endif // JUCI_SELECTIONDIALOG_H_ \ No newline at end of file diff --git a/juci/source.h b/juci/source.h index 52d754f..dce111d 100644 --- a/juci/source.h +++ b/juci/source.h @@ -11,10 +11,6 @@ #include #include "gtksourceviewmm.h" -namespace Notebook { - class Controller; -} - namespace Source { class Config { public: diff --git a/juci/terminal.cc b/juci/terminal.cc index 83f8205..d6b653d 100644 --- a/juci/terminal.cc +++ b/juci/terminal.cc @@ -87,11 +87,9 @@ void Terminal::Controller::ExecuteCommand(std::string command, std::string mode) FILE* p = NULL; std::cout << command << std::endl; p = popen(command.c_str(), mode.c_str()); - std::cout << "KJØRTE FINT!" << std::endl; if (p == NULL) { PrintMessage("juCi++ ERROR: Failed to run command" + command + "\n"); }else { - std::cout << "SKRIVER UT KOMMANDO RESULAT" << std::endl; char buffer[1028]; while (fgets(buffer, 1028, p) != NULL) { PrintMessage(buffer); diff --git a/juci/window.cc b/juci/window.cc index e3c116a..47ef193 100644 --- a/juci/window.cc +++ b/juci/window.cc @@ -73,7 +73,7 @@ Window::Window() : terminal_.SetFolderCommand(path); } terminal_.Compile(); - std::string executable = notebook_.directories(). + std::string executable = notebook_.directories. GetCmakeVarValue(path,"add_executable"); terminal_.Run(executable); running.unlock(); @@ -140,7 +140,7 @@ void Window::OnFileOpenFolder() { { std::string project_path=dialog.get_filename(); notebook_.project_path=project_path; - notebook_.directories().open_folder(project_path); + notebook_.directories.open_folder(project_path); break; } case(Gtk::RESPONSE_CANCEL):