<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugs.kde.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.6"
          urlbase="https://bugs.kde.org/"
          
          maintainer="sysadmin@kde.org"
>

    <bug>
          <bug_id>438526</bug_id>
          
          <creation_ts>2021-06-12 23:39:32 +0000</creation_ts>
          <short_desc>Page Scrollbar doesn&apos;t handle scroll events</short_desc>
          <delta_ts>2023-01-24 14:58:41 +0000</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>3</classification_id>
          <classification>Frameworks and Libraries</classification>
          <product>frameworks-kirigami</product>
          <component>general</component>
          <version>5.83.0</version>
          <rep_platform>Other</rep_platform>
          <op_sys>Linux</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc>https://invent.kde.org/frameworks/kirigami/-/merge_requests/896</bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>NOR</priority>
          <bug_severity>normal</bug_severity>
          <target_milestone>Not decided</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Carl Schwan">carl</reporter>
          <assigned_to name="ratijas">me</assigned_to>
          <cc>ahiemstra</cc>
    
    <cc>kde</cc>
    
    <cc>me</cc>
    
    <cc>nate</cc>
    
    <cc>noahadvs</cc>
    
    <cc>postix</cc>
          
          <cf_commitlink>https://invent.kde.org/frameworks/kirigami/commit/87f0ed1af4d43d3e23aad68d484a9863690dd832</cf_commitlink>
          <cf_versionfixedin>5.103</cf_versionfixedin>
          <cf_sentryurl></cf_sentryurl>
          <votes>0</votes>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>2040104</commentid>
    <comment_count>0</comment_count>
    <who name="Carl Schwan">carl</who>
    <bug_when>2021-06-12 23:39:32 +0000</bug_when>
    <thetext>SUMMARY


STEPS TO REPRODUCE
1. move mouse to scrollbar
2. scroll with the touchpad

OBSERVED RESULT

The ScrollablePage associated with the scrollbar doesn&apos;t scroll

EXPECTED RESULT

The ScrollablePage associated with the scrollbar scroll exactly like how it would work anywhere else on the page.

SOFTWARE/OS VERSIONS
openSUSE Tumbleweed
KDE Frameworks Version: master
Qt Version: 5.15.3</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2040364</commentid>
    <comment_count>1</comment_count>
    <who name="Arjen Hiemstra">ahiemstra</who>
    <bug_when>2021-06-14 08:07:34 +0000</bug_when>
    <thetext>Hmm, Kirigami&apos;s ScrollablePage uses a custom implementation of scroll views so I&apos;m not sure if this is the fault of the scrollbar control or the scrollview in Kirigami.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2090538</commentid>
    <comment_count>2</comment_count>
    <who name="Nate Graham">nate</who>
    <bug_when>2022-01-06 17:28:29 +0000</bug_when>
    <thetext>I checked and it&apos;s a Kirigami issue. This is handled properly in the base component.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2105101</commentid>
    <comment_count>3</comment_count>
    <who name="Nate Graham">nate</who>
    <bug_when>2022-02-15 21:35:20 +0000</bug_when>
    <thetext>*** Bug 450307 has been marked as a duplicate of this bug. ***</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2200909</commentid>
    <comment_count>4</comment_count>
    <who name="ratijas">me</who>
    <bug_when>2023-01-23 14:55:38 +0000</bug_when>
    <thetext>Git commit dba5d112db45f93f916525cf19d4a05b90bfdab7 by ivan tkachenko.
Committed on 23/01/2023 at 14:33.
Pushed by ratijas into branch &apos;master&apos;.

WheelHandler: Fix scrolling over ScrollBars on ScrollablePage

Code didn&apos;t handle re-parenting of Flickable to a ScrollView, which
occurs in ScrollablePage by design. This prevented scrolling over
scrollbars in such prominent places as list of System Settings modules
or content pane of Discover.

This solution handles re-parenting as well re-assigning either of the
two scrollbars via attached properties. The only precondition is that
ScrollBar attached object should exist before the target is set, so that
we have something to subscribe to.

M  +83   -33   src/wheelhandler.cpp
M  +5    -0    src/wheelhandler.h

https://invent.kde.org/frameworks/kirigami/commit/dba5d112db45f93f916525cf19d4a05b90bfdab7</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2200910</commentid>
    <comment_count>5</comment_count>
    <who name="ratijas">me</who>
    <bug_when>2023-01-23 14:57:32 +0000</bug_when>
    <thetext>Git commit 87f0ed1af4d43d3e23aad68d484a9863690dd832 by ivan tkachenko.
Committed on 23/01/2023 at 14:57.
Pushed by ratijas into branch &apos;kf5&apos;.

WheelHandler: Fix scrolling over ScrollBars on ScrollablePage

Code didn&apos;t handle re-parenting of Flickable to a ScrollView, which
occurs in ScrollablePage by design. This prevented scrolling over
scrollbars in such prominent places as list of System Settings modules
or content pane of Discover.

This solution handles re-parenting as well re-assigning either of the
two scrollbars via attached properties. The only precondition is that
ScrollBar attached object should exist before the target is set, so that
we have something to subscribe to.
(cherry picked from commit dba5d112db45f93f916525cf19d4a05b90bfdab7)

M  +83   -33   src/wheelhandler.cpp
M  +5    -0    src/wheelhandler.h

https://invent.kde.org/frameworks/kirigami/commit/87f0ed1af4d43d3e23aad68d484a9863690dd832</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>