@b4n commented on this pull request.


>               gboolean ro = (response == GEANY_RESPONSE_VIEW);        /* View 
> clicked */
 
-               filesel_state.open.more_options_visible = 
gtk_expander_get_expanded(GTK_EXPANDER(expander));
-               filesel_state.open.filter_idx = 
file_chooser_get_filter_idx(GTK_FILE_CHOOSER(dialog));
-               filesel_state.open.filetype_idx = 
filetype_combo_box_get_active_filetype(GTK_COMBO_BOX(filetype_combo));
+               if (!GTK_IS_NATIVE_DIALOG(dialog))
+               {
+                       GtkWidget *expander = 
ui_lookup_widget(GTK_WIDGET(dialog), "more_options_expander");
+                       GtkWidget *filetype_combo = 
ui_lookup_widget(GTK_WIDGET(dialog), "filetype_combo");
+                       GtkWidget *encoding_combo = 
ui_lookup_widget(GTK_WIDGET(dialog), "encoding_combo");
+
+                       filesel_state.open.more_options_visible = 
gtk_expander_get_expanded(GTK_EXPANDER(expander));
+                       filesel_state.open.filter_idx = 
file_chooser_get_filter_idx(GTK_FILE_CHOOSER(dialog));

OK.  Sad (as we try to keep it) but not the end of the world indeed.  If it 
doesn't break anything I'd keep it for the sake of implementations that do 
support it.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/3861#discussion_r1632407130
You are receiving this because you are subscribed to this thread.

Message ID: <geany/geany/pull/3861/review/[email protected]>

Reply via email to