Merge branch 'taiko_hitobject_drawable' into taiko_drumroll_drawable

Conflicts:
	osu.Game.Modes.Taiko/osu.Game.Modes.Taiko.csproj

osu.Desktop/Overlays/VersionManager.cs

@@ -215,6 +215,7 @@ namespace osu.Desktop.Overlays
                         Origin = Anchor.Centre,
                         Icon = FontAwesome.fa_upload,
                         Colour = Color4.White,
+                        TextSize = 20
                     }
                 });
             }

osu.Game.Modes.Taiko/Judgements/TaikoHitResult.cs

@@ -3,7 +3,7 @@
 
 namespace osu.Game.Modes.Taiko.Judgements
 {
-    public enum TaikoScoreResult
+    public enum TaikoHitResult
     {
         Good,
         Great

osu.Game.Modes.Taiko/Judgements/TaikoJudgementInfo.cs

@@ -10,32 +10,32 @@ namespace osu.Game.Modes.Taiko.Judgements
         /// <summary>
         /// The maximum score value.
         /// </summary>
-        public const TaikoScoreResult MAX_SCORE = TaikoScoreResult.Great;
+        public const TaikoHitResult MAX_HIT_RESULT = TaikoHitResult.Great;
 
         /// <summary>
         /// The score value.
         /// </summary>
-        public TaikoScoreResult Score;
+        public TaikoHitResult TaikoResult;
 
         /// <summary>
         /// The score value for the combo portion of the score.
         /// </summary>
-        public int ScoreValue => ScoreToInt(Score);
+        public int ScoreValue => NumericResultForScore(TaikoResult);
         
         /// <summary>
         /// The score value for the accuracy portion of the score.
         /// </summary>
-        public int AccuracyScoreValue => AccuracyScoreToInt(Score);
+        public int AccuracyScoreValue => NumericResultForAccuracy(TaikoResult);
 
         /// <summary>
         /// The maximum score value for the combo portion of the score.
         /// </summary>
-        public int MaxScoreValue => ScoreToInt(MAX_SCORE);
+        public int MaxScoreValue => NumericResultForScore(MAX_HIT_RESULT);
         
         /// <summary>
         /// The maximum score value for the accuracy portion of the score.
         /// </summary>
-        public int MaxAccuracyScoreValue => AccuracyScoreToInt(MAX_SCORE);
+        public int MaxAccuracyScoreValue => NumericResultForAccuracy(MAX_HIT_RESULT);
 
         /// <summary>
         /// Whether this Judgement has a secondary hit in the case of finishers.
@@ -43,38 +43,39 @@ namespace osu.Game.Modes.Taiko.Judgements
         public bool SecondHit;
 
         /// <summary>
-        /// Computes the score value for the combo portion of the score.
+        /// Computes the numeric score value for the combo portion of the score.
-        /// For the accuracy portion of the score (including accuracy percentage), see <see cref="AccuracyScoreToInt(TaikoScoreResult)"/>.
+        /// For the accuracy portion of the score (including accuracy percentage), see <see cref="NumericResultForAccuracy(TaikoHitResult)"/>.
         /// </summary>
         /// <param name="result">The result to compute the score value for.</param>
-        /// <returns>The int score value.</returns>
+        /// <returns>The numeric score value.</returns>
-        protected virtual int ScoreToInt(TaikoScoreResult result)
+        protected virtual int NumericResultForScore(TaikoHitResult result)
         {
             switch (result)
             {
                 default:
                     return 0;
-                case TaikoScoreResult.Good:
+                case TaikoHitResult.Good:
                     return 100;
-                case TaikoScoreResult.Great:
+                case TaikoHitResult.Great:
                     return 300;
             }
         }
 
         /// <summary>
-        /// Computes the score value for the accurac portion of the score.
+        /// Computes the numeric score value for the accuracy portion of the score.
+        /// For the combo portion of the score, see <see cref="NumericResultForScore(TaikoHitResult)"/>.
         /// </summary>
         /// <param name="result">The result to compute the score value for.</param>
-        /// <returns>The int score value.</returns>
+        /// <returns>The numeric score value.</returns>
-        protected virtual int AccuracyScoreToInt(TaikoScoreResult result)
+        protected virtual int NumericResultForAccuracy(TaikoHitResult result)
         {
             switch (result)
             {
                 default:
                     return 0;
-                case TaikoScoreResult.Good:
+                case TaikoHitResult.Good:
                     return 150;
-                case TaikoScoreResult.Great:
+                case TaikoHitResult.Great:
                     return 300;
             }
         }

osu.Game.Modes.Taiko/Objects/Drawable/DrawableTaikoHitObject.cs

@@ -41,7 +41,7 @@ namespace osu.Game.Modes.Taiko.Objects.Drawable
         /// a time value and the HitObject's StartTime.
         /// </summary>
         /// <param name="time"></param>
-        protected void UpdateScrollPosition(double time)
+        protected virtual void UpdateScrollPosition(double time)
         {
             MoveToX((float)((HitObject.StartTime - time) / HitObject.PreEmpt));
         }

osu.Game.Modes.Taiko/osu.Game.Modes.Taiko.csproj

@@ -51,7 +51,7 @@
     <Compile Include="Beatmaps\TaikoBeatmapProcessor.cs" />
     <Compile Include="Judgements\TaikoDrumRollTickJudgementInfo.cs" />
     <Compile Include="Judgements\TaikoJudgementInfo.cs" />
-    <Compile Include="Judgements\TaikoScoreResult.cs" />
+    <Compile Include="Judgements\TaikoHitResult.cs" />
     <Compile Include="Objects\Drawable\DrawableDrumRoll.cs" />
     <Compile Include="Objects\Drawable\DrawableDrumRollTick.cs" />
     <Compile Include="Objects\Drawable\DrawableTaikoHitObject.cs" />

osu.Game/Beatmaps/Formats/OsuLegacyDecoder.cs

@@ -207,7 +207,8 @@ namespace osu.Game.Beatmaps.Formats
                     VelocityAdjustment = beatLength < 0 ? -beatLength / 100.0 : 1,
                     TimingChange = split.Length <= 6 || split[6][0] == '1',
                     KiaiMode = (effectFlags & 1) > 0,
-                    OmitFirstBarLine = (effectFlags & 8) > 0
+                    OmitFirstBarLine = (effectFlags & 8) > 0,
+                    TimeSignature = (TimeSignatures)int.Parse(split[2])
                 };
             }
 

osu.Game/Beatmaps/Timing/ControlPoint.cs

@@ -11,18 +11,12 @@ namespace osu.Game.Beatmaps.Timing
             TimingChange = true,
         };
 
+        public TimeSignatures TimeSignature;
         public double Time;
         public double BeatLength;
         public double VelocityAdjustment;
         public bool TimingChange;
         public bool KiaiMode;
         public bool OmitFirstBarLine;
-
-    }
-
-    internal enum TimeSignatures
-    {
-        SimpleQuadruple = 4,
-        SimpleTriple = 3
     }
 }

osu.Game/Beatmaps/Timing/TimeSignatures.cs

@@ -0,0 +1,11 @@
+// Copyright (c) 2007-2017 ppy Pty Ltd <contact@ppy.sh>.
+// Licensed under the MIT Licence - https://raw.githubusercontent.com/ppy/osu/master/LICENCE
+
+namespace osu.Game.Beatmaps.Timing
+{
+    public enum TimeSignatures
+    {
+        SimpleQuadruple = 4,
+        SimpleTriple = 3
+    }
+}

osu.Game/Graphics/TextAwesome.cs

@@ -1,11 +1,11 @@
 // Copyright (c) 2007-2017 ppy Pty Ltd <contact@ppy.sh>.
 // Licensed under the MIT Licence - https://raw.githubusercontent.com/ppy/osu/master/LICENCE
 
-using osu.Framework.Graphics.Sprites;
+using osu.Game.Graphics.Sprites;
 
 namespace osu.Game.Graphics
 {
-    public class TextAwesome : SpriteText
+    public class TextAwesome : OsuSpriteText
     {
         //public override FontFace FontFace => (int)Icon < 0xf000 ? FontFace.OsuFont : FontFace.FontAwesome;
 

osu.Game/Graphics/UserInterface/OsuDropDownHeader.cs

@@ -54,6 +54,7 @@ namespace osu.Game.Graphics.UserInterface
                     Anchor = Anchor.CentreRight,
                     Origin = Anchor.CentreRight,
                     Margin = new MarginPadding { Right = 4 },
+                    TextSize = 20
                 }
             };
         }

osu.Game/Modes/UI/ModIcon.cs

@@ -66,12 +66,14 @@ namespace osu.Game.Modes.UI
                     Anchor = Anchor.Centre,
                     Icon = FontAwesome.fa_osu_mod_bg,
                     Shadow = true,
+                    TextSize = 20
                 },
                 modIcon = new TextAwesome
                 {
                     Origin = Anchor.Centre,
                     Anchor = Anchor.Centre,
                     Colour = OsuColour.Gray(84),
+                    TextSize = 20
                 },
             };
 

osu.Game/Modes/UI/Playfield.cs

@@ -7,6 +7,7 @@ using osu.Game.Modes.Objects;
 using osu.Game.Modes.Objects.Drawables;
 using OpenTK;
 using osu.Game.Modes.Judgements;
+using osu.Framework.Allocation;
 
 namespace osu.Game.Modes.UI
 {
@@ -45,10 +46,16 @@ namespace osu.Game.Modes.UI
                 }
             });
 
-            Add(HitObjects = new HitObjectContainer<DrawableHitObject<TObject, TJudgement>>
+            HitObjects = new HitObjectContainer<DrawableHitObject<TObject, TJudgement>>
             {
                 RelativeSizeAxes = Axes.Both,
-            });
+            };
+        }
+
+        [BackgroundDependencyLoader]
+        private void load()
+        {
+            Add(HitObjects);
         }
 
         /// <summary>

osu.Game/OsuGame.cs

@@ -279,6 +279,7 @@ namespace osu.Game
             //central game mode change logic.
             if (!currentScreen.ShowOverlays)
             {
+                options.State = Visibility.Hidden;
                 Toolbar.State = Visibility.Hidden;
                 musicController.State = Visibility.Hidden;
                 chat.State = Visibility.Hidden;

osu.Game/Overlays/Notifications/Notification.cs

@@ -169,6 +169,7 @@ namespace osu.Game.Overlays.Notifications
                         Anchor = Anchor.Centre,
                         Origin = Anchor.Centre,
                         Icon = FontAwesome.fa_times_circle,
+                        TextSize = 20
                     }
                 };
             }

osu.Game/Overlays/Notifications/SimpleNotification.cs

@@ -53,6 +53,7 @@ namespace osu.Game.Overlays.Notifications
                     Anchor = Anchor.Centre,
                     Origin = Anchor.Centre,
                     Icon = icon,
+                    TextSize = 20
                 }
             });
 

osu.Game/Overlays/Options/OptionsFooter.cs

@@ -32,6 +32,7 @@ namespace osu.Game.Overlays.Options
                 {
                     Icon = Ruleset.GetRuleset(m).Icon,
                     Colour = Color4.Gray,
+                    TextSize = 20
                 });
 
             Children = new Drawable[]

osu.Game/Overlays/Options/Sections/Graphics/LayoutOptions.cs

@@ -5,6 +5,7 @@ using osu.Framework.Allocation;
 using osu.Framework.Configuration;
 using osu.Framework.Graphics;
 using osu.Game.Graphics.UserInterface;
+using System;
 
 namespace osu.Game.Overlays.Options.Sections.Graphics
 {
@@ -12,9 +13,16 @@ namespace osu.Game.Overlays.Options.Sections.Graphics
     {
         protected override string Header => "Layout";
 
+        private OptionSlider<int> letterboxPositionX;
+        private OptionSlider<int> letterboxPositionY;
+
+        private Bindable<bool> letterboxing;
+
         [BackgroundDependencyLoader]
         private void load(FrameworkConfigManager config)
         {
+            letterboxing = config.GetBindable<bool>(FrameworkConfig.Letterboxing);
+
             Children = new Drawable[]
             {
                 new OptionLabel { Text = "Resolution: TODO dropdown" },
@@ -26,19 +34,36 @@ namespace osu.Game.Overlays.Options.Sections.Graphics
                 new OsuCheckbox
                 {
                     LabelText = "Letterboxing",
-                    Bindable = config.GetBindable<bool>(FrameworkConfig.Letterboxing),
+                    Bindable = letterboxing,
                 },
-                new OptionSlider<int>
+                letterboxPositionX = new OptionSlider<int>
                 {
                     LabelText = "Horizontal position",
                     Bindable = (BindableInt)config.GetBindable<int>(FrameworkConfig.LetterboxPositionX)
                 },
-                new OptionSlider<int>
+                letterboxPositionY = new OptionSlider<int>
                 {
                     LabelText = "Vertical position",
                     Bindable = (BindableInt)config.GetBindable<int>(FrameworkConfig.LetterboxPositionY)
                 },
             };
+
+            letterboxing.ValueChanged += visibilityChanged;
+            letterboxing.TriggerChange();
+        }
+
+        private void visibilityChanged(object sender, EventArgs e)
+        {
+            if (letterboxing)
+            {
+                letterboxPositionX.Show();
+                letterboxPositionY.Show();
+            }
+            else
+            {
+                letterboxPositionX.Hide();
+                letterboxPositionY.Hide();
+            }
         }
     }
 }

osu.Game/Overlays/Options/SidebarButton.cs

@@ -87,6 +87,7 @@ namespace osu.Game.Overlays.Options
                         {
                             Anchor = Anchor.Centre,
                             Origin = Anchor.Centre,
+                            TextSize = 20
                         },
                     }
                 },

osu.Game/Overlays/Toolbar/ToolbarButton.cs

@@ -98,6 +98,7 @@ namespace osu.Game.Overlays.Toolbar
                         {
                             Anchor = Anchor.Centre,
                             Origin = Anchor.Centre,
+                            TextSize = 20
                         },
                         DrawableText = new OsuSpriteText
                         {

osu.Game/Screens/Select/BeatmapInfoWedge.cs

@@ -237,14 +237,16 @@ namespace osu.Game.Screens.Select
                         Icon = FontAwesome.fa_square,
                         Origin = Anchor.Centre,
                         Colour = new Color4(68, 17, 136, 255),
-                        Rotation = 45
+                        Rotation = 45,
+                        TextSize = 20
                     },
                     new TextAwesome
                     {
                         Icon = statistic.Icon,
                         Origin = Anchor.Centre,
                         Colour = new Color4(255, 221, 85, 255),
-                        Scale = new Vector2(0.8f)
+                        Scale = new Vector2(0.8f),
+                        TextSize = 20
                     },
                     new OsuSpriteText
                     {

osu.Game/Screens/Select/SearchTextBox.cs

@@ -26,6 +26,7 @@ namespace osu.Game.Screens.Select
                     Origin = Anchor.CentreRight,
                     Anchor = Anchor.CentreRight,
                     Margin = new MarginPadding { Right = 10 },
+                    TextSize = 20
                 }
             });
 

osu.Game/Screens/Tournament/Drawings.cs

@@ -160,7 +160,7 @@ namespace osu.Game.Screens.Tournament
 
                                     Text = "Control Panel",
                                     TextSize = 22f,
-                                    Font = "Exo2.0-Boldd"
+                                    Font = "Exo2.0-Bold"
                                 },
                                 new FillFlowContainer
                                 {

osu.Game/osu.Game.csproj

@@ -76,6 +76,7 @@
     <Compile Include="Beatmaps\IBeatmapCoverter.cs" />
     <Compile Include="Beatmaps\IBeatmapProcessor.cs" />
     <Compile Include="Beatmaps\Legacy\LegacyBeatmap.cs" />
+    <Compile Include="Beatmaps\Timing\TimeSignatures.cs" />
     <Compile Include="Beatmaps\Timing\TimingInfo.cs" />
     <Compile Include="Database\ScoreDatabase.cs" />
     <Compile Include="Graphics\Backgrounds\Triangles.cs" />