Merge lp:~therp-nl/web-addons/6.1-web_print_from_browser-hide_kanban_controls into lp:~webaddons-core-editors/web-addons/6.1

Proposed by Holger Brunn (Therp) on 2014-01-30
Status: Rejected
Rejected by: Holger Brunn (Therp) on 2014-07-07
Proposed branch: lp:~therp-nl/web-addons/6.1-web_print_from_browser-hide_kanban_controls
Merge into: lp:~webaddons-core-editors/web-addons/6.1
Diff against target: 12 lines (+2/-0)
1 file modified
web_print_from_browser/static/src/css/web_print_from_browser.css (+2/-0)
To merge this branch: bzr merge lp:~therp-nl/web-addons/6.1-web_print_from_browser-hide_kanban_controls
Reviewer Review Type Date Requested Status
Holger Brunn (Therp) Resubmit on 2014-07-07
Yannick Vaucher @ Camptocamp 2014-01-30 Approve on 2014-03-17
Review via email: mp+203916@code.launchpad.net
To post a comment you must log in.
Holger Brunn (Therp) (hbrunn) wrote :

...fixed the branch to only contain the commit it should contain

LGTM

review: Approve
Holger Brunn (Therp) (hbrunn) wrote :

This project is now hosted on https://github.com/OCA/web. Please move your proposal there. This guide may help you https://github.com/OCA/maintainers-tools/wiki/How-to-move-a-Merge-Proposal-to-GitHub

review: Resubmit

Unmerged revisions

24. By Holger Brunn (Therp) on 2014-01-30

[ADD] hide controls when printing a kanban view

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'web_print_from_browser/static/src/css/web_print_from_browser.css'
2--- web_print_from_browser/static/src/css/web_print_from_browser.css 2013-06-10 12:52:02 +0000
3+++ web_print_from_browser/static/src/css/web_print_from_browser.css 2014-01-30 10:56:30 +0000
4@@ -14,6 +14,8 @@
5 .oe_debug_view,
6 .oe_searchable_view,
7 .oe-process-view,
8+ .oe_kanban_show_more,
9+ .oe_kanban_header,
10 #oe_secondary_menu,
11 table.main_table > tbody > tr:first-child,
12 table.main_table > tbody > tr:last-child,

Subscribers

People subscribed via source and target branches