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

Add arrow color variables to menu components #7515

Merged
merged 4 commits into from Dec 17, 2015
Merged

Add arrow color variables to menu components #7515

merged 4 commits into from Dec 17, 2015

Conversation

brettsmason
Copy link
Contributor

Adds arrow color variables to dropdown menu, drilldown and accordion menu. I recently tried to change the arrow colors in a recent project but found I wasn't able to, hence this!

Adds accordion menu arrow color variable.
I noticed I couldn't easily change this when working on a new project.
Added dropdown menu anchor color variable so this can be customised
Add drilldown arrow color variable so this can be changed.
@zurbchris
Copy link
Contributor

I like this idea @gakimball what do you think?

@gakimball
Copy link
Contributor

I figured this would be requested eventually, thanks for putting it together!

Can you resolve the merge conflicts?

@brettsmason
Copy link
Contributor Author

@gakimball Any pointers on how to do that? Sorry I'm pretty new to using Git and haven't worked out how to do that yet...

@gakimball
Copy link
Contributor

In the folder of your Foundation fork, run this:

# Adds a remote reference to the official repo
git remote add foundation https://github.com/zurb/foundation-sites.git

# Downloads references to branches
git fetch foundation

# Merge the "develop" branch of Foundation into your work
git merge foundation/develop

I think that works.

# Conflicts:
#	scss/components/_dropdown-menu.scss
@brettsmason
Copy link
Contributor Author

thanks @gakimball I hope I've done that correctly! This and 7514 will be a big help in my current project 😃

@gakimball
Copy link
Contributor

@brettdewoody Yep! Now I can merge it automatically, thanks :)

gakimball added a commit that referenced this pull request Dec 17, 2015
Add arrow color variables to menu components
@gakimball gakimball merged commit e13aac3 into foundation:develop Dec 17, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants