Sulkta-OSS/straw
3
0
Fork 0
Code Issues 1 Pull requests 10 Projects Releases Packages Wiki Activity Actions

Improved accessibility of player interfaces

Browse source
This commit is contained in:
TobiGr 2023-09-18 03:11:28 +02:00
parent f6714ac356
commit f4f2a8ed73
8 changed files with 86 additions and 34 deletions
Download patch file Download diff file Expand all files Collapse all files

14
app/src/main/res/layout-land/activity_player_queue_control.xml
View file

@ -123,7 +123,7 @@
android:scaleType="fitCenter"
android:src="@drawable/exo_controls_rewind"
android:tint="?attr/colorAccent"
tools:ignore="ContentDescription" />
android:contentDescription="@string/rewind" />
<ImageButton
android:id="@+id/control_play_pause"
@ -140,7 +140,7 @@
android:scaleType="fitCenter"
android:src="@drawable/ic_pause"
android:tint="?attr/colorAccent"
tools:ignore="ContentDescription" />
android:contentDescription="@string/pause" />
<ProgressBar
android:id="@+id/control_progress_bar"
@ -173,7 +173,7 @@
android:scaleType="fitCenter"
android:src="@drawable/exo_controls_fastforward"
android:tint="?attr/colorAccent"
tools:ignore="ContentDescription" />
android:contentDescription="@string/forward" />
</RelativeLayout>
<RelativeLayout
@ -201,7 +201,7 @@
android:scaleType="fitCenter"
android:src="@drawable/ic_previous"
android:tint="?attr/colorAccent"
tools:ignore="ContentDescription" />
android:contentDescription="@string/previous_stream" />
<ImageButton
android:id="@+id/control_repeat"
@ -216,7 +216,7 @@
android:scaleType="fitXY"
android:src="@drawable/ic_repeat"
android:tint="?attr/colorAccent"
tools:ignore="ContentDescription" />
android:contentDescription="@string/notification_action_repeat" />
<View
android:id="@+id/anchor"
@ -237,7 +237,7 @@
android:scaleType="fitXY"
android:src="@drawable/ic_shuffle"
android:tint="?attr/colorAccent"
tools:ignore="ContentDescription" />
android:contentDescription="@string/notification_action_shuffle" />
<androidx.appcompat.widget.AppCompatImageButton
android:id="@+id/control_forward"
@ -255,7 +255,7 @@
android:scaleType="fitCenter"
android:src="@drawable/ic_next"
android:tint="?attr/colorAccent"
tools:ignore="ContentDescription" />
android:contentDescription="@string/next_stream" />
</RelativeLayout>
</RelativeLayout>

5
app/src/main/res/layout/fragment_video_detail.xml
View file

@ -614,7 +614,8 @@
android:gravity="center_vertical"
android:paddingLeft="@dimen/video_item_search_padding"
android:paddingRight="@dimen/video_item_search_padding"
android:scaleType="fitCenter" />
android:scaleType="fitCenter"
tools:ignore="ContentDescription" />
<LinearLayout
android:id="@+id/overlay_metadata_layout"
@ -675,7 +676,7 @@
android:contentDescription="@string/title_activity_play_queue"
android:scaleType="center"
android:src="@drawable/ic_list"
tools:ignore="ContentDescription,RtlHardcoded" />
tools:ignore="RtlHardcoded" />
<ImageButton
android:id="@+id/overlay_play_pause_button"

1
app/src/main/res/layout/play_queue_item.xml
View file

@ -39,6 +39,7 @@
android:textAppearance="?android:attr/textAppearanceSmall"
android:textColor="@color/duration_text_color"
android:textSize="@dimen/video_item_search_duration_text_size"
android:contentDescription="@string/duration"
app:layout_constraintBottom_toBottomOf="@+id/itemThumbnailView"
app:layout_constraintEnd_toEndOf="@+id/itemThumbnailView"
tools:text="1:09:10" />

39
app/src/main/res/layout/player.xml
View file

@ -115,7 +115,8 @@
android:src="@drawable/ic_close"
android:visibility="gone"
app:tint="@color/white"
tools:ignore="ContentDescription,RtlHardcoded" />
android:contentDescription="@string/close"
tools:ignore="RtlHardcoded" />
<LinearLayout
android:id="@+id/metadataView"
@ -217,7 +218,8 @@
android:src="@drawable/ic_list"
android:visibility="gone"
app:tint="@color/white"
tools:ignore="ContentDescription,RtlHardcoded" />
android:contentDescription="@string/open_play_queue"
tools:ignore="RtlHardcoded" />
<androidx.appcompat.widget.AppCompatImageButton
android:id="@+id/segmentsButton"
@ -235,7 +237,8 @@
android:src="@drawable/ic_menu_book"
android:visibility="gone"
app:tint="@color/white"
tools:ignore="ContentDescription,RtlHardcoded" />
android:contentDescription="@string/chapters"
tools:ignore="RtlHardcoded" />
<androidx.appcompat.widget.AppCompatImageButton
android:id="@+id/moreOptionsButton"
@ -248,7 +251,8 @@
android:scaleType="fitXY"
android:src="@drawable/ic_expand_more"
app:tint="@color/white"
tools:ignore="ContentDescription,RtlHardcoded" />
android:contentDescription="@string/more_options"
tools:ignore="RtlHardcoded" />
</LinearLayout>
@ -368,9 +372,10 @@
android:padding="@dimen/player_main_buttons_padding"
android:scaleType="fitCenter"
android:src="@drawable/ic_fullscreen"
android:contentDescription="@string/toggle_fullscreen"
android:visibility="gone"
app:tint="@color/white"
tools:ignore="ContentDescription,RtlHardcoded"
tools:ignore="RtlHardcoded"
tools:visibility="visible" />
</LinearLayout>
@ -493,8 +498,9 @@
android:scaleType="fitCenter"
android:src="@drawable/ic_fullscreen"
android:visibility="gone"
android:contentDescription="@string/toggle_screen_orientation"
app:tint="@color/white"
tools:ignore="ContentDescription,RtlHardcoded"
tools:ignore="RtlHardcoded"
tools:visibility="visible" />
</LinearLayout>
</RelativeLayout>
@ -517,8 +523,8 @@
android:focusable="true"
android:scaleType="fitCenter"
android:src="@drawable/ic_previous"
app:tint="@color/white"
tools:ignore="ContentDescription" />
android:contentDescription="@string/previous_stream"
app:tint="@color/white" />
<androidx.appcompat.widget.AppCompatImageButton
@ -529,8 +535,8 @@
android:background="?attr/selectableItemBackgroundBorderless"
android:scaleType="fitCenter"
android:src="@drawable/ic_pause"
app:tint="@color/white"
tools:ignore="ContentDescription" />
android:contentDescription="@string/pause"
app:tint="@color/white" />
<androidx.appcompat.widget.AppCompatImageButton
android:id="@+id/playNextButton"
@ -543,8 +549,8 @@
android:focusable="true"
android:scaleType="fitCenter"
android:src="@drawable/ic_next"
app:tint="@color/white"
tools:ignore="ContentDescription" />
android:contentDescription="@string/next_stream"
app:tint="@color/white" />
</LinearLayout>
@ -595,7 +601,8 @@
android:scaleType="fitXY"
android:src="@drawable/exo_controls_repeat_off"
android:tint="?attr/colorAccent"
tools:ignore="ContentDescription,RtlHardcoded" />
android:contentDescription="@string/notification_action_repeat"
tools:ignore="RtlHardcoded" />
<androidx.appcompat.widget.AppCompatImageButton
android:id="@+id/shuffleButton"
@ -610,7 +617,8 @@
android:scaleType="fitXY"
android:src="@drawable/ic_shuffle"
android:tint="?attr/colorAccent"
tools:ignore="ContentDescription,RtlHardcoded" />
android:contentDescription="@string/notification_action_shuffle"
tools:ignore="RtlHardcoded" />
<androidx.appcompat.widget.AppCompatTextView
android:id="@+id/itemsListHeaderDuration"
@ -636,7 +644,8 @@
android:scaleType="fitXY"
android:src="@drawable/ic_playlist_add"
android:tint="?attr/colorAccent"
tools:ignore="ContentDescription,RtlHardcoded" />
android:contentDescription="@string/add_to_playlist"
tools:ignore="RtlHardcoded" />
<androidx.appcompat.widget.AppCompatImageButton
android:id="@+id/itemsListClose"

1
app/src/main/res/layout/player_popup_close_overlay.xml
View file

@ -11,6 +11,7 @@
android:layout_gravity="bottom|center_horizontal"
android:layout_marginBottom="24dp"
android:src="@drawable/ic_close"
android:contentDescription="@string/close"
app:backgroundTint="@color/light_youtube_primary_color"
app:borderWidth="0dp"
app:fabSize="normal" />

11
app/src/main/res/values/strings.xml
View file

@ -811,4 +811,15 @@
<string name="channel_tab_about">About</string>
<string name="show_channel_tabs">Channel tabs</string>
<string name="show_channel_tabs_summary">What tabs are shown on the channel pages</string>
<string name="open_play_queue">Open play queue</string>
<string name="toggle_fullscreen">Toggle fullscreen</string>
<string name="toggle_screen_orientation">Toggle screen orientation</string>
<string name="previous_stream">Previous stream</string>
<string name="next_stream">Next stream</string>
<string name="play">Play</string>
<string name="replay">Replay</string>
<string name="more_options">More options</string>
<string name="duration">Duration</string>
<string name="rewind">Rewind</string>
<string name="forward">Forward</string>
</resources>