Remove reader speed dial control

KSemenenko · KSemenenko · commit b1be92d66c22 · 2026-04-27T15:06:16.000+02:00
diff --git a/src/PrompterOne.Shared/Contracts/UiDomIds.cs b/src/PrompterOne.Shared/Contracts/UiDomIds.cs
@@ -92,8 +92,6 @@ public static class Teleprompter
         public const string FocalGuide = "rd-guide-h";
         public const string HeaderSegment = "rd-header-segment";
         public const string ProgressFill = "rd-progress-fill";
-        public const string SpeedDial = "rd-speed-dial";
-        public const string SpeedDialValue = "rd-speed-dial-val";
         public const string SpeedValue = "rd-speed-val";
         public const string Stage = "rd-stage";
         public const string Time = "rd-time";
diff --git a/src/PrompterOne.Shared/Contracts/UiTestIds.Teleprompter.cs b/src/PrompterOne.Shared/Contracts/UiTestIds.Teleprompter.cs
@@ -61,9 +61,6 @@ public static class Teleprompter
         public const string SpeedCueDisplayMultiplier = "teleprompter-speed-cue-display-multiplier";
         public const string SpeedCueDisplayTooltipKey = "speed-cue-display";
         public const string SpeedCueDisplayWpm = "teleprompter-speed-cue-display-wpm";
-        public const string SpeedDial = "teleprompter-speed-dial";
-        public const string SpeedDialTooltipKey = "speed-dial";
-        public const string SpeedDialValue = "teleprompter-speed-dial-value";
         public const string SpeedUp = "teleprompter-speed-up";
         public const string SpeedValue = "teleprompter-speed-value";
         public const string Sliders = "teleprompter-sliders";
diff --git a/src/PrompterOne.Shared/Localization/SharedResource.de.resx b/src/PrompterOne.Shared/Localization/SharedResource.de.resx
@@ -561,9 +561,6 @@
   <data name="TooltipReaderSpeedCueDisplayMultiplier" xml:space="preserve">
     <value>Geschwindigkeitsmarken als Multiplikatoren anzeigen</value>
   </data>
-  <data name="TooltipReaderSpeedDial" xml:space="preserve">
-    <value>Live-Geschwindigkeitsregler für den Reader anpassen</value>
-  </data>
   <data name="TooltipSmallerText" xml:space="preserve">
     <value>Kleinerer Text</value>
   </data>
diff --git a/src/PrompterOne.Shared/Localization/SharedResource.es.resx b/src/PrompterOne.Shared/Localization/SharedResource.es.resx
@@ -561,9 +561,6 @@
   <data name="TooltipReaderSpeedCueDisplayMultiplier" xml:space="preserve">
     <value>Mostrar las etiquetas de velocidad como multiplicadores</value>
   </data>
-  <data name="TooltipReaderSpeedDial" xml:space="preserve">
-    <value>Ajustar el dial de velocidad en vivo del lector</value>
-  </data>
   <data name="TooltipSmallerText" xml:space="preserve">
     <value>Texto más pequeño</value>
   </data>
diff --git a/src/PrompterOne.Shared/Localization/SharedResource.fr.resx b/src/PrompterOne.Shared/Localization/SharedResource.fr.resx
@@ -561,9 +561,6 @@
   <data name="TooltipReaderSpeedCueDisplayMultiplier" xml:space="preserve">
     <value>Afficher les repères de vitesse en multiplicateurs</value>
   </data>
-  <data name="TooltipReaderSpeedDial" xml:space="preserve">
-    <value>Régler le cadran de vitesse en direct du lecteur</value>
-  </data>
   <data name="TooltipSmallerText" xml:space="preserve">
     <value>Texte plus petit</value>
   </data>
diff --git a/src/PrompterOne.Shared/Localization/SharedResource.it.resx b/src/PrompterOne.Shared/Localization/SharedResource.it.resx
@@ -561,9 +561,6 @@
   <data name="TooltipReaderSpeedCueDisplayMultiplier" xml:space="preserve">
     <value>Mostra le etichette di velocità come moltiplicatori</value>
   </data>
-  <data name="TooltipReaderSpeedDial" xml:space="preserve">
-    <value>Regola il selettore della velocità live del lettore</value>
-  </data>
   <data name="TooltipSmallerText" xml:space="preserve">
     <value>Testo più piccolo</value>
   </data>
diff --git a/src/PrompterOne.Shared/Localization/SharedResource.pt.resx b/src/PrompterOne.Shared/Localization/SharedResource.pt.resx
@@ -561,9 +561,6 @@
   <data name="TooltipReaderSpeedCueDisplayMultiplier" xml:space="preserve">
     <value>Mostrar os marcadores de velocidade como multiplicadores</value>
   </data>
-  <data name="TooltipReaderSpeedDial" xml:space="preserve">
-    <value>Ajustar o seletor de velocidade ao vivo do leitor</value>
-  </data>
   <data name="TooltipSmallerText" xml:space="preserve">
     <value>Texto menor</value>
   </data>
diff --git a/src/PrompterOne.Shared/Localization/SharedResource.resx b/src/PrompterOne.Shared/Localization/SharedResource.resx
@@ -582,9 +582,6 @@
   <data name="TooltipReaderSpeedCueDisplayMultiplier" xml:space="preserve">
     <value>Show speed cues as multipliers</value>
   </data>
-  <data name="TooltipReaderSpeedDial" xml:space="preserve">
-    <value>Adjust the live reader speed dial</value>
-  </data>
   <data name="TooltipSmallerText" xml:space="preserve">
     <value>Smaller text</value>
   </data>
diff --git a/src/PrompterOne.Shared/Localization/SharedResource.uk.resx b/src/PrompterOne.Shared/Localization/SharedResource.uk.resx
@@ -603,9 +603,6 @@
   <data name="TooltipReaderSpeedCueDisplayMultiplier" xml:space="preserve">
     <value>Показувати позначки швидкості як множники</value>
   </data>
-  <data name="TooltipReaderSpeedDial" xml:space="preserve">
-    <value>Налаштувати живий регулятор швидкості читання</value>
-  </data>
   <data name="TooltipSmallerText" xml:space="preserve">
     <value>Менший текст</value>
   </data>
diff --git a/src/PrompterOne.Shared/Localization/UiTextKey.cs b/src/PrompterOne.Shared/Localization/UiTextKey.cs
@@ -300,7 +300,6 @@ public enum UiTextKey
     TooltipReaderSpeedCueDisplay,
     TooltipReaderSpeedCueDisplayWpm,
     TooltipReaderSpeedCueDisplayMultiplier,
-    TooltipReaderSpeedDial,
     TooltipSmallerText,
     TooltipLargerText,
     TooltipPreviousBlock,
diff --git a/src/PrompterOne.Shared/Teleprompter/Pages/TeleprompterPage.ReaderPlayback.cs b/src/PrompterOne.Shared/Teleprompter/Pages/TeleprompterPage.ReaderPlayback.cs
@@ -18,17 +18,6 @@ public partial class TeleprompterPage
 
     private Task IncreaseReaderPlaybackSpeedAsync() => ChangeReaderPlaybackSpeedAsync(ReaderPlaybackSpeedStepWpm);
 
-    private Task HandleReaderSpeedDialInputAsync(ChangeEventArgs args)
-    {
-        var nextDialValue = ParseReaderControlValue(
-            args.Value,
-            ReaderSpeedDialMinimum,
-            ReaderSpeedDialMaximum,
-            BuildReaderSpeedDialValue());
-
-        return SetReaderPlaybackSpeedAsync(MapReaderSpeedDialToWpm(nextDialValue));
-    }
-
     private Task StepReaderBackwardAsync() => StepReaderWordAsync(ReaderBackwardStep);
 
     private Task StepReaderForwardAsync() => StepReaderWordAsync(ReaderForwardStep);
@@ -72,20 +61,6 @@ private async Task SetReaderPlaybackSpeedAsync(int speedWpm)
         }
     }
 
-    private static int MapReaderSpeedDialToWpm(int dialValue)
-    {
-        var normalizedDial = Math.Clamp(dialValue, ReaderSpeedDialMinimum, ReaderSpeedDialMaximum);
-        var dialRange = ReaderSpeedDialMaximum - ReaderSpeedDialMinimum;
-        if (dialRange <= 0)
-        {
-            return ReaderMinimumPlaybackSpeedWpm;
-        }
-
-        var speedRange = ReaderMaximumPlaybackSpeedWpm - ReaderMinimumPlaybackSpeedWpm;
-        var dialOffset = normalizedDial - ReaderSpeedDialMinimum;
-        return ReaderMinimumPlaybackSpeedWpm + (int)Math.Round(speedRange * dialOffset / (double)dialRange, MidpointRounding.AwayFromZero);
-    }
-
     private async Task HandleReaderFontSizeInputAsync(ChangeEventArgs args)
     {
         var nextFontSize = ParseReaderControlValue(
diff --git a/src/PrompterOne.Shared/Teleprompter/Pages/TeleprompterPage.ReaderRail.cs b/src/PrompterOne.Shared/Teleprompter/Pages/TeleprompterPage.ReaderRail.cs
@@ -22,8 +22,6 @@ public partial class TeleprompterPage
 
     private string ReaderSpeedCueDisplayWpmTitle => Text(UiTextKey.TooltipReaderSpeedCueDisplayWpm);
 
-    private string ReaderSpeedDialTitle => Text(UiTextKey.TooltipReaderSpeedDial);
-
     private string ReaderFullscreenTitle => Text(UiTextKey.TooltipToggleBrowserFullscreen);
 
     private string ReaderMirrorHorizontalTitle => Text(UiTextKey.TooltipMirrorReaderHorizontally);
diff --git a/src/PrompterOne.Shared/Teleprompter/Pages/TeleprompterPage.ReaderRendering.cs b/src/PrompterOne.Shared/Teleprompter/Pages/TeleprompterPage.ReaderRendering.cs
@@ -528,23 +528,6 @@ private string BuildReaderSpeedLabel() =>
             ? BuildReaderSpeedMultiplierLabel(_readerBaseTpsWpm, _readerPlaybackSpeedWpm)
             : $"{_readerPlaybackSpeedWpm} {ReaderWordsPerMinuteSuffix}";
 
-    private int BuildReaderSpeedDialValue()
-    {
-        var speedRange = ReaderMaximumPlaybackSpeedWpm - ReaderMinimumPlaybackSpeedWpm;
-        if (speedRange <= 0)
-        {
-            return ReaderSpeedDialMinimum;
-        }
-
-        var dialRange = ReaderSpeedDialMaximum - ReaderSpeedDialMinimum;
-        var normalizedSpeed = Math.Clamp(_readerPlaybackSpeedWpm, ReaderMinimumPlaybackSpeedWpm, ReaderMaximumPlaybackSpeedWpm);
-        var speedOffset = normalizedSpeed - ReaderMinimumPlaybackSpeedWpm;
-        return ReaderSpeedDialMinimum + (int)Math.Round(speedOffset * dialRange / (double)speedRange, MidpointRounding.AwayFromZero);
-    }
-
-    private string BuildReaderSpeedDialValueLabel() =>
-        $"{BuildReaderSpeedDialValue()}/{ReaderSpeedDialMaximum}";
-
     private string? BuildReaderWordSpeedCueLabel(ReaderWordViewModel word, int targetWpm)
     {
         if (!ShouldShowReaderSpeedCueLabel(word) || word.EffectiveWpm is not int effectiveWpm)
diff --git a/src/PrompterOne.Shared/Teleprompter/Pages/TeleprompterPage.razor b/src/PrompterOne.Shared/Teleprompter/Pages/TeleprompterPage.razor
@@ -228,10 +228,8 @@
                                          RecordingStatusLabel="@ReaderRecordingStatusLabel" RecordingStopLabel="@ReaderRecordingStopLabel"
                                          RecordingToggleTooltip="@ReaderRecordingToggleTooltip" SelectedRecordingCameraId="@_readerRecordingCameraId"
                                          SelectedRecordingMicrophoneId="@_readerRecordingMicrophoneId"
-                                         SpeedDialMax="@ReaderSpeedDialMaximum" SpeedDialMin="@ReaderSpeedDialMinimum"
-                                         SpeedDialTitle="@ReaderSpeedDialTitle" SpeedDialValue="@BuildReaderSpeedDialValue()"
-                                         SpeedDialValueLabel="@BuildReaderSpeedDialValueLabel()" SpeedUpTooltip="@ReaderSpeedUpTooltip"
-                                         SpeedValueLabel="@BuildReaderSpeedLabel()" OnSpeedDialInput="HandleReaderSpeedDialInputAsync"
+                                         SpeedUpTooltip="@ReaderSpeedUpTooltip"
+                                         SpeedValueLabel="@BuildReaderSpeedLabel()"
                                          TimeDataAttributes="@BuildReaderTimeDataAttributes()" />
         </div>
     </div>
diff --git a/src/PrompterOne.Shared/Teleprompter/Pages/TeleprompterPage.razor.cs b/src/PrompterOne.Shared/Teleprompter/Pages/TeleprompterPage.razor.cs
@@ -36,8 +36,6 @@ public partial class TeleprompterPage : IAsyncDisposable
     private const int ReaderMinimumPlaybackSpeedWpm = 60;
     private const int ReaderMinTextWidthPercent = 35;
     private const int ReaderPlaybackSpeedStepWpm = 10;
-    private const int ReaderSpeedDialMaximum = 10;
-    private const int ReaderSpeedDialMinimum = 1;
     private const double ReaderLegacyMaximumScrollSpeedMultiplier = 5d;
     private const int DefaultReaderTextWidthPercent = ReaderMaxTextWidthPercent;
 
diff --git a/src/PrompterOne.Shared/Teleprompter/Pages/TeleprompterReaderTransport.razor b/src/PrompterOne.Shared/Teleprompter/Pages/TeleprompterReaderTransport.razor
@@ -55,30 +55,6 @@
             </button>
         </TooltipAnchor>
         <span class="rd-ctrl-label" id="@UiDomIds.Teleprompter.SpeedValue" data-test="@UiTestIds.Teleprompter.SpeedValue">@SpeedValueLabel</span>
-        <TooltipAnchor OwnerTestId="@UiTestIds.Teleprompter.SpeedDial" Text="@SpeedDialTitle">
-            <label class="rd-speed-dial-inline" aria-label="@SpeedDialTitle">
-                <span class="rd-speed-dial-icon" aria-hidden="true">
-                    <svg width="14" height="14" viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="2">
-                        <path d="M12 15l3-3" />
-                        <path d="M4 14a8 8 0 1 1 16 0" />
-                        <path d="M5 19h14" />
-                    </svg>
-                </span>
-                <input type="range"
-                       class="rd-speed-dial"
-                       id="@UiDomIds.Teleprompter.SpeedDial"
-                       min="@SpeedDialMin"
-                       max="@SpeedDialMax"
-                       value="@SpeedDialValue"
-                       @oninput="OnSpeedDialInput"
-                       @onkeydown:stopPropagation="true"
-                       aria-label="@SpeedDialTitle"
-                       data-test="@UiTestIds.Teleprompter.SpeedDial">
-                <span class="rd-speed-dial-value"
-                      id="@UiDomIds.Teleprompter.SpeedDialValue"
-                      data-test="@UiTestIds.Teleprompter.SpeedDialValue">@SpeedDialValueLabel</span>
-            </label>
-        </TooltipAnchor>
         <TooltipAnchor OwnerTestId="@UiTestIds.Teleprompter.SpeedUp" Text="@SpeedUpTooltip">
             <button class="rd-ctrl-btn"
                     @onclick="OnSpeedUp"
@@ -247,7 +223,6 @@
     [Parameter] public EventCallback<ChangeEventArgs> OnRecordingCameraChange { get; set; }
     [Parameter] public EventCallback<ChangeEventArgs> OnRecordingMicrophoneChange { get; set; }
     [Parameter] public EventCallback<ChangeEventArgs> OnRecordingModeChange { get; set; }
-    [Parameter, EditorRequired] public EventCallback<ChangeEventArgs> OnSpeedDialInput { get; set; }
     [Parameter] public EventCallback OnSpeedDown { get; set; }
     [Parameter] public EventCallback OnSpeedUp { get; set; }
     [Parameter, EditorRequired] public string NextBlockTooltip { get; set; } = string.Empty;
@@ -273,11 +248,6 @@
     [Parameter, EditorRequired] public string RecordingToggleTooltip { get; set; } = string.Empty;
     [Parameter, EditorRequired] public string SelectedRecordingCameraId { get; set; } = string.Empty;
     [Parameter, EditorRequired] public string SelectedRecordingMicrophoneId { get; set; } = string.Empty;
-    [Parameter] public int SpeedDialMax { get; set; }
-    [Parameter] public int SpeedDialMin { get; set; }
-    [Parameter] public int SpeedDialValue { get; set; }
-    [Parameter, EditorRequired] public string SpeedDialTitle { get; set; } = string.Empty;
-    [Parameter, EditorRequired] public string SpeedDialValueLabel { get; set; } = string.Empty;
     [Parameter, EditorRequired] public string SpeedDownTooltip { get; set; } = string.Empty;
     [Parameter, EditorRequired] public string SpeedUpTooltip { get; set; } = string.Empty;
     [Parameter, EditorRequired] public string SpeedValueLabel { get; set; } = string.Empty;
diff --git a/src/PrompterOne.Shared/wwwroot/design/modules/reader/20-controls.css b/src/PrompterOne.Shared/wwwroot/design/modules/reader/20-controls.css
@@ -568,82 +568,6 @@
     color: rgba(208,188,152,.46);
 }
 
-.rd-speed-dial-inline {
-    display: inline-grid;
-    grid-template-columns: 14px minmax(72px, 104px) 34px;
-    align-items: center;
-    gap: 6px;
-    height: 34px;
-    padding: 0 8px;
-    border: 1px solid rgba(232,213,176,.11);
-    border-radius: 999px;
-    background: rgba(255,255,255,.03);
-    color: rgba(232,213,176,.66);
-}
-
-.rd-speed-dial-inline:hover,
-.rd-speed-dial-inline:focus-within {
-    border-color: rgba(232,213,176,.22);
-    color: rgba(244,232,204,.86);
-    background: rgba(255,255,255,.055);
-}
-
-.rd-speed-dial-icon {
-    display: grid;
-    place-items: center;
-}
-
-.rd-speed-dial {
-    width: 100%;
-    min-width: 0;
-    height: 18px;
-    appearance: none;
-    -webkit-appearance: none;
-    background: transparent;
-    cursor: pointer;
-}
-
-.rd-speed-dial::-webkit-slider-runnable-track {
-    height: 4px;
-    border-radius: 999px;
-    background: rgba(232,213,176,.18);
-}
-
-.rd-speed-dial::-webkit-slider-thumb {
-    width: 14px;
-    height: 14px;
-    margin-top: -5px;
-    border: 2px solid rgba(14,12,9,.72);
-    border-radius: 50%;
-    background: rgba(232,213,176,.86);
-    -webkit-appearance: none;
-}
-
-.rd-speed-dial::-moz-range-track {
-    height: 4px;
-    border: 0;
-    border-radius: 999px;
-    background: rgba(232,213,176,.18);
-}
-
-.rd-speed-dial::-moz-range-thumb {
-    width: 14px;
-    height: 14px;
-    border: 2px solid rgba(14,12,9,.72);
-    border-radius: 50%;
-    background: rgba(232,213,176,.86);
-}
-
-.rd-speed-dial-value {
-    overflow: hidden;
-    color: rgba(232,213,176,.64);
-    font-family: var(--mono);
-    font-size: 9px;
-    font-weight: 700;
-    line-height: 1;
-    text-align: right;
-    white-space: nowrap;
-}
 .rd-block-indicator {
     font-size: 10px;
     font-weight: 600;
diff --git a/tests/PrompterOne.Web.Tests/Teleprompter/TeleprompterControlRailTests.cs b/tests/PrompterOne.Web.Tests/Teleprompter/TeleprompterControlRailTests.cs
@@ -82,7 +82,7 @@ public void TeleprompterPage_RendersRailTooltipsForControlsAndSliders()
     }
 
     [Test]
-    public void TeleprompterPage_RendersSpeedDialInBottomTransportControls()
+    public void TeleprompterPage_RendersSimpleSpeedControlsWithoutInlineDial()
     {
         TestHarnessFactory.Create(this);
         Services.GetRequiredService<NavigationManager>()
@@ -93,12 +93,12 @@ public void TeleprompterPage_RendersSpeedDialInBottomTransportControls()
         cut.WaitForAssertion(() =>
         {
             var controls = cut.FindByTestId(UiTestIds.Teleprompter.Controls);
-            var speedDial = cut.FindByTestId(UiTestIds.Teleprompter.SpeedDial);
-            var speedDialShell = speedDial.ParentElement;
 
-            Assert.Contains(UiTestIds.Teleprompter.SpeedDial, controls.InnerHtml, StringComparison.Ordinal);
-            Assert.NotNull(speedDialShell);
-            Assert.Contains("rd-speed-dial-inline", speedDialShell!.ClassName, StringComparison.Ordinal);
+            Assert.Contains(UiTestIds.Teleprompter.SpeedDown, controls.InnerHtml, StringComparison.Ordinal);
+            Assert.Contains(UiTestIds.Teleprompter.SpeedValue, controls.InnerHtml, StringComparison.Ordinal);
+            Assert.Contains(UiTestIds.Teleprompter.SpeedUp, controls.InnerHtml, StringComparison.Ordinal);
+            Assert.DoesNotContain("rd-speed-dial", controls.InnerHtml, StringComparison.Ordinal);
+            Assert.DoesNotContain("teleprompter-speed-dial", controls.InnerHtml, StringComparison.Ordinal);
         });
     }
 
diff --git a/tests/PrompterOne.Web.Tests/Teleprompter/TeleprompterPersistenceTests.cs b/tests/PrompterOne.Web.Tests/Teleprompter/TeleprompterPersistenceTests.cs
diff --git a/tests/PrompterOne.Web.UITests.Reader/Teleprompter/TeleprompterSettingsFlowTests.cs b/tests/PrompterOne.Web.UITests.Reader/Teleprompter/TeleprompterSettingsFlowTests.cs
