Setting editable should not recreate the treeview in DictionaryGrid

Bug #812266 reported by Jo-Erlend Schinstad
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Quickly Widgets
Fix Released
Undecided
Unassigned
quickly-widgets (Ubuntu)
Fix Released
Undecided
Unassigned

Bug Description

When you set mydictgrid.editable = True, then the widget is refreshed. This has several negative side effects:

1) if you've scrolled to a specific row, then that is forgotten, which is a bad thing. Consider an application with lots of rows that are normally not edited. You have a toggle button on the toolbar to make the treeview editable when that's necessary. So you've scrolled to a row and want to edit it. After making the table editable, you'll have to scroll back to the row in order to edit it. Then you untoggle editable, and then selection and scroll position is again forgotten.

2) If you've set column titles, then those are forgotten if you toggle editable.

I think a much better solution would be to make the editable setter loop through the columns, setting the editable property for each. That wouldn't disrupt anything. It's a little more work though.

Related branches

Revision history for this message
Jo-Erlend Schinstad (joerlend.schinstad-deactivatedaccount) wrote :

This will be easily fixed once grid_column gets an editable property: https://bugs.launchpad.net/quidgets/+bug/812293

Changed in quidgets:
status: New → Fix Released
Michael Terry (mterry)
Changed in quidgets:
milestone: none → 12.03
status: Fix Released → Fix Committed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package quickly-widgets - 12.03

---------------
quickly-widgets (12.03) precise; urgency=low

  * Setting editable should not recreate the treeview in DictionaryGrid
    (LP: #812266)
  * DictionaryGrid.editable does not work for CheckColumn (LP: #841357)
  * Quickly Widgets should use GOI (LP: #678225)
  * debian/control:
    - Update Dependencies for python-gi
 -- Michael Terry <email address hidden> Tue, 13 Mar 2012 12:54:42 -0400

Changed in quickly-widgets (Ubuntu):
status: New → Fix Released
Michael Terry (mterry)
Changed in quidgets:
status: Fix Committed → Fix Released
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.