Merge lp:~openerp-dev/openobject-client-web/calender_view into lp:openobject-client-web/trunk

Proposed by Chirag Patel (OpenERP)
Status: Merged
Merged at revision: 4121
Proposed branch: lp:~openerp-dev/openobject-client-web/calender_view
Merge into: lp:openobject-client-web/trunk
Diff against target: 20 lines (+2/-2)
1 file modified
addons/view_calendar/static/css/calendar.css (+2/-2)
To merge this branch: bzr merge lp:~openerp-dev/openobject-client-web/calender_view
Reviewer Review Type Date Requested Status
Xavier (Open ERP) (community) Needs Fixing
Review via email: mp+43745@code.launchpad.net

Description of the change

Hello
Please refer this bug:687706

Thanks.

To post a comment you must log in.
Revision history for this message
Xavier (Open ERP) (xmo-deactivatedaccount) wrote :

Wrapping everything in <pre> is not correct: it changes the font of the title.

You should replace that by fixing the styles of the relevant classes and set the `white-space` CSS property to `pre-wrap` and/or `pre-line` (more sensible behavior but lower compatibility).

review: Needs Fixing
4109. By Chirag Patel (OpenERP)

[FIX] Fixed wrap text using pre-wrap in css.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'addons/view_calendar/static/css/calendar.css'
2--- addons/view_calendar/static/css/calendar.css 2010-07-07 14:15:02 +0000
3+++ addons/view_calendar/static/css/calendar.css 2010-12-16 07:29:09 +0000
4@@ -118,7 +118,7 @@
5 padding: 2px;
6 font-weight: 600;
7 line-height: 1em;
8- white-space: nowrap;
9+ white-space: pre-wrap;
10 overflow: hidden;
11 cursor: pointer;
12 border-radius: 5px;
13@@ -144,7 +144,7 @@
14 color: #000;
15 font-weight: bold;
16 padding: 2px;
17- white-space: nowrap;
18+ white-space: pre-wrap;
19 cursor: move;
20 }
21