AamAadmiParty / india-standalone-drupal

A drupal based website that currently powers http://aamaadmiparty.org/, the "main" site. It is different from other AAP websites in drupal for primarily large scale email sending capabilities, among many other small things. (This website is a contender for merging with 'regional-domainsite-drupal' reposiroty when the latter is ready)
GNU General Public License v2.0
0 stars 1 forks source link

Replace 'contact us' in the header with links to state sub domain sites. #197

Open sumeetpareek opened 10 years ago

sumeetpareek commented 10 years ago

Replace 'contact us' in the header with links to state sub domain sites.. Do this by creating a new contrib module (unless there is such an existing module).. Create a sub domains switcher for domain access module..

tkuldeep commented 10 years ago

@sumeetpareek Please see header, Is it appropriate..? screenshot from 2014-04-02 18 08 02

tkuldeep commented 10 years ago

On dev site I have deployed it.. http://dev.aamaadmiparty.org/

sumeetpareek commented 10 years ago

@tkuldeep Nice work. Can you get the below feedback items done, before we release it to the PROD site?

screenshot__2014-04-08__001

tkuldeep commented 10 years ago

@sumeetpareek When I use any jquery plugin for dropdown, it converts items of select into list element.. so click event of select element does not work. which I am using for redirecting in js. And in my case list item are not visbie like drop-down.

Instead of using plugin should I change from background orange to white..?

sumeetpareek commented 10 years ago

@tkuldeep You will need to remove your custom JS code and use the onchange provided by the plugin. Do a CRTL+F for onchange on this page and read through the details - http://patrickkunka.github.io/easydropdown/

tkuldeep commented 10 years ago

@sumeetpareek This jquey error is coming. Uncaught TypeError: Object [object Object] has no method 'on'

I have changed css and now looking like this.. timer

tkuldeep commented 10 years ago

@sumeetpareek @ankurshrivastav Deployed to prod...

sumeetpareek commented 10 years ago

The dropdown has a UL element which by default has both margin and a padding. To nullify the padding effect a left margin of -35px is added. This is wrong approach to theme the dropdown and makes the scrollbar hard to use. The right way of doing this should be NOT set the left margin at all and instead set both the maring and the padding of this UL to 0px.

@tkuldeep Please get this done.