feat: Improve segments visibility logic (#346)

Co-authored-by: PartyDonut <PartyDonut@users.noreply.github.com>
This commit is contained in:
PartyDonut 2025-05-17 16:36:33 +02:00 committed by GitHub
parent 1a8765fbd6
commit 947da2390f
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
3 changed files with 30 additions and 12 deletions

View file

@ -66,13 +66,13 @@ class OpenQueueButton extends ConsumerWidget {
class SkipSegmentButton extends ConsumerWidget {
final MediaSegment? segment;
final SegmentSkip? skipType;
final bool isOverlayVisible;
final SegmentVisibility visibility;
final Function() pressedSkip;
const SkipSegmentButton({
required this.segment,
this.skipType,
required this.isOverlayVisible,
required this.visibility,
required this.pressedSkip,
super.key,
});
@ -82,7 +82,11 @@ class SkipSegmentButton extends ConsumerWidget {
return AnimatedFadeSize(
child: segment != null && skipType != SegmentSkip.none
? AnimatedOpacity(
opacity: isOverlayVisible ? 1 : 0.15,
opacity: switch (visibility) {
SegmentVisibility.hidden => 0,
SegmentVisibility.partially => 0.15,
SegmentVisibility.visible => 1.0,
},
duration: const Duration(milliseconds: 500),
child: ElevatedButton(
onPressed: pressedSkip,

View file

@ -168,11 +168,13 @@ class _DesktopControlsState extends ConsumerState<DesktopControls> {
builder: (context, ref, child) {
final position = ref.watch(mediaPlaybackProvider.select((value) => value.position));
MediaSegment? segment = mediaSegments?.atPosition(position);
bool forceShow = segment?.forceShow(position) ?? false;
SegmentVisibility forceShow =
segment?.visibility(position, force: showOverlay) ?? SegmentVisibility.hidden;
final segmentSkipType = ref
.watch(videoPlayerSettingsProvider.select((value) => value.segmentSkipSettings[segment?.type]));
final autoSkip =
forceShow == true && segmentSkipType == SegmentSkip.skip && player.lastState?.buffering == false;
final autoSkip = forceShow != SegmentVisibility.hidden &&
segmentSkipType == SegmentSkip.skip &&
player.lastState?.buffering == false;
if (autoSkip) {
skipToSegmentEnd(segment);
}
@ -185,7 +187,7 @@ class _DesktopControlsState extends ConsumerState<DesktopControls> {
child: SkipSegmentButton(
segment: segment,
skipType: segmentSkipType,
isOverlayVisible: forceShow ? true : showOverlay,
visibility: forceShow,
pressedSkip: () => skipToSegmentEnd(segment),
),
),
@ -441,10 +443,9 @@ class _DesktopControlsState extends ConsumerState<DesktopControls> {
final List<String?> details = [
if (AdaptiveLayout.of(context).isDesktop) item?.label(context),
mediaPlayback.duration.inMinutes > 1
? context.localized.endsAt(DateTime.now().add(
Duration(milliseconds:
(mediaPlayback.duration.inMilliseconds - mediaPlayback.position.inMilliseconds) ~/ ref.read(playbackRateProvider)))
)
? context.localized.endsAt(DateTime.now().add(Duration(
milliseconds: (mediaPlayback.duration.inMilliseconds - mediaPlayback.position.inMilliseconds) ~/
ref.read(playbackRateProvider))))
: null
];
return Column(