Ticket #1235 (closed defect: fixed)

Opened 4 years ago

Last modified 4 years ago

Journal list view: jumping back to first page when popping up a palette

Reported by: sascha_silbe Owned by: alsroot
Priority: Immediate Milestone: 0.86
Component: sugar Version: Git as of bugdate
Severity: Major Keywords: r+
Cc: erikos, garycmartin Distribution/OS: Unspecified
Bug Status: New

Description

The Journal list view jumps back to the first page when opening a palette, making it impossible to resume old entries.
The attached patch fixes that by refreshing the list only on change from fully-obscured to visible, not on every minor visibility change.

Attachments

0002-fix-Journal-list-view-jumping-back-to-first-page-on-.patch Download (1.3 KB) - added by sascha_silbe 4 years ago.
fix Journal list view jumping back to first page on showing a Palette
sugar-1235.patch Download (7.2 KB) - added by alsroot 4 years ago.
sugar-1235.2.patch Download (7.5 KB) - added by alsroot 4 years ago.
sugar-1235.3.patch Download (4.1 KB) - added by alsroot 4 years ago.
sugar-1235.4.patch Download (1.5 KB) - added by alsroot 4 years ago.

Change History

Changed 4 years ago by sascha_silbe

fix Journal list view jumping back to first page on showing a Palette

  Changed 4 years ago by tomeu

  • keywords r+ added; r? removed
  • owner changed from tomeu to sascha_silbe
  • status changed from new to assigned

Thanks, please push.

  Changed 4 years ago by alsroot

  Changed 4 years ago by alsroot

e.g. in case of changing entries w/o opening details view

  Changed 4 years ago by alsroot

last three comments are obsoleted

  Changed 4 years ago by alsroot

last patch is adapted version of previous two commit links

Changed 4 years ago by alsroot

  Changed 4 years ago by alsroot

last patch was rewrote according to irc discussion

Changed 4 years ago by alsroot

  Changed 4 years ago by alsroot

  • cc erikos, garycmartin added
  • status changed from assigned to closed
  • resolution set to fixed

at the end another approach was chosen - just change vadjustment value
 http://git.sugarlabs.org/projects/sugar/repos/mainline/commits/18b089a07f68aa669d49ebdac726e5d71ab9985c

the bad thing with this way is after requery list view loses focus highliting(e.g. select row, after clicking on Favorite icon for this row it losses focus). One of possible decision is change focus color(for shell process not in gtkrc)

so, if its not ok with losing focus, please reopen ticket

  Changed 4 years ago by alsroot

  • keywords r? added; r+ removed
  • status changed from closed to reopened
  • resolution fixed deleted

Changed 4 years ago by alsroot

  Changed 4 years ago by tomeu

  • keywords r+ added; r? removed
  • owner changed from sascha_silbe to alsroot
  • status changed from reopened to assigned

Sounds good, thanks.

  Changed 4 years ago by alsroot

  • status changed from closed to reopened
  • resolution fixed deleted

in reply to: ↑ 11 ; follow-up: ↓ 15   Changed 4 years ago by alsroot

Replying to alsroot:

 http://git.sugarlabs.org/projects/sugar/repos/mainline/commits/12f562c59323a008ed16a306f419ab6f642153e1

This commit leads to flickering while hide/show invoking in development ubuntu and mandriva repos, w/o this commit Journal doesn't flickering like in gentoo(just scrollbars movements).

So, in my mind we have odd consequences of tying to particular gtk version behaviour.

in reply to: ↑ 14   Changed 4 years ago by alsroot

Replying to alsroot:

Replying to alsroot:

 http://git.sugarlabs.org/projects/sugar/repos/mainline/commits/12f562c59323a008ed16a306f419ab6f642153e1


This commit leads to flickering while hide/show invoking in development ubuntu and mandriva repos, w/o this commit Journal doesn't flickering like in gentoo(just scrollbars movements).

So, in my mind we have odd consequences of tying to particular gtk version behaviour.

|TestCase|
* have move then one page of entries in Jorunal
* scroll to top
* scroll to one row down
* chage Favorite icon
* in last ubuntu/mandriva its flickering

  Changed 4 years ago by erikos

  • priority changed from Unspecified by Maintainer to Immediate

Changed 4 years ago by alsroot

follow-up: ↓ 19   Changed 4 years ago by alsroot

  • keywords r? added; r+ removed

Its still not ideal, but it works w/o glitches in all of my environments.

And I guess we need to rewrite listview code in 0.88.

  Changed 4 years ago by alsroot

  • owner changed from alsroot to tomeu
  • status changed from reopened to assigned

in reply to: ↑ 17   Changed 4 years ago by tomeu

  • keywords r+ added; r? removed
  • owner changed from tomeu to alsroot

Replying to alsroot:


And I guess we need to rewrite listview code in 0.88.

Well, mostly the model I guess, so we don't reset it. It may help if the journal accesses directly the xapian index, instead of going through the DS.

  Changed 4 years ago by alsroot

  • status changed from assigned to closed
  • resolution set to fixed
Note: See TracTickets for help on using tickets.