Skip to content
Snippets Groups Projects

qml: make playlist listview's background opaque in small screen mode

Merged Fatih Uzunoğlu requested to merge fuzun/vlc:qt/smallwidthcorrection into master
All threads resolved!

Playqueue pane's background uses a hack to block what is shown beneath but instead makes window background to pass through. This hack is used because play queue is hierarchically placed in a position where there is background that is not fully transparent.

This conflicts with the "small screen" view, since the stack view does not get unusably small but rather blows up into the area used by the play queue. This constitutes a problem, because play queue effectively prevents rendering the part of the stack view that overlaps with its size.

Request review @chub.

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Author Reporter

    Play queue's opaque background comes from its parent, which is a Rectangle with color theme.bg.primary. One alternative might be to make it transparent, and use frosted glass effect for the stack view content. This might require moving the play queue to a new hierarchical position, where we might not be able to use anchors anymore.

    One may ask: Why currently play queue's parent is an opaque rectangle if play queue is meant to be transparent? That is done because the same parent acts as a background for the mini player, and mini player extends to the play queue (width-wise). We want to have consistent effect for the mini player background.

  • Pierre Lamot
  • added 1 commit

    • 652d8f94 - qml: make playlist listview's background opaque in small screen mode

    Compare with previous version

  • Fatih Uzunoğlu changed title from qml: do not show stack view beneath the play queue in "small screen" mode to qml: make playlist listview's background opaque in small screen mode

    changed title from qml: do not show stack view beneath the play queue in "small screen" mode to qml: make playlist listview's background opaque in small screen mode

  • Steve Lhomme changed milestone to %4.0

    changed milestone to %4.0

  • Pierre Lamot approved this merge request

    approved this merge request

  • Pierre Lamot resolved all threads

    resolved all threads

  • added MRStatus::Acceptable label and removed MRStatus::InReview label

  • added MRStatus::Accepted label and removed MRStatus::Acceptable label

  • MR Acceptance result

    :tada: This MergeRequest has been Accepted! Congratulations.

    MR acceptance checks details:

    • :white_check_mark: MR should be considered mergeable by Gitlab
    • :white_check_mark: Last pipeline should be successful
    • :white_check_mark: MergeRequest should have at least one external review and/or vote
    • :white_check_mark: All threads should be resolved, have votes and score > 0
    • :white_check_mark: MergeRequest should have no activity (threads/votes) for (72h/72h)

    This message was automatically generated by homer-bot.

  • Felix Paul Kühne added 159 commits

    added 159 commits

    • 652d8f94...9d84c7fa - 158 commits from branch videolan:master
    • 694d51a7 - qml: make playlist listview's background opaque in small screen mode

    Compare with previous version

  • Felix Paul Kühne enabled an automatic merge when the pipeline for 694d51a7 succeeds

    enabled an automatic merge when the pipeline for 694d51a7 succeeds

  • Please register or sign in to reply
    Loading