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

Bug #1413194 reported by Michael Sheldon
18
This bug affects 4 people
Affects Status Importance Assigned to Milestone
Canonical System Image
Fix Released
High
Unassigned
webbrowser-app
Invalid
Undecided
Unassigned
ubuntu-ui-toolkit (Ubuntu)
Fix Released
High
Cris Dywan
Vivid
New
Undecided
Unassigned
unity8 (Ubuntu)
Invalid
Undecided
Unassigned

Bug Description

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

Related branches

description: updated
Revision history for this message
Olivier Tilloy (osomon) wrote :

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).

Changed in webbrowser-app:
status: New → Invalid
Revision history for this message
Olivier Tilloy (osomon) wrote :

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()
    }
}

summary: - Cut/Copy/Paste options displayed off the top of the screen from the
- addressbar
+ [regression] Cut/Copy/Paste options displayed off the top of the screen
+ from the addressbar
Revision history for this message
Dan Chapman  (dpniel) wrote :

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

Revision history for this message
Launchpad Janitor (janitor) wrote :

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

Changed in ubuntu-ui-toolkit (Ubuntu):
status: New → Confirmed
Revision history for this message
Dan Chapman  (dpniel) wrote :

Screenshot of bug

Revision history for this message
Cris Dywan (kalikiana) wrote :

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

Changed in ubuntu-ui-toolkit (Ubuntu):
assignee: nobody → Christian Dywan (kalikiana)
importance: Undecided → High
Revision history for this message
Cris Dywan (kalikiana) wrote :

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.

Revision history for this message
Michał Sawicz (saviq) wrote :

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

Cris Dywan (kalikiana)
Changed in ubuntu-ui-toolkit (Ubuntu):
status: Confirmed → In Progress
Revision history for this message
Launchpad Janitor (janitor) wrote :

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

Changed in unity8 (Ubuntu):
status: New → Confirmed
Cris Dywan (kalikiana)
Changed in ubuntu-ui-toolkit (Ubuntu):
status: In Progress → Fix Committed
Changed in unity8 (Ubuntu):
status: Confirmed → Invalid
Revision history for this message
Launchpad Janitor (janitor) wrote :
Download full text (3.7 KiB)

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).
...

Read more...

Changed in ubuntu-ui-toolkit (Ubuntu):
status: Fix Committed → Fix Released
Changed in canonical-devices-system-image:
importance: Undecided → High
milestone: none → ww11-2015
status: New → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Bug attachments

Remote bug watches

Bug watches keep track of this bug in other bug trackers.