Skip to content

Commit

Permalink
Enable popup menu on sidebar and message window notebooks (#1726)
Browse files Browse the repository at this point in the history
  • Loading branch information
codebrainz committed Dec 22, 2017
1 parent a46d875 commit 1f71ccd
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion data/geany.glade
Original file line number Diff line number Diff line change
Expand Up @@ -6578,8 +6578,8 @@
<property name="can_focus">False</property>
<property name="label" translatable="yes">_Edit</property>
<property name="use_underline">True</property>
<signal name="select" handler="on_edit1_select" swapped="no"/>
<signal name="deselect" handler="on_edit1_deselect" swapped="no"/>
<signal name="select" handler="on_edit1_select" swapped="no"/>
<child type="submenu">
<object class="GtkMenu" id="edit1_menu">
<property name="can_focus">False</property>
Expand Down Expand Up @@ -8199,6 +8199,7 @@
<property name="visible">True</property>
<property name="can_focus">True</property>
<property name="scrollable">True</property>
<property name="enable_popup">True</property>
<signal name="key-press-event" handler="on_escape_key_press_event" swapped="no"/>
<signal name="switch-page" handler="on_tv_notebook_switch_page" swapped="no"/>
<signal name="switch-page" handler="on_tv_notebook_switch_page_after" after="yes" swapped="no"/>
Expand Down Expand Up @@ -8283,6 +8284,7 @@
<property name="can_focus">True</property>
<property name="tab_pos">left</property>
<property name="scrollable">True</property>
<property name="enable_popup">True</property>
<child>
<object class="GtkScrolledWindow" id="scrolledwindow4">
<property name="visible">True</property>
Expand Down

4 comments on commit 1f71ccd

@kugel-
Copy link
Member

@kugel- kugel- commented on 1f71ccd Jan 12, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This also enables the popup menu in the terminal widget, so I get two popup menus.

grafik

@codebrainz
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hmm, must be something funky about the VTE widget.

@codebrainz
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe this function needs to return TRUE for the two conditional branches?

@codebrainz
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yep. Fixed in 6622574

Please sign in to comment.