notes-app

popup menu display corrupted in notes-app

Bug #1226990 reported by kylea on 2013-09-18

This bug affects 3 people

	Status	Importance	Assigned to
Ubuntu UI Toolkit	Invalid	Critical	Zsombor Egri
notes-app	Fix Released	Undecided	Unassigned
notes-app (Ubuntu)	Fix Released	High	Ugo Riboni
Saucy	Fix Released	High	Ugo Riboni
ubuntu-ui-toolkit (Ubuntu)	Invalid	Critical	Zsombor Egri
Saucy	Invalid	Critical	Zsombor Egri

Bug Description

Create a Note - with text, press and hold to select part of the text to Copy the text. Select All control appears, however no other options present to copy and paste the text.

Sometimes the menu is collapse and you see text on top of each other which looks corrupted.

See original description

Related branches

lp:~amanzi-team/notes-app/notes-app-remove-deprecated-action-prop

Merged into lp:notes-app at revision 200

PS Jenkins bot: Approve (continuous-integration) on 2013-10-06

Bill Filler (community): Approve on 2013-10-06

Zsombor Egri (community): Approve on 2013-10-02

lp:ubuntu/saucy-proposed/notes-app

Revision history for this message

Riccardo Padovani (rpadovani) wrote on 2013-09-19:

device-2013-09-19-195958.png Edit (68.1 KiB, image/png)

I can confirm, it seems that menu is collapsed (see screenshot)

Nexus 4 build 20130919.1

Revision history for this message

Olivier Tilloy (osomon) wrote on 2013-09-20:

I can reliably reproduce the issue in notes-app. My guess is that it’s a bug in the UITK.

Changed in notes-app:
status:	New → Confirmed

Revision history for this message

Launchpad Janitor (janitor) wrote on 2013-09-27:

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

Changed in notes-app (Ubuntu):
status:	New → Confirmed
Changed in ubuntu-ui-toolkit (Ubuntu):
status:	New → Confirmed

Bill Filler (bfiller) on 2013-09-30

Changed in ubuntu-ui-toolkit:
importance:	Undecided → Critical
assignee:	nobody → Zsombor Egri (zsombi)
Changed in notes-app (Ubuntu Saucy):
importance:	Undecided → High
assignee:	nobody → Ugo Riboni (uriboni)
Changed in ubuntu-ui-toolkit (Ubuntu Saucy):
assignee:	nobody → Zsombor Egri (zsombi)
importance:	Undecided → Critical
summary:	- Cannot Copy or Cut and Paste text on a Note + popup menu display corrupted in notes-app
description:	updated

Revision history for this message

Zsombor Egri (zsombi) wrote on 2013-10-01:

Are you using the TextField's default Popover, or a custom one?

Revision history for this message

Zsombor Egri (zsombi) wrote on 2013-10-01:

Sorry, I meant TextArea not TextField.

Revision history for this message

Ugo Riboni (uriboni) wrote on 2013-10-01:

I'm using a custom one because I need to enable disable options based on what mime types are in the clipboard, and react to them in a special way as well (to paste images).
If there's a way to use the standard popup and still do these things, please poke me and let's figure out how to do that. I'm all for simpler code that uses standard stuff.

Revision history for this message

Zsombor Egri (zsombi) wrote on 2013-10-02:

There has to be something on how the Popover is used...

Open Browser, copy some frame into the Clipboard and then go to Notes, open a card and long press to invoke the Popover. There will be two rows, one messed up with Select All|Copy|Other unreadable and a row with Paste.

But then I saw you are using the deprecated Action.visible property. You should use Action.enabled property. Then all will work nicely. I'd recommend to check the logs time to time as you would have seen the "Action.visible is a DEPRECATED property. Use ActionItems to specify the representation of an Action." note there, though even that should be fixed...

Changed in ubuntu-ui-toolkit (Ubuntu Saucy):
status:	Confirmed → Invalid

Ugo Riboni (uriboni) on 2013-10-02

Changed in notes-app (Ubuntu Saucy):
status:	Confirmed → In Progress

Revision history for this message

PS Jenkins bot (ps-jenkins) wrote on 2013-10-06:

Fix committed into lp:notes-app at revision 200, scheduled for release in notes-app, milestone ubuntu-13.04-month-5

Changed in notes-app:
status:	Confirmed → Fix Committed

Revision history for this message

Launchpad Janitor (janitor) wrote on 2013-10-07:

#10

This bug was fixed in the package notes-app - 1.4+13.10.20131007.1-0ubuntu1

---------------
notes-app (1.4+13.10.20131007.1-0ubuntu1) saucy; urgency=low

  [ Ugo Riboni ]
  * Use Action.enabled instead of Action.visible which was deprecated
    and causing visual glitches. (LP: #1226990)

  [ Francis Ginther ]
  * Provide alternate location of offline storage in response to sdk
    change.

  [ Olivier Tilloy ]
  * Attempt at fixing a race condition in autopilot tests: wait for the
    contents of a collapsed note to be populated before interacting with
    the note.

[ Ubuntu daily release ]
* Automatic snapshot from revision 200
-- Ubuntu daily release <email address hidden> Mon, 07 Oct 2013 05:18:36 +0000

Changed in notes-app (Ubuntu Saucy):
status:	In Progress → Fix Released

Zsombor Egri (zsombi) on 2013-10-08

Changed in ubuntu-ui-toolkit:
status:	New → Invalid

Olivier Tilloy (osomon) on 2013-10-30

Changed in notes-app:
status:	Fix Committed → Fix Released

Report a bug

This report contains Public information

Everyone can see this information.

Duplicates of this bug

Bug #1231880

You are

Subscribing...

Edit bug mail

Other bug subscribers

Bug attachments

device-2013-09-19-195958.png Edit

Add attachment

Remote bug watches

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