[regression] Cut/Copy/Paste options displayed off the top of the screen from the addressbar

[Michael Sheldon]

Steps to reproduce

1) Long press on the addressbar

Expected result

Cut/Copy/Paste option should appear in a pop-up below the addressbar

Actual result

The pop-up is displayed above the addressbar, meaning only the bottom of it is visible, the rest is cut off by the notification area/top of the screen.

mako, vivid-proposed #72

[Olivier Tilloy (osomon)]

This is a rather recent regression, most likely in the UITK (as the issue is not present on RTM, although webbrowser-app was recently synced back from vivid to RTM, so both versions are virtually identical).

[Olivier Tilloy (osomon)]

I can easily reproduce with the following standalone test app:

import QtQuick 2.0
import Ubuntu.Components 1.1

MainView {
    TextField {
        anchors {
            top: parent.top
            left: parent.left
            right: parent.right
            margins: units.gu(2)
        }
        focus: true
        text: "long-press me to invoke contextual options"
        Component.onCompleted: selectAll()
    }
}

[Dan Chapman  (dpniel)]

I can confirm i'm seeing this too on Vivid-proposed #72.

[Launchpad Janitor (janitor)]

Status changed to 'Confirmed' because the bug affects multiple users.

[Dan Chapman  (dpniel)]

Screenshot of bug

[Christian Dywan (kalikiana)]

Confirmed it's a regression in UITK, I'm in the process of tracking down the change.

[Christian Dywan (kalikiana)]

FTR offending change is http://bazaar.launchpad.net/~ubuntu-sdk-team/ubuntu-ui-toolkit/staging/revision/1371 and I'm working on a way to test the positioning in code (the test case I have relies on the human eye to see the bug), and afterwards will look into a fix.

[Michał Sawicz (saviq)]

This also seems to be the cause of the dash search history appearing off-screen.

[Launchpad Janitor (janitor)]

Status changed to 'Confirmed' because the bug affects multiple users.

[Launchpad Janitor (janitor)]

This bug was fixed in the package ubuntu-ui-toolkit - 1.2.1433+15.04.20150306-0ubuntu1

---------------
ubuntu-ui-toolkit (1.2.1433+15.04.20150306-0ubuntu1) vivid; urgency=medium

  [ Albert Astals Cid ]
  * Fix infinite icon size grow when asked for a non square icon
    Fixes: LP: #1422835
  * IconProvider: Make sure icons fit in the given size
  [ Christian Dywan ]
  * Skip select_text_longtap_when_active under opengles2.
    Fixes: LP: #1401140
  * Support for multiple devices with push_to_phone.sh.
  * Take OrientationHelper y coordinate from real parent height.
    Fixes: LP: #1413194
  * Update popover position upon pointer movement.
    Fixes: LP: #1199502, LP: #1415532
  * Change <code> to be inline-block. Fixes LP: #1427534
  * Set right anchor on tab overflow buttons. Fixes LP: #1425433
  * Add tst_listitems_itemselector.qml to EXCEPTIONS. So warnings are
    no longer fatal for unit test.
  [ Sebastien Bacher ]
  * document that MainView's applicationName is used to set the gettext
    domain. Fixes: LP: #1421661.
  [ Tiago Salem Herrmann ]
  * Drop cache once properties are successfully restored.
  [ Timo Jyrinki ]
  * Add dist-upgrade option to the UITK Test Plan script. Output the
    selected parameters in the beginning.
  [ Tim Peeters ]
  * Add MainView 1.2 which removes the deprecated toolbar support.
  * Enable PageStack header animations for MainView 1.2.
  * Fix the size and anchors of a Page that is loaded inside a Loader.
    This is useful when a Page is Loaded by a Tab. Fixes: LP: #1424635
  * Fix UITK gallery content page width.
    Fixes: LP: #1424635, LP: #1425470
  * Set objectName for MainView int est_textinput.header.qml and fix
    scenarios in test_textinput.py
  [ Zsombor Egri ]
  * Alarm services regression fix. Fixes: LP: #1401883
  * Implement ListItem dragging support in ListView.
    Fixes: LP: #119535
  * Introducing select/multiselect mode in ListItem.
  * Moving the tst_focus.qml under exceptions till we fix DatePicker
    binding loops.
  * RTL (Right To Left) support in ListItem.
  * RTM branch diff merge. Fixes: LP: #1358610.
  * Second stage of simplification - divider group property is an item,
    simplifying the anchoring of the contentItem. PropertyChange does
    not backup the property binding, uses QQmlPropertyPrivate::write()
    with parameter to keep bindings. ListItemAnimator uses behavior to
    animate snapping. Styling got "styledItem" context property to obey
    general styling rules. clicked() and pressAndHold() emission fix
    when content is swiped.
  * Transferring the ListItem attached properties to ListItemStyle API.
    Most of the style API became redundant, and removed. Few API from
    ListItem were also removed as not needed. ListItemAnimator,
    UCListItemAttached and UCActionPanel classes removed.
    Panel swiping overshoot made elastic. Snapping animation transformed
    into SmoothedAnimation, uses velocity of 60GU/second. Turning
    warnings to be reported in the error property.
    SHOW_SERVICEPROPERTIES_WARNINGS environment variable set will
    display them in the console. Fixes: LP: #1423203.
  * Updating documentation and imports for 1.2 & Vivid (Qt 5.4).
...