Skip to content

Commit a6f57a8

Browse files
authored
Merge pull request TeamNewPipe#9173 from Theta-Dev/video-sub-count
Show subscriber count on video details page
2 parents 3bb5187 + 0df6967 commit a6f57a8

3 files changed

Lines changed: 25 additions & 10 deletions

File tree

app/src/main/java/org/schabi/newpipe/fragments/detail/VideoDetailFragment.java

Lines changed: 25 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1561,9 +1561,9 @@ public void handleResult(@NonNull final StreamInfo info) {
15611561
binding.detailSubChannelThumbnailView.setVisibility(View.GONE);
15621562

15631563
if (!isEmpty(info.getSubChannelName())) {
1564-
displayBothUploaderAndSubChannel(info);
1564+
displayBothUploaderAndSubChannel(info, activity);
15651565
} else if (!isEmpty(info.getUploaderName())) {
1566-
displayUploaderAsSubChannel(info);
1566+
displayUploaderAsSubChannel(info, activity);
15671567
} else {
15681568
binding.detailUploaderTextView.setVisibility(View.GONE);
15691569
binding.detailUploaderThumbnailView.setVisibility(View.GONE);
@@ -1676,23 +1676,42 @@ public void handleResult(@NonNull final StreamInfo info) {
16761676
noVideoStreams ? R.drawable.ic_headset_shadow : R.drawable.ic_play_arrow_shadow);
16771677
}
16781678

1679-
private void displayUploaderAsSubChannel(final StreamInfo info) {
1679+
private void displayUploaderAsSubChannel(final StreamInfo info, final Context context) {
16801680
binding.detailSubChannelTextView.setText(info.getUploaderName());
16811681
binding.detailSubChannelTextView.setVisibility(View.VISIBLE);
16821682
binding.detailSubChannelTextView.setSelected(true);
1683-
binding.detailUploaderTextView.setVisibility(View.GONE);
1683+
1684+
if (info.getUploaderSubscriberCount() > -1) {
1685+
binding.detailUploaderTextView.setText(
1686+
Localization.shortSubscriberCount(context, info.getUploaderSubscriberCount()));
1687+
binding.detailUploaderTextView.setVisibility(View.VISIBLE);
1688+
} else {
1689+
binding.detailUploaderTextView.setVisibility(View.GONE);
1690+
}
16841691
}
16851692

1686-
private void displayBothUploaderAndSubChannel(final StreamInfo info) {
1693+
private void displayBothUploaderAndSubChannel(final StreamInfo info, final Context context) {
16871694
binding.detailSubChannelTextView.setText(info.getSubChannelName());
16881695
binding.detailSubChannelTextView.setVisibility(View.VISIBLE);
16891696
binding.detailSubChannelTextView.setSelected(true);
16901697

16911698
binding.detailSubChannelThumbnailView.setVisibility(View.VISIBLE);
16921699

1700+
final StringBuilder subText = new StringBuilder();
16931701
if (!isEmpty(info.getUploaderName())) {
1694-
binding.detailUploaderTextView.setText(
1702+
subText.append(
16951703
String.format(getString(R.string.video_detail_by), info.getUploaderName()));
1704+
}
1705+
if (info.getUploaderSubscriberCount() > -1) {
1706+
if (subText.length() > 0) {
1707+
subText.append(Localization.DOT_SEPARATOR);
1708+
}
1709+
subText.append(
1710+
Localization.shortSubscriberCount(context, info.getUploaderSubscriberCount()));
1711+
}
1712+
1713+
if (subText.length() > 0) {
1714+
binding.detailUploaderTextView.setText(subText);
16961715
binding.detailUploaderTextView.setVisibility(View.VISIBLE);
16971716
binding.detailUploaderTextView.setSelected(true);
16981717
} else {

app/src/main/res/layout-large-land/fragment_video_detail.xml

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -319,9 +319,7 @@
319319
android:marqueeRepeatLimit="marquee_forever"
320320
android:scrollHorizontally="true"
321321
android:singleLine="true"
322-
android:textAppearance="?android:attr/textAppearanceLarge"
323322
android:textSize="@dimen/video_item_detail_uploader_text_size"
324-
android:textStyle="bold"
325323
tools:ignore="RtlHardcoded"
326324
tools:text="Uploader" />
327325
</LinearLayout>

app/src/main/res/layout/fragment_video_detail.xml

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -306,9 +306,7 @@
306306
android:marqueeRepeatLimit="marquee_forever"
307307
android:scrollHorizontally="true"
308308
android:singleLine="true"
309-
android:textAppearance="?android:attr/textAppearanceLarge"
310309
android:textSize="@dimen/video_item_detail_uploader_text_size"
311-
android:textStyle="bold"
312310
tools:ignore="RtlHardcoded"
313311
tools:text="Uploader" />
314312

0 commit comments

Comments
 (0)