Doh! refresh_left() should do the same. Attempt 2


Jasper Groenewegen wrote:
I got annoyed by the unsorted bookmarks list so.. voila!

Jasper

# Bazaar merge directive format 2 (Bazaar 0.90)
# revision_id: [EMAIL PROTECTED]
# target_branch: file:///home/jasper/Programming/bzr-gtk/trunk/
# testament_sha1: 1e6701fe010352d27d6ae37e305938bbe7c13cd5
# timestamp: 2008-07-10 13:57:31 +0200
# base_revision_id: [EMAIL PROTECTED]
# 
# Begin patch
=== modified file 'olive/__init__.py'
--- olive/__init__.py	2008-07-01 21:56:25 +0000
+++ olive/__init__.py	2008-07-10 11:57:04 +0000
@@ -945,9 +945,12 @@
         
         # Add them to the TreeStore
         titer = treestore.append(None, [_i18n('Bookmarks'), None])
-        for item in bookmarks:
-            title = self.pref.get_bookmark_title(item)
-            treestore.append(titer, [title, item])
+
+        # Get titles and sort by title
+        bookmarks = [[self.pref.get_bookmark_title(item), item] for item in bookmarks]
+        bookmarks.sort()
+        for title_item in bookmarks:
+            treestore.append(titer, title_item)
         
         # Create the column and add it to the TreeView
         self.treeview_left.set_model(treestore)
@@ -1243,10 +1246,13 @@
 
         # Add them to the TreeStore
         titer = treestore.append(None, [_i18n('Bookmarks'), None])
-        for item in bookmarks:
-            title = self.pref.get_bookmark_title(item)
-            treestore.append(titer, [title, item])
 
+        # Get titles and sort by title
+        bookmarks = [[self.pref.get_bookmark_title(item), item] for item in bookmarks]
+        bookmarks.sort()
+        for title_item in bookmarks:
+            treestore.append(titer, title_item)
+        
         # Add the TreeStore to the TreeView
         self.treeview_left.set_model(treestore)
 

# Begin bundle
IyBCYXphYXIgcmV2aXNpb24gYnVuZGxlIHY0CiMKQlpoOTFBWSZTWRo/AnoAAxxfgARQWWf//3KQ
iAq////wUAX+AAAAAAAA5gEwAmAAEwABMAA5gEwAmAAEwABMAA5gEwAmAAEwABMAA5gEwAmAAEwA
BMAAKlIAmgaBGhpMmKnqfomFNoaZTTNSuRv47eOfavo1/v9nH8Up2Ldmr5PVY6Mk2FEvUduWy+li
0zpIyzlSmPSyvXhvgXyPsw5HyXKKUKUs+pp4Wr1KUbddzStZ1Lo/Dsfhx9H62tWwoqqc3N2/bi/x
j+CtqeTTWuWVKUfxQpeqFksXLnX05+Hq7VbFSVdlMLv4wqr539USyyxuWSHk/DxXKdbg5Pc31PRy
eBwPuxcj7msNa1xR8U8nM5V2aHRih9yp908nUduvyz0VVVfRHv1M8YmfHcr9hJiyTkGg4GJ6rMrW
SiZJ2kalasioxL8VSjBFq9ixetiW6xfRrtNIW99l5HovjDy2XbM2B9T5lR2lqL16VJvRW2M1rX7/
p4118leRc2tDWuzVGWrUrROSzYnFxZ7pTduuXtaYFb3lMNDhvUsksZv8TdMldVpw4LV6pasYq1zQ
rb61MnBvYtrByeCx0daMjYmyoXskV4rEVKOFatz1+MpWqmB4IVq09UbdnYWmVDBYq7KRZLHJXxbV
W1LVErdTF1tLcxbEbl6uzFGFrellhRRYXN9JWxsanVFax5md+lsqw0mgWKkN5m62G1jSNac2DzaX
9OOpu3XkZtDVUuopqb7VcvjwORlMTgw0+SMaqNRum5ngub213MZuVrlTuctM32SrNlSUM0r0GqS3
ocdHzNKhPoob3W2TAvNyjFRTzopRbSdr6nW1K16ovUVK2B3oa7NcolJ/FNhetqn/WL9HwTNL2xRU
ya0qWpXI8bGFVu1SejzLB3ujvdjudH5uc0PF0z7lGZbJ5+CHZYcTguWPVlkTsifhSR6a2aj0dbBc
eTe83W6Po20Ufoo/2h4tiuekPc72ltXOj2LFrW2NxHg/vo8LKPmn1e5vKfGUVSnBqRjtocNjmrit
LVT56kdHo72DOeJ1JhwPydyfl6Knev55u9VXSlb5u51NGPF6vM+hdxPVelknw1Njbsmcl+58H5Mk
ZOp1MG9KIrWu3fU3mi0aXcYYlycH9tSu0OTJK1xi9vl0dr/TtTe2n7Pa82yk96kbjY1PNN52m2Ra
+T1P6bng5O6qg089yU9DuNjg+KLTrOaVHKPpJouT7dita1hpDJL1ii9Uxid7OYKd+c2m6ZIqZSe2
JVIrQvQwTFOCOwvQ0u9RRSihoo7H+1r/Okh71pV3NatHOJ7m6hQuFp8Xa2Ouje5JsYOo+KNa+LT2
JQmJe/yi5dJaqVf8UStHkhYK6Ionj3fA+XCYRNqHgowD9GlHxiXwyjvhrKo/OtPlamqjqcJzc3NL
C9gwFXJ36mLEovK0/eTNKTYeqc2llSVHJF6pOCFklFG55NrfiTLa9VjNRK2JljmYxLIy+a3SLJLy
lCie1UpPuhQqXlFSrE/J/LndQ0HQKL089rm+7NifUomiSinzWTwnXarlZTxfq0qzYlz5uJwUWEzZ
mC/lpLD6KOCkmEjRDJOj8qowolqo8fA3sLZclp+prD+VZ9St7DrPcjJPYhamtyZzBK37IsVP2R2Z
zBuf/F3JFOFCQGj8Ceg=
-- 
bzr-gtk mailing list
[email protected]
Modify settings or unsubscribe at: 
https://lists.canonical.com/mailman/listinfo/bzr-gtk

Reply via email to