Merge lp:~danrabbit/midori/fix-1172589 into lp:midori
Proposed by
Danielle Foré
Status: | Superseded | ||||||||
---|---|---|---|---|---|---|---|---|---|
Proposed branch: | lp:~danrabbit/midori/fix-1172589 | ||||||||
Merge into: | lp:midori | ||||||||
Diff against target: |
166 lines (+38/-37) 1 file modified
midori/midori-browser.c (+38/-37) |
||||||||
To merge this branch: | bzr merge lp:~danrabbit/midori/fix-1172589 | ||||||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Danielle Foré (community) | Needs Resubmitting | ||
Paweł Forysiuk | Needs Fixing | ||
Review via email: mp+164652@code.launchpad.net |
This proposal has been superseded by a proposal from 2013-05-20.
Commit message
clean up bookmarks dialog
Description of the change
This branch removes the unecessary icon, adds a "," in the label, organized things into a neatly spaced vbox (instead of packing directly into the dialog), Changes the label for Folders, and generally un-fucks the Add Bookmark dialog.
To post a comment you must log in.
Please clean up this patch before proposing to merge.
You should not change code style (adding brackets, changing where they appear) unless it really helps readability. Your style changes makes this patch way bigger than it needs to be and makes it hard to see real changes. Patches should be as small as possible generally. Also there is a typo in line 157 - "Boomarks".