From d97eb4f9a23dd8dd6019b0d3e2121294d8bb7dea Mon Sep 17 00:00:00 2001
From: sr55 <sr55.hb@outlook.com>
Date: Fri, 26 Jun 2020 21:30:37 +0100
Subject: WinGui:  Experimental new Dimensions Tab Design. (Part 1 of Several).
   Adding the Resolution Limit on display and removing upscale limitations.
 (These will come back in some less restrictive than current form later)  
 #2437

---
 .../HandBrakeWPF/Services/Presets/Model/Preset.cs  | 123 +--------------------
 1 file changed, 5 insertions(+), 118 deletions(-)

(limited to 'win/CS/HandBrakeWPF/Services/Presets/Model/Preset.cs')

diff --git a/win/CS/HandBrakeWPF/Services/Presets/Model/Preset.cs b/win/CS/HandBrakeWPF/Services/Presets/Model/Preset.cs
index df21ccf9b..c67298c32 100644
--- a/win/CS/HandBrakeWPF/Services/Presets/Model/Preset.cs
+++ b/win/CS/HandBrakeWPF/Services/Presets/Model/Preset.cs
@@ -17,64 +17,32 @@ namespace HandBrakeWPF.Services.Presets.Model
     using HandBrakeWPF.Services.Presets.Interfaces;
 
     using EncodeTask = HandBrakeWPF.Services.Encode.Model.EncodeTask;
-    using PresetPictureSettingsMode = HandBrakeWPF.Model.Picture.PresetPictureSettingsMode;
 
-    /// <summary>
-    /// A Preset for encoding with.
-    /// </summary>
-    public class Preset : PropertyChangedBase, IPresetObject // Delibery not 
+    public class Preset : PropertyChangedBase, IPresetObject
     {
-        #region Constants and Fields
-
-        /// <summary>
-        /// The is default.
-        /// </summary>
         private bool isDefault;
 
         private bool isSelected;
 
-        #endregion
-
-        /// <summary>
-        /// Initializes a new instance of the <see cref="Preset"/> class. 
-        /// </summary>
         public Preset()
         {
         }
 
-        /// <summary>
-        /// Initializes a new instance of the <see cref="Preset"/> class. 
-        /// </summary>
-        /// <param name="preset">
-        /// The preset.
-        /// </param>
         public Preset(Preset preset)
         {
             this.Category = preset.Category;
             this.Description = preset.Description;
             this.IsBuildIn = preset.IsBuildIn;
             this.Name = preset.Name;
-            this.PictureSettingsMode = preset.PictureSettingsMode;
             this.Task = new EncodeTask(preset.Task);
             this.AudioTrackBehaviours = new AudioBehaviours(preset.AudioTrackBehaviours);
             this.SubtitleTrackBehaviours = new SubtitleBehaviours(preset.SubtitleTrackBehaviours);
         }
 
-        #region Properties
-
-        /// <summary>
-        /// Gets or sets the category which the preset resides under
-        /// </summary>
         public string Category { get; set; }
 
-        /// <summary>
-        /// Gets or sets the Description for the preset
-        /// </summary>
         public string Description { get; set; }
 
-        /// <summary>
-        /// Reflects the visual state of this preset.
-        /// </summary>
         public bool IsExpanded { get; set; }
 
         public string DisplayValue
@@ -88,10 +56,7 @@ namespace HandBrakeWPF.Services.Presets.Model
 
         public bool IsSelected
         {
-            get
-            {
-                return this.isSelected;
-            }
+            get => this.isSelected;
             set
             {
                 this.isSelected = value;
@@ -99,14 +64,8 @@ namespace HandBrakeWPF.Services.Presets.Model
             }
         }
 
-        /// <summary>
-        /// Gets or sets a value indicating whether this is a built in preset
-        /// </summary>
         public bool IsBuildIn { get; set; }
 
-        /// <summary>
-        /// Gets or sets a value indicating whether IsDefault.
-        /// </summary>
         public bool IsDefault
         {
             get
@@ -120,125 +79,53 @@ namespace HandBrakeWPF.Services.Presets.Model
             }
         }
 
-        /// <summary>
-        /// Gets or sets the preset name
-        /// </summary>
         public string Name { get; set; }
 
-        /// <summary>
-        /// Gets or sets PictureSettingsMode.
-        /// Source Maximum, Custom or None
-        /// </summary>
-        public PresetPictureSettingsMode PictureSettingsMode { get; set; }
-
-        /// <summary>
-        /// Gets or sets task.
-        /// </summary>
         public EncodeTask Task { get; set; }
 
-        /// <summary>
-        /// Gets or sets the audio track behaviours.
-        /// </summary>
         public AudioBehaviours AudioTrackBehaviours { get; set; }
 
-        /// <summary>
-        /// Gets or sets the subtitle track behaviours.
-        /// </summary>
         public SubtitleBehaviours SubtitleTrackBehaviours { get; set; }
 
         public bool IsPresetDisabled { get; set; }
 
-        #endregion
-
-        #region Public Methods
-
-        /// <summary>
-        /// Update this preset.
-        /// The given parameters should be copy-constructed.
-        /// </summary>
-        /// <param name="task">
-        /// The task.
-        /// </param>
-        /// <param name="audioBehaviours">
-        /// The audio behaviours.
-        /// </param>
-        /// <param name="subtitleBehaviours">
-        /// The subtitle behaviours.
-        /// </param>
         public void Update(EncodeTask task, AudioBehaviours audioBehaviours, SubtitleBehaviours subtitleBehaviours)
         {
-            // Copy over Max Width / Height for the following picture settings modes.
-            if (this.PictureSettingsMode == PresetPictureSettingsMode.Custom
-                || this.PictureSettingsMode == PresetPictureSettingsMode.SourceMaximum)
-            {
-                task.MaxWidth = this.Task.MaxWidth;
-                task.MaxHeight = this.Task.MaxHeight;
-            }
-
             this.Task = task;
             this.AudioTrackBehaviours = new AudioBehaviours(audioBehaviours);
             this.SubtitleTrackBehaviours = new SubtitleBehaviours(subtitleBehaviours);
         }
 
-        /// <summary>
-        ///  Override the ToString Method
-        /// </summary>
-        /// <returns>
-        /// The Preset Name
-        /// </returns>
         public override string ToString()
         {
             return this.Name;
         }
 
-        #endregion
-
-        /// <summary>
-        /// The equals.
-        /// </summary>
-        /// <param name="obj">
-        /// The obj.
-        /// </param>
-        /// <returns>
-        /// The <see cref="bool"/>.
-        /// </returns>
         public override bool Equals(object obj)
         {
             if (ReferenceEquals(null, obj))
             {
                 return false;
             }
+
             if (ReferenceEquals(this, obj))
             {
                 return true;
             }
+
             if (obj.GetType() != this.GetType())
             {
                 return false;
             }
+
             return Equals((Preset)obj);
         }
 
-        /// <summary>
-        /// The get hash code.
-        /// </summary>
-        /// <returns>
-        /// The <see cref="int"/>.
-        /// </returns>
         public override int GetHashCode()
         {
             return (this.Name != null ? this.Name.GetHashCode() : 0);
         }
 
-        /// <summary>
-        /// The equals.
-        /// </summary>
-        /// <param name="other">
-        /// The other.
-        /// </param>
-        /// <returns>
-        /// The <see cref="bool"/>.
-        /// </returns>
         protected bool Equals(Preset other)
         {
             return string.Equals(this.Name, other.Name);
-- 
cgit v1.2.3