Scrollbars in TextArea don't receive mouse hover events

[Andrea Bernabei]

r1773

Scrollbars in TextArea currently don't receive mouse hover events, and that prevents them from providing the expected UX (the new scrollbar expands and shows steppers when hovered with a mouse)

By having a quick look at the code I noticed that moving the Scrollbars below the Flickable fixes the issue.

I believe that what is happening is that Mouse.forwardTo: [main] inside InputHandler is forcing the mouse events to skip the scrollbars and go directly to the root item, though I'm not sure about this theory.

[Andrea Bernabei (faenil)]

This is currently waiting on Visual/UX design.

@Designers: please provide the UX/visuals for scrollbars in a textarea.

TextAreas are currently rounded, while scrollbars "steppers"-style (the pointer friendly component you get when you hover over the scrollbar) is square.

Plus the Scrollbar currently has some padding inside the TextArea, afair.

[Grazina (boroskograzina)]

Design comment:
I suggest to leave 2gu between text and scrollbar. Image attached below shows text input design with scrollbar and when one of the steppers is on hover state.

[Grazina (boroskograzina)]

Design comment:

This bug is related to https://bugs.launchpad.net/ubuntu/+source/ubuntu-ui-toolkit/+bug/1578190
bug.

[Grazina (boroskograzina)]

Hi Christian,

it's almost there, could you amend bottom stepper background on hover state so it looks like in the image attached below.

Thank you.

[Launchpad Janitor (janitor)]

This bug was fixed in the package ubuntu-ui-toolkit - 1.3.1984+16.10.20160527.2

---------------
ubuntu-ui-toolkit (1.3.1984+16.10.20160527.2) yakkety; urgency=medium

  [ Christian Dywan ]
  * Slimmer frame for TextFields: 0.5dp. Fixes LP: #1578190.

  [ Albert Astals Cid ]
  * Add override
    The override specifier (since C++11) specifies that a virtual function
    overrides another virtual function. In a member function declaration or
    definition, override ensures that the function is virtual and is overriding
    a virtual function from the base class.

  [ Tim Peeters ]
  * Fix reference error in PullToRefreshStyle. Fixes LP: #1582843
  * Mark Tab, Tabs, TabBar, PageHeadConfiguration, PageHeadSections,
    PageHeadState, ToolbarButton, ToolbarItems as deprecated in the
    documentation. Fixes LP: #1566735, LP: #1566741.

  [ CI Train Bot ]
  * Resync trunk.

 -- Zoltan Balogh <email address hidden> Fri, 27 May 2016 07:08:44 +0000