From dde487258df5572f87c102bfd3a3bd6c9bd34a9e Mon Sep 17 00:00:00 2001
From: inotia00 <ars258000@gmail.com>
Date: Thu, 20 Apr 2023 07:42:00 +0900
Subject: [PATCH] feat(sponsorblock): skip to video highlight

---
 .../youtube/settings/host/values/strings.xml  |  8 ++++-
 .../layout/inline_sponsor_overlay.xml         | 32 +++++++++++++------
 .../layout/skip_sponsor_button.xml            |  1 -
 3 files changed, 29 insertions(+), 12 deletions(-)

diff --git a/src/main/resources/youtube/settings/host/values/strings.xml b/src/main/resources/youtube/settings/host/values/strings.xml
index f40359f54..3965520bc 100644
--- a/src/main/resources/youtube/settings/host/values/strings.xml
+++ b/src/main/resources/youtube/settings/host/values/strings.xml
@@ -573,6 +573,8 @@ Since these setting is quite outdated, it may not be valid"</string>
     <string name="sb_segments_selfpromo_sum">Similar to \'Sponsor\' except for unpaid or self promotion. Includes sections about merchandise, donations, or information about who they collaborated with</string>
     <string name="sb_segments_interaction">Interaction Reminder (Subscribe)</string>
     <string name="sb_segments_interaction_sum">A short reminder to like, subscribe or follow them in the middle of content. If it is long or about something specific, it should instead be under self promotion</string>
+    <string name="sb_segments_highlight">Highlight</string>
+    <string name="sb_segments_highlight_sum">The part of the video that most people are looking for</string>
     <string name="sb_segments_intro">Intermission/Intro Animation</string>
     <string name="sb_segments_intro_sum">An interval without actual content. Could be a pause, static frame, or repeating animation. Does not include transitions containing information</string>
     <string name="sb_segments_outro">Endcards/Credits</string>
@@ -585,9 +587,11 @@ Since these setting is quite outdated, it may not be valid"</string>
     <string name="sb_segments_nomusic_sum">Only for use in music videos. Sections of music videos without music, that aren\'t already covered by another category</string>
 
     <string name="sb_skip_button_compact">Skip</string>
+    <string name="sb_skip_button_compact_highlight">Highlight</string>
     <string name="sb_skip_button_sponsor">Skip sponsor</string>
     <string name="sb_skip_button_selfpromo">Skip promo</string>
     <string name="sb_skip_button_interaction">Skip interact</string>
+    <string name="sb_skip_button_highlight">Skip to highlight</string>
     <string name="sb_skip_button_intro_beginning">Skip intro</string>
     <string name="sb_skip_button_outro">Skip outro</string>
     <string name="sb_skip_button_preview_beginning">Skip preview</string>
@@ -598,6 +602,7 @@ Since these setting is quite outdated, it may not be valid"</string>
     <string name="sb_skipped_sponsor">Skipped sponsor</string>
     <string name="sb_skipped_selfpromo">Skipped self promotion</string>
     <string name="sb_skipped_interaction">Skipped annoying reminder</string>
+    <string name="sb_skipped_highlight">Skipped to highlight</string>
     <string name="sb_skipped_intro_beginning">Skipped intro</string>
     <string name="sb_skipped_outro">Skipped outro</string>
     <string name="sb_skipped_preview_beginning">Skipped preview</string>
@@ -633,7 +638,7 @@ Since these setting is quite outdated, it may not be valid"</string>
     <string name="sb_new_segment_choose_category">Choose the segment category</string>
     <string name="sb_new_segment_disabled_category">Category is disabled in settings. Enable category to submit.</string>
     <string name="sb_new_segment_title">New SponsorBlock segment</string>
-    <string name="sb_new_segment_mark_time_as_question" formatted="false">Set %02d:%02d:%04d as the start or end of a new segment?</string>
+    <string name="sb_new_segment_mark_time_as_question" formatted="false">Set %02d:%02d:%03d as the start or end of a new segment?</string>
     <string name="sb_new_segment_mark_start">start</string>
     <string name="sb_new_segment_mark_end">end</string>
     <string name="sb_new_segment_now">now</string>
@@ -641,6 +646,7 @@ Since these setting is quite outdated, it may not be valid"</string>
     <string name="sb_new_segment_time_end">Time the segment ends at</string>
     <string name="sb_new_segment_confirm_title">Are the times correct?</string>
     <string name="sb_new_segment_confirm_content" formatted="false">The segment lasts from %02d:%02d to %02d:%02d (%d minutes %02d seconds)\nIs it ready to submit?</string>
+    <string name="sb_new_segment_start_is_before_end">Start must be before the end</string>
     <string name="sb_new_segment_mark_locations_first">Mark two locations on the time bar first</string>
     <string name="sb_new_segment_preview_segment_first">Preview the segment, and ensure it skips smoothly</string>
     <string name="sb_new_segment_edit_by_hand_title">Edit timing of segment manually</string>
diff --git a/src/main/resources/youtube/sponsorblock/layout/inline_sponsor_overlay.xml b/src/main/resources/youtube/sponsorblock/layout/inline_sponsor_overlay.xml
index 9f4221735..2c0385549 100644
--- a/src/main/resources/youtube/sponsorblock/layout/inline_sponsor_overlay.xml
+++ b/src/main/resources/youtube/sponsorblock/layout/inline_sponsor_overlay.xml
@@ -1,6 +1,28 @@
 <?xml version="1.0" encoding="utf-8"?>
 <merge xmlns:android="http://schemas.android.com/apk/res/android">
 
+    <app.revanced.integrations.sponsorblock.ui.SkipSponsorButton
+        android:id="@+id/sb_skip_highlight_button"
+        android:contentDescription="@string/sb_skip_button_compact_highlight"
+        android:layout_width="wrap_content"
+        android:layout_height="wrap_content"
+        android:layout_alignParentLeft="true"
+        android:layout_alignParentBottom="true"
+        android:layout_marginBottom="@dimen/inline_controls_bottom_bar_height"
+        android:focusable="true"
+        android:visibility="gone" />
+
+    <app.revanced.integrations.sponsorblock.ui.SkipSponsorButton
+        android:id="@+id/sb_skip_sponsor_button"
+        android:contentDescription="@string/sb_skip_button_compact"
+        android:layout_width="wrap_content"
+        android:layout_height="wrap_content"
+        android:layout_alignParentRight="true"
+        android:layout_alignParentBottom="true"
+        android:layout_marginBottom="@dimen/inline_controls_bottom_bar_height"
+        android:focusable="true"
+        android:visibility="gone" />
+
     <app.revanced.integrations.sponsorblock.ui.NewSegmentLayout
         android:id="@+id/sb_new_segment_view"
         android:layout_width="wrap_content"
@@ -10,14 +32,4 @@
         android:layout_marginBottom="@dimen/brand_interaction_default_bottom_margin"
         android:focusable="true"
         android:visibility="gone" />
-
-    <app.revanced.integrations.sponsorblock.ui.SkipSponsorButton
-        android:id="@+id/sb_skip_sponsor_button"
-        android:layout_width="wrap_content"
-        android:layout_height="wrap_content"
-        android:layout_alignParentRight="true"
-        android:layout_alignParentBottom="true"
-        android:layout_marginBottom="@dimen/inline_controls_bottom_bar_height"
-        android:focusable="true"
-        android:visibility="gone" />
 </merge>
\ No newline at end of file
diff --git a/src/main/resources/youtube/sponsorblock/layout/skip_sponsor_button.xml b/src/main/resources/youtube/sponsorblock/layout/skip_sponsor_button.xml
index 3a3167bb8..185960ec9 100644
--- a/src/main/resources/youtube/sponsorblock/layout/skip_sponsor_button.xml
+++ b/src/main/resources/youtube/sponsorblock/layout/skip_sponsor_button.xml
@@ -7,7 +7,6 @@
         android:layout_width="wrap_content"
         android:layout_height="32dp"
         android:layout_gravity="center_vertical"
-        android:contentDescription="@string/sb_skip_button_compact"
         android:orientation="horizontal"
         android:padding="8dp">