Setting editable should not recreate the treeview in DictionaryGrid

Bug #812266 reported by Jo-Erlend Schinstad on 2011-07-18
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Quickly Widgets
Undecided
Unassigned
quickly-widgets (Ubuntu)
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

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) on 2012-03-13
Changed in quidgets:
milestone: none → 12.03
status: Fix Released → Fix Committed
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) on 2012-03-13
Changed in quidgets:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers