arielsalminen / ResponsiveSlides.js

Simple & lightweight responsive slider plugin (in 1kb)
http://responsiveslides.com
3.46k stars 1.52k forks source link

Slider image changes size during transition #239

Open mickolosmaxius opened 11 years ago

mickolosmaxius commented 11 years ago

Hi all. My first comment here. I'm working on a site for a client and having problems with the slider here: http://michaelmcgillivray.com/clients/cvhelp/index.html

You can find all the javascript files by viewing the source (ctrl + U) The jquery and

The image changes size during transition from one slide to another.

It looks fine without the javascript but obviously does not work so I'm guessing

Any help would be much appreciated!

FYI: The three images used are just photoshoped versions of the first one just for me to test with.

mi-mi-mi commented 11 years ago

you are using

$("#slides1").responsiveSlides(

to call the slider, but you don't have that ID (slides1) on any element on your page you are using

 < ul class="rslides rslides1"> 

so you should be using something like

< ul id="slides1".... 
mickolosmaxius commented 11 years ago

Thanks your response. OK did that but now it's completely messed up??? http://michaelmcgillivray.com/clients/cvhelp/index.html#

-M

On 10 September 2013 15:11, mi-mi-mi notifications@github.com wrote:

you are using

to call the slider, but you don't have that ID (slides1) on any element on your page you are using <ul class="rslides rslides1">

so you should be using something like <ul id="slides1"

— Reply to this email directly or view it on GitHubhttps://github.com/viljamis/ResponsiveSlides.js/issues/239#issuecomment-24162572 .

mi-mi-mi commented 11 years ago

You are missing the basic CSS styling for it. as per the instructions on the main page http://responsiveslides.com/

you have to Usage: 1) link the js files: jquery and responsive slides (which you did) 2) add the markup, ul class, li, img (which you did) 3) Add CSS (you missed this) 4) Hook up the slideshow (you corrected the ID on this) 5) Options to customize (you did this)

also right now you are hooking up 2 slideshows, don't know if you want that or not.

mickolosmaxius commented 11 years ago

Thanks again for your timely response. It is very much appreciated.

The CSS is there it's at the bottom of the style.css page This is what is says. ( .rslides { position: relative; list-style: none; overflow: hidden; width: 100%; padding: 0; margin: 0 auto 40px; }

.rslides2 { position: relative; list-style: none; overflow: hidden; width: 100%; padding: 0; box-shadow: none; -moz-box-shadow: none; -webkit-box-shadow: none; margin: 0 auto; } .rslides img {width:100%;} .rslides_tabs li:first-child { margin-left: 0; } )

Is there other css I need?

-M

On 10 September 2013 23:05, mi-mi-mi notifications@github.com wrote:

You are missing the basic CSS styling for it. as per the instructions on the main page http://responsiveslides.com/

you have to Usage: 1) link the js files: jquery and responsive slides (which you did) 2) add the markup, ul class, li, img (which you did) 3) Add CSS (you missed this) 4) Hook up the slideshow (you corrected the ID on this) 5) Options to customize (you did this)

also right now you are hooking up 2 slideshows, don't know if you want that or not.

— Reply to this email directly or view it on GitHubhttps://github.com/viljamis/ResponsiveSlides.js/issues/239#issuecomment-24198662 .

mickolosmaxius commented 9 years ago
a,
a:visited {
    color: #D26000;
    text-decoration: none;
}

         

                Join me on PeoplePerHour

                I have been using PeoplePerHour.com a marketplace connecting curated freelance talent with remote work, anything from 1 hour upwards.

                ACCEPT INVITATION

                                                    © 2007 - 2014 PeoplePerHour All rights reserved                                                    

                                                    This is an important notification helping you quickly get the most out of PeoplePerHour.  You can change your notification settings here
mickolosmaxius commented 9 years ago
a,
a:visited {
    color: #D26000;
    text-decoration: none;
}

      News about Michael and other latest on PeoplePerHour

Hello, Michael has been active on PeoplePerHour for 6 days now and they've invited you to join! Accept the invitation today and get 10% off your first purchase on PeoplePerHour... it's great to have friends with benefits, right?

                Join now

Here's some of the latest activity on PeoplePerHour that you'll find interesting

                Write an SEO article / blog post in 400 words in 1 day

                        Katie S.

                                    €10

                Do any kind of data entry work for 5 hrs (data research / ex...

                        Caroline F.

                                    €9

                Design infographics using almost any statistics you provide

                        Dave F.

                                    €14

                Create a Video Animation with your Logo

                        LogoDreamS â.

                                    €7

                Design 7 professional Logo drafts with unlimited revisions

                        Kamran K.

                                    €8

                Design 3 conceptual and Unique Logo with unlimited revisions...

                        Muhammad H.

                                    €17

            Kind Regards,
            The PPH Team

                                                    © 2007 - 2014 PeoplePerHour All rights reserved                                                    

                                                    This is an important notification helping you quickly get the most out of PeoplePerHour.  You can change your notification settings here