@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));

FWIW, the *show hidden file* would also be implemented that way, but isn't 
actually needed for the GTK portal implementation because it remembers this 
itself.  Not sure if it's the same for Windows and macos?  If it's not, it's 
easy enough to add.

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

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

Reply via email to