Opened 13 years ago

Closed 13 years ago

Last modified 11 years ago

#2608 closed defect (fixed)

Cursor moves to beginning while trying to edit activity instance name

Reported by: m_anish Owned by:
Priority: Unspecified by Maintainer Milestone: Unspecified
Component: Sugar Version: Unspecified
Severity: Unspecified Keywords: 11.2.0
Cc: alsroot, dirakx Distribution/OS: Unspecified
Bug Status: New

Description

When we try to edit the activity instance name to save a meaningful journal entry, the cursor moves to the beginning of the line after every 3-4 keystrokes.

Seen on Dextrose-2 XO-1 and XO-1.5

Attachments (6)

org.laptop.AbiWordActivity-1.log (2.0 KB) - added by m_anish 13 years ago.
Write Activity: log 1
org.laptop.AbiWordActivity-2.log (2.1 KB) - added by m_anish 13 years ago.
Write Activity: log 2
shell.log (11.8 KB) - added by m_anish 13 years ago.
shell.log
org.laptop.Terminal-1.log (1.5 KB) - added by m_anish 13 years ago.
Terminal activity log
com.garycmartin.Moon-1.log (1.7 KB) - added by m_anish 13 years ago.
Moon activity log
0001-Cursor-moves-to-beginning-while-trying-to-edit-activ.patch (1.1 KB) - added by alsroot 13 years ago.

Download all attachments as: .zip

Change History (21)

comment:1 follow-up: Changed 13 years ago by sascha_silbe

Probably a dupe of #1948.

comment:2 in reply to: ↑ 1 Changed 13 years ago by alsroot

Replying to sascha_silbe:

Probably a dupe of #1948.

And we already have this fix in dx-2

comment:3 Changed 13 years ago by alsroot

  • Owner changed from bernie, smparrish to alsroot
  • Status changed from new to assigned

comment:4 Changed 13 years ago by alsroot

In Write's title widget it works fine on dx2 on XO-1/1.5.
m_anish: are there any errors in shell.log and activity log while you can reproduce the issue?

comment:5 follow-up: Changed 13 years ago by alsroot

  • Bug Status changed from New to Needinfo

Changed 13 years ago by m_anish

Write Activity: log 1

Changed 13 years ago by m_anish

Write Activity: log 2

Changed 13 years ago by m_anish

shell.log

Changed 13 years ago by m_anish

Terminal activity log

Changed 13 years ago by m_anish

Moon activity log

comment:6 in reply to: ↑ 5 Changed 13 years ago by m_anish

  • Bug Status changed from Needinfo to New
  • Component changed from Dextrose to Write

Replying to alsroot:

This issue seems limited to Write activity. See all the logfiles attached. Reproduced on XO-1; dx-2

comment:7 Changed 13 years ago by alsroot

  • Component changed from Write to sugar-toolkit
  • Keywords r? added; dx2 removed
  • Priority changed from Urgent to Unspecified by Maintainer
  • Severity changed from Critical to Unspecified
  • Summary changed from [dx2] Cursor moves to beginning while trying to edit activity instance name to Cursor moves to beginning while trying to edit activity instance name

Patch was sent to sugar-devel@.
Removing any dx related metadata since patch is applied to dx-2 repo.

comment:8 Changed 13 years ago by alsroot

  • Owner alsroot deleted

comment:9 Changed 13 years ago by erikos

  • Keywords 11.2.0 added

comment:10 Changed 13 years ago by dsd

I briefly revisited old discussions on this issue and I did some thinking, and I think the following approach is sensible:

  • If the instance name textbox has the gtk.HAS_FOCUS flag, ignore all updates from datastore (don't update title)
  • Update the title widget with the new title if the datastore sends an update and the textbox isn't focused
  • Write the new title to the datastore on the focus-out event.
  • If editing the title then clicking the Stop button doesn't cause a focus-out event, and it can't be seen how to obviously make that focus-out event occur before Stop's clicked event is triggered, make the activity stop handler look for changes in the activity title and update the datastore if necessary

I think this should solve all the issues that were raised before, but I might have missed something.

comment:12 Changed 13 years ago by erikos

#2914 is a follow up issue about a minor glitch.

comment:13 Changed 13 years ago by erikos

Can be tested in build os871.

comment:14 Changed 13 years ago by erikos

  • Keywords r? removed

comment:15 Changed 11 years ago by dnarvaez

  • Component changed from sugar-toolkit to Sugar
Note: See TracTickets for help on using tickets.