Merge lp:~openerp-dev/openerp-web/7.0-improve-searchbox-cursor-bth-psa into lp:openerp-web/7.0

Proposed by Paramjit Singh Sahota(OpenERP)
Status: Superseded
Proposed branch: lp:~openerp-dev/openerp-web/7.0-improve-searchbox-cursor-bth-psa
Merge into: lp:openerp-web/7.0
Diff against target: 39 lines (+2/-2)
2 files modified
addons/web/static/src/css/base.css (+1/-1)
addons/web/static/src/css/base.sass (+1/-1)
To merge this branch: bzr merge lp:~openerp-dev/openerp-web/7.0-improve-searchbox-cursor-bth-psa
Reviewer Review Type Date Requested Status
Xavier (Open ERP) (community) Needs Fixing
Review via email: mp+154317@code.launchpad.net

This proposal has been superseded by a proposal from 2013-05-17.

Description of the change

Hello,

Fixed the issue,
>In the search box, when you do a advanced filter, the "or" seems clickable (hand cursor) but it is not, it is confusing.
>The "or" should not be clickable.

Thankz YoU.
 -PSA.

To post a comment you must log in.
3855. By bth <bth@bth-laptop>

[MERGE] Merge lp:openerp-web/7.0.

3856. By bth <bth@bth-laptop>

[FIX] Put arrow on or.

3857. By Bhumi Thakkar (Open ERP)

[MERGE] Merge lp:openerp-web/7.0.

Revision history for this message
Xavier (Open ERP) (xmo-deactivatedaccount) wrote :

Change is overly complex, why not just remove `cursor` on `.openerp .oe_searchview .oe_searchview_drawer .oe_searchview_advanced li` and then set `cursor:default` on `.openerp .oe_searchview .oe_searchview_drawer` (to offset `cursor:text` set on `.oe_searchview`)?

review: Needs Fixing
3858. By Paramjit Singh Sahota(OpenERP)

[MERGE] Merged lp:openerp-web/7.0

3859. By Paramjit Singh Sahota(OpenERP)

[IMP] Removed 'cursor on '.openerp .oe_searchview .oe_searchview_drawer .oe_searchview_advanced li' and then added 'cursor:default' on '.openerp .oe_searchview .oe_searchview_drawer'.

3860. By Bhumi Thakkar (Open ERP)

[MERGE]Merge lp:openerp-web/7.0

Unmerged revisions

3860. By Bhumi Thakkar (Open ERP)

[MERGE]Merge lp:openerp-web/7.0

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'addons/web/static/src/css/base.css'
2--- addons/web/static/src/css/base.css 2013-04-04 10:23:06 +0000
3+++ addons/web/static/src/css/base.css 2013-05-17 06:27:27 +0000
4@@ -1812,6 +1812,7 @@
5 display: block;
6 }
7 .openerp .oe_searchview .oe_searchview_drawer {
8+ cursor: default;
9 position: absolute;
10 z-index: 2;
11 margin-top: 4px;
12@@ -1958,7 +1959,6 @@
13 padding: 0;
14 }
15 .openerp .oe_searchview .oe_searchview_drawer .oe_searchview_advanced li {
16- cursor: pointer;
17 position: relative;
18 list-style: none;
19 margin: 0;
20
21=== modified file 'addons/web/static/src/css/base.sass'
22--- addons/web/static/src/css/base.sass 2013-04-04 10:23:06 +0000
23+++ addons/web/static/src/css/base.sass 2013-05-17 06:27:27 +0000
24@@ -1441,6 +1441,7 @@
25 display: block
26
27 .oe_searchview_drawer
28+ cursor: default
29 position: absolute
30 z-index: 2
31 // detach drawer from field slightly
32@@ -1554,7 +1555,6 @@
33 list-style: none
34 padding: 0
35 li
36- cursor: pointer
37 position: relative
38 list-style: none
39 margin: 0