Merge lp://qastaging/~midori/midori/headerBarSupport into lp://qastaging/midori
Proposed by
Cris Dywan
Status: | Merged | ||||||||
---|---|---|---|---|---|---|---|---|---|
Approved by: | Paweł Forysiuk | ||||||||
Approved revision: | 6968 | ||||||||
Merged at revision: | 6961 | ||||||||
Proposed branch: | lp://qastaging/~midori/midori/headerBarSupport | ||||||||
Merge into: | lp://qastaging/midori | ||||||||
Diff against target: |
1091 lines (+370/-376) 12 files modified
extensions/adblock/extension.vala (+2/-5) extensions/adblock/widgets.vala (+38/-72) extensions/statusbar-features.c (+4/-7) midori/midori-browser.c (+58/-289) midori/midori-browser.h (+1/-1) midori/midori-contextaction.vala (+12/-0) midori/midori-frontend.c (+3/-1) midori/midori-panedaction.vala (+4/-0) midori/midori-preferences.c (+7/-0) midori/midori-window.vala (+239/-0) midori/midori.vapi (+1/-1) po/POTFILES.in (+1/-0) |
||||||||
To merge this branch: | bzr merge lp://qastaging/~midori/midori/headerBarSupport | ||||||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Paweł Forysiuk | Approve | ||
Danielle Foré (community) | ux, testing | Approve | |
Review via email:
|
Commit message
Implement Midori.Window class with toolbar/ headerbar
To post a comment you must log in.
Packing the urlbar into the title are looks weird and means autocompletion results just aren't legible. I think it would make more sense to just pack it left and expand like you'd do in a regular toolbar.
Since the "Toolbar Editor" extension exposes itself through right clicking on the toolbar, there's now no way to use this extension. It should probably get a little settings icon like other extensions with settings have