LimitProxyModel doesn't properly handle insertions

Bug #1213959 reported by Michal Hruby
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
unity8 (Ubuntu)
Fix Released
High
Albert Astals Cid

Bug Description

When doing insertions (for example insert to pos #0, when there are already 10 rows), the LimitProxyModel doesn't properly remove the rows that are pushed beyond the limit, resulting in incorrect behavior.

(similar issue when removing result from the middle which should push currently invisible rows to visible positions)

To reproduce, try searching for "f" in the applications lens, if the "Dash plugins" category is collapsed you'll see only one result, but if it's expanded before the search it'll show 5 results.

Related branches

Michal Hruby (mhr3)
Changed in unity8:
assignee: nobody → Albert Astals Cid (aacid)
status: New → Confirmed
importance: Undecided → High
description: updated
Changed in unity8:
status: Confirmed → In Progress
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

Fix committed into lp:unity8 at revision 230, scheduled for release in unity8, milestone Unknown

Changed in unity8:
status: In Progress → Fix Committed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package unity8 - 7.81.3+13.10.20130821.1-0ubuntu1

---------------
unity8 (7.81.3+13.10.20130821.1-0ubuntu1) saucy; urgency=low

  [ Michael Zanetti ]
  * allow testing at 11:13. Old code failed because the text actually
    says "11:13 AM".
  * add some checks if we actually clicked an item or in the spacing
    between them gets rid of some warnings printed by the launcher .

  [ Michał Sawicz ]
  * Update runtime deps in the build script.

  [ Nick Dedekind ]
  * Added/Updated legacy network indicator components to use with new
    indicator backend.

  [ Albert Astals ]
  * Fix insertions/removals on the qlimitproxymodel . (LP: #1213959)

  [ Ubuntu daily release ]
  * Automatic snapshot from revision 230
 -- Ubuntu daily release <email address hidden> Wed, 21 Aug 2013 15:04:59 +0000

Changed in unity8 (Ubuntu):
status: New → Fix Released
Changed in unity8:
status: Fix Committed → Fix Released
Michał Sawicz (saviq)
Changed in unity8 (Ubuntu):
assignee: nobody → Albert Astals Cid (aacid)
importance: Undecided → High
no longer affects: unity8
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

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