[gparted] prepare-for-gtk3: Remove calls to Gtk::Dialog::set_has_separator() (#7)



commit 38d9d92013f0348f6079bf71814d1c8fed1b9ed2
Author: Luca Bacci <luca bacci982 gmail com>
Date:   Fri Aug 3 18:39:05 2018 +0200

    prepare-for-gtk3: Remove calls to Gtk::Dialog::set_has_separator() (#7)
    
    Originally in Gtk2, by default, dialogs showed a horizontal separator
    between the content area and the action area (buttons).  GParted
    explicitly called Gtk::Dialog::set_has_separator(false) for all it's
    dialogs to remove the separator.
    
    In Gtk2/Gtkmm2 2.22, separators were deprecated [1].  In Gtk3/Gtkmm3
    separators in dialogs were removed altogether, including all the related
    APIs [2][3][4].  Therefore remove all calls.
    
    References:
    [1] Commit - "Document separator-related api in GtkDialog as deprecated"
        https://gitlab.gnome.org/GNOME/gtk/commit/6f6650e6cff06ff7c7e70db634ab10510a80c04c
    
    [2] Commit - "Remove separators from dialogs"
        https://gitlab.gnome.org/GNOME/gtk/commit/d433a606111d89727530f71d7b956ba40655bcbf
    
    [3] GTK+ 3.0.0 NEWS file
        https://gitlab.gnome.org/GNOME/gtk/blob/3.0.0/NEWS#L779
        "
        Overview of Changed from GTK+ 2.90.6 to 2.90.7
        ==============================================
    
        * Various deprecated APIs have been removed:
        ...
        - GtkDialog separators, including the GtkDialog::has-separator
          property, including setter/getter, the GTK_DIALOG_NO_SEPARATOR
          flag and the GtkMessageDialog::use-separator style property
        "
    
    [4] Gtkmm 3.0.0 NEWS file
        https://gitlab.gnome.org/GNOME/gtkmm/blob/3.0.0/NEWS#406
        "
        * Dialog:
        - Remove get/set_has_separator() and property.
        - Constructors: Remove use_separator parameters.
        "
    
    Closes #7 - Port to Gtk3

 src/DialogFeatures.cc          | 1 -
 src/DialogManageFlags.cc       | 1 -
 src/DialogPasswordEntry.cc     | 1 -
 src/Dialog_Base_Partition.cc   | 1 -
 src/Dialog_Disklabel.cc        | 1 -
 src/Dialog_FileSystem_Label.cc | 1 -
 src/Dialog_Partition_Info.cc   | 1 -
 src/Dialog_Partition_Name.cc   | 1 -
 src/Dialog_Progress.cc         | 1 -
 9 files changed, 9 deletions(-)
---
diff --git a/src/DialogFeatures.cc b/src/DialogFeatures.cc
index 289fa287..e01cfbf6 100644
--- a/src/DialogFeatures.cc
+++ b/src/DialogFeatures.cc
@@ -27,7 +27,6 @@ namespace GParted
 DialogFeatures::DialogFeatures() 
 {
        set_title( _("File System Support") ) ;
-       set_has_separator( false ) ;
        //Set minimum dialog height so it fits on an 800x600 screen
        set_size_request( -1, 500 ) ;
 
diff --git a/src/DialogManageFlags.cc b/src/DialogManageFlags.cc
index 26c61f1d..edb502cf 100644
--- a/src/DialogManageFlags.cc
+++ b/src/DialogManageFlags.cc
@@ -30,7 +30,6 @@ DialogManageFlags::DialogManageFlags( const Partition & partition, std::map<Glib
        any_change = false ;
 
        set_title( String::ucompose( _("Manage flags on %1"), partition .get_path() ) );
-       set_has_separator( false ) ;
        set_resizable( false ) ;
 
        Glib::ustring str_temp = "<span weight=\"bold\" size=\"larger\">" ;
diff --git a/src/DialogPasswordEntry.cc b/src/DialogPasswordEntry.cc
index a95ff34f..905472f3 100644
--- a/src/DialogPasswordEntry.cc
+++ b/src/DialogPasswordEntry.cc
@@ -29,7 +29,6 @@ namespace GParted
 DialogPasswordEntry::DialogPasswordEntry( const Partition & partition )
 {
        this->set_resizable( false );
-       this->set_has_separator( false );
        this->set_size_request( 400, -1 );
 
        /* TO TRANSLATORS: dialog title, looks like   LUKS Passphrase /dev/sda1 */
diff --git a/src/Dialog_Base_Partition.cc b/src/Dialog_Base_Partition.cc
index 65a70699..fbbc5839 100644
--- a/src/Dialog_Base_Partition.cc
+++ b/src/Dialog_Base_Partition.cc
@@ -23,7 +23,6 @@ namespace GParted
        
 Dialog_Base_Partition::Dialog_Base_Partition()
 {
-       this ->set_has_separator( false ) ;
        frame_resizer_base = NULL;
        GRIP = false ;
        this ->fixed_start = false ;
diff --git a/src/Dialog_Disklabel.cc b/src/Dialog_Disklabel.cc
index 315f7147..40874220 100644
--- a/src/Dialog_Disklabel.cc
+++ b/src/Dialog_Disklabel.cc
@@ -28,7 +28,6 @@ Dialog_Disklabel::Dialog_Disklabel( const Device & device )
 
        /*TO TRANSLATORS: dialogtitle, looks like Create partition table on /dev/hda */
        this ->set_title( String::ucompose( _("Create partition table on %1"), device_path ) );
-       this ->set_has_separator( false ) ;
        this ->set_resizable( false );
 
        {
diff --git a/src/Dialog_FileSystem_Label.cc b/src/Dialog_FileSystem_Label.cc
index f84c25a5..e665eb40 100644
--- a/src/Dialog_FileSystem_Label.cc
+++ b/src/Dialog_FileSystem_Label.cc
@@ -28,7 +28,6 @@ namespace GParted
 Dialog_FileSystem_Label::Dialog_FileSystem_Label( const Partition & partition )
 {
        this ->set_resizable( false ) ;
-       this ->set_has_separator( false ) ;
        this->set_size_request( 400, -1 );
 
        /* TO TRANSLATORS: dialog title, looks like   Set file system label on /dev/hda3 */
diff --git a/src/Dialog_Partition_Info.cc b/src/Dialog_Partition_Info.cc
index b6cc78d1..d33f3f0e 100644
--- a/src/Dialog_Partition_Info.cc
+++ b/src/Dialog_Partition_Info.cc
@@ -31,7 +31,6 @@ namespace GParted
 
 Dialog_Partition_Info::Dialog_Partition_Info( const Partition & partition ) : partition( partition )
 {
-       this ->set_has_separator( false ) ;
        // Set minimum dialog height so it fits on an 800x600 screen without too much
        // whitespace (~500 px max for GNOME desktop).  Allow extra space if have any
        // messages or for LVM2 PV or LUKS encryption.
diff --git a/src/Dialog_Partition_Name.cc b/src/Dialog_Partition_Name.cc
index dcf79e99..0815794c 100644
--- a/src/Dialog_Partition_Name.cc
+++ b/src/Dialog_Partition_Name.cc
@@ -28,7 +28,6 @@ namespace GParted
 Dialog_Partition_Name::Dialog_Partition_Name( const Partition & partition, int max_length )
 {
        this->set_resizable( false );
-       this->set_has_separator( false );
        this->set_size_request( 400, -1 );
 
        /* TO TRANSLATORS: dialog title, looks like   Set partition name on /dev/hda3 */
diff --git a/src/Dialog_Progress.cc b/src/Dialog_Progress.cc
index abee2000..7db4b0d7 100644
--- a/src/Dialog_Progress.cc
+++ b/src/Dialog_Progress.cc
@@ -29,7 +29,6 @@ namespace GParted
 
 Dialog_Progress::Dialog_Progress( const std::vector<Operation *> & operations )
 {
-       this ->set_has_separator( false ) ;
        this ->set_title( _("Applying pending operations") ) ;
        this ->operations = operations ;
        succes = true ;


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]