Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make the arrows not appear if first or last item #37

Closed
jase7777 opened this issue Sep 12, 2011 · 11 comments
Closed

Make the arrows not appear if first or last item #37

jase7777 opened this issue Sep 12, 2011 · 11 comments

Comments

@jase7777
Copy link

It's a little misleading if there are arrows on the first or last item. People click but there are no more to display anyway.
Any way to hide arrows if there are no more items to cycle?

@Mottie
Copy link
Contributor

Mottie commented Sep 12, 2011

Good point... I just updated the plugin to version 2.1.4 with a new disabled option which contains the css class added to the arrow. Please note that this class is only added when the wrap option is false, when it is true, the arrows will rewind/fast forward the slides.

@Mottie Mottie closed this as completed Sep 12, 2011
@jase7777
Copy link
Author

Oh cool thanks. The site I'm working on uses dynamic content to fill moving boxes...sometimes there is only one item though so looping or fast forwarding / rewinding still gives the illusion that it's not doing anything. If the arrows weren't there at all on these items it would be better, but since this was not how it was intended to be used I can fully understand if you choose not to make it so.

Anyway I appreciate the work you have done and very fast reply. Thank you.

@Mottie
Copy link
Contributor

Mottie commented Sep 12, 2011

If you want to completely hide the arrows, just add this css:

a.mb-scrollButtons {
  display: none;
}

Hmmm, it should hide both arrow buttons if there is only one panel. Is there something else you needed changed to make this better work for you?

@jase7777
Copy link
Author

Oh I didn't even need to add that CSS. I updated to the latest version and it all looks and works beautifully.

Thank you so much!

@tengallen
Copy link

how to use arrow instead of pointer for left and right panel?

I still don't know how to modify css....

@Mottie
Copy link
Contributor

Mottie commented Nov 29, 2011

@tengallen Look in the movingboxes.css file for these lines:

/*** Slider panel ***/
.mb-slider .mb-panel {
width: 350px; /* default, this is overridden by script settings */
margin: 5px 0;
padding: 5px;
display: block;
cursor: pointer;
float: left;
list-style: none;
}

/* Cursor to arrow over current panel, pointer for all others,
change .current class name using plugin option, currentPanel : 'current' */
.mb-slider .mb-panel.current {
cursor: auto;
}

It is opposite of what you think. The top part makes ALL panels have a cursor:pointer set, but the second panel used only on the current or middle panel has the cursor set to auto to make the cursor back into an arrow. If you only want arrows, then just change cursor:pointer to cursor:auto or completely remove it from the css.

@tengallen
Copy link

got it and thanks for your promptly reply...

If I only have 3 images and all of them are shown in slider panel completed. No other images to slide anymore...
I could hide scroll button and buildNav but don't know how to make right / left image unclickable.

thanks for reply

@Mottie
Copy link
Contributor

Mottie commented Nov 29, 2011

I guess I'm confused... if you have all slides showing and don't want the scrolling to work, then why even use the plugin?

@tengallen
Copy link

never mind.
is it possible to make right / left image unclickable in slider panel?? I wish to slide images by scrollButtons only.

@Mottie
Copy link
Contributor

Mottie commented Nov 29, 2011

The easiest method, without modifying the plugin, would be to add this line of code in the initialized callback (demo)

$('#slider').movingBoxes({

    initialized: function(e, slider, tar){
        slider.$el.undelegate('.mb-panel', 'click')
    }

});

@tengallen
Copy link

awesome!!!! many thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants