Skip to content

Commit 8635bdd

Browse files
committed
simplyfy calling of CustomMessageDialog
1 parent 46fa3ce commit 8635bdd

File tree

3 files changed

+5
-26
lines changed

3 files changed

+5
-26
lines changed

src/Gtk/MainWindow.vala

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -450,9 +450,7 @@ class MainWindow : Gtk.Window{
450450
var msg = _("Select another device?");
451451

452452
var type = Gtk.MessageType.ERROR;
453-
var buttons_type = Gtk.ButtonsType.YES_NO;
454-
455-
var dlg = new CustomMessageDialog(title, msg, type, this, buttons_type);
453+
var dlg = new CustomMessageDialog(title, msg, type, this, Gtk.ButtonsType.YES_NO);
456454
var response = dlg.run();
457455
dlg.destroy();
458456

src/Gtk/RestoreWindow.vala

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -230,11 +230,8 @@ class RestoreWindow : Gtk.Window{
230230
var title = _("Cancel restore?");
231231

232232
var msg = _("Cancelling the restore process will leave the target system in an inconsistent state. The system may fail to boot or you may run into various issues. After cancelling, you need to restore another snapshot, to bring the system to a consistent state. Click Yes to confirm.");
233-
234-
var type = Gtk.MessageType.ERROR;
235-
var buttons_type = Gtk.ButtonsType.YES_NO;
236-
237-
var dlg = new CustomMessageDialog(title, msg, type, this, buttons_type);
233+
234+
var dlg = new CustomMessageDialog(title, msg, Gtk.MessageType.ERROR, this, Gtk.ButtonsType.YES_NO);
238235
var response = dlg.run();
239236
dlg.destroy();
240237

src/Utility/GtkHelper.vala

Lines changed: 2 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -43,27 +43,11 @@ namespace TeeJee.GtkHelper{
4343
gtk_do_events ();
4444
}
4545

46-
public void gtk_messagebox(string title, string message, Gtk.Window? parent_win, bool is_error = false){
46+
public static void gtk_messagebox(string title, string message, Gtk.Window? parent_win, bool is_error = false){
4747

4848
/* Shows a simple message box */
4949

50-
var type = Gtk.MessageType.INFO;
51-
if (is_error){
52-
type = Gtk.MessageType.ERROR;
53-
}
54-
else{
55-
type = Gtk.MessageType.INFO;
56-
}
57-
58-
/*var dlg = new Gtk.MessageDialog.with_markup(null, Gtk.DialogFlags.MODAL, type, Gtk.ButtonsType.OK, message);
59-
dlg.title = title;
60-
dlg.set_default_size (200, -1);
61-
if (parent_win != null){
62-
dlg.set_transient_for(parent_win);
63-
dlg.set_modal(true);
64-
}
65-
dlg.run();
66-
dlg.destroy();*/
50+
Gtk.MessageType type = is_error ? Gtk.MessageType.ERROR : Gtk.MessageType.INFO;
6751

6852
var dlg = new CustomMessageDialog(title,message,type,parent_win, Gtk.ButtonsType.OK);
6953
dlg.run();

0 commit comments

Comments
 (0)