master #30

Merged
asmeron merged 6 commits from YanTheKaller/ubl-settings-repomanager:master into master 2 years ago
Collaborator
There is no content yet.
YanTheKaller added 4 commits 2 years ago
user-166 requested changes 2 years ago
GtkWidget *chooser = gtk_file_chooser_dialog_new(LOAD_REPO_LABEL,NULL,GTK_FILE_CHOOSER_ACTION_SELECT_FOLDER,CANCEL_LABEL,GTK_RESPONSE_CANCEL,OPEN_LABEL,GTK_RESPONSE_ACCEPT,NULL);
// gtk_file_filter_add_pattern(file_filter,"*db.tar.gz");
// gtk_file_filter_set_name(file_filter, "*db.tar.gz");
// gtk_file_chooser_add_filter(GTK_FILE_CHOOSER(chooser),file_filter);
Collaborator

С какой целью фильр типа файла закомментирован?

С какой целью фильр типа файла закомментирован?
Poster
Collaborator

Заменены на отдельную фильтрующую функцию. Убрал

Заменены на отдельную фильтрующую функцию. Убрал
GtkFileFilter *file_filter = gtk_file_filter_new();
file_filter = gtk_file_filter_new();
gtk_file_filter_add_pattern(file_filter,"*.pkg.tar.*");
// gtk_file_filter_add_pattern(file_filter,"*.pkg.tar.*");
Collaborator

Тут фильтр тоже с какой целью закомментирован?

Тут фильтр тоже с какой целью закомментирован?
Poster
Collaborator

Убрал

Убрал
YanTheKaller added 1 commit 2 years ago
YanTheKaller added 1 commit 2 years ago
user-166 approved these changes 2 years ago
asmeron reviewed 2 years ago
gtk_file_filter_set_name(file_filter, "*db.tar.gz");
gtk_file_chooser_add_filter(GTK_FILE_CHOOSER(chooser),file_filter);
GtkWidget *chooser = gtk_file_chooser_dialog_new(LOAD_REPO_LABEL,NULL,GTK_FILE_CHOOSER_ACTION_SELECT_FOLDER,CANCEL_LABEL,GTK_RESPONSE_CANCEL,OPEN_LABEL,GTK_RESPONSE_ACCEPT,NULL);
gtk_window_set_icon_name(GTK_WINDOW(chooser),"com.ublinux.ubl-settings-repomanager");
Owner

Вынести в переменную внешнее имя иконки.
Старайтесь имена выносить в переменные в шапу в #define , что-бы в случае изменения не бегать по коду и не искать старое имя для замены на новое.

Вынести в переменную внешнее имя иконки. Старайтесь имена выносить в переменные в шапу в #define , что-бы в случае изменения не бегать по коду и не искать старое имя для замены на новое.
asmeron merged commit a540c87463 into master 2 years ago

Reviewers

user-166 approved these changes 2 years ago
The pull request has been merged as a540c87463.
Sign in to join this conversation.
Loading…
There is no content yet.