Merge lp:~openerp-dev/openerp-web/7.0-improve-searchbox-cursor-bth into lp:openerp-web/7.0
Proposed by
Bhumi Thakkar (Open ERP)
Status: | Rejected |
---|---|
Rejected by: | Xavier (Open ERP) |
Proposed branch: | lp:~openerp-dev/openerp-web/7.0-improve-searchbox-cursor-bth |
Merge into: | lp:openerp-web/7.0 |
Diff against target: |
29 lines (+8/-0) 2 files modified
addons/web/static/src/css/base.css (+4/-0) addons/web/static/src/css/base.sass (+4/-0) |
To merge this branch: | bzr merge lp:~openerp-dev/openerp-web/7.0-improve-searchbox-cursor-bth |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Xavier (Open ERP) (community) | Needs Fixing | ||
Review via email: mp+155940@code.launchpad.net |
Description of the change
Hello,
In the search box, when you do a advanced filter, the or seems clickable (hand cursor) but it is not, it is confusing i would change into "and" when clicking on it
The 'or' should not be clickable.
Thanks.
To post a comment you must log in.
Unmerged revisions
- 3868. By Bhumi Thakkar (Open ERP)
-
[IMP] Set cursor on or in advance search.
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`)?