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

CSS color-box conflict with SmartAdmin Jarvis Color Picker... and resolution #76

Open
RonLaughton opened this issue Aug 4, 2015 · 0 comments

Comments

@RonLaughton
Copy link

Hi Lauren,

I love what you have done.

In an odd set of coincidences, there was a CSS conflict with the "color-box" when using the SmartAdmin template with the Jarvis Color Picker.

So, I recoded my local copy of Pick-A-Color to use "pick-a-color-box".

The steps are below. Let me know if you would like my updated source (based on version 1.2.3).

Kind regards,

Ron

The updates are as follows:

CSS File:

Renamed FROM:

.color-

TO:

.pick-a-color-

FROM:

.pick-a-color-markup .advanced-content .pick-a-color-preview .pick-a-color-select.btn.advanced{margin-top:15px;display:none}

TO:

.pick-a-color-markup .advanced-content .pick-a-color-preview .pick-a-color-select.btn.advanced{margin-top:15px;}

JS File Due to class conflict with SmartAdmin Jarvis color-box:

CHANGED FROM (By adding two "-" in the name, needed to increase the split array position from 2 to 4):

var p=k.attr("class").split("-")[2]

TO:

var p=k.attr("class").split("-")[4]

Renamed FROM:

.color-

TO:

.pick-a-color-

Renamed FROM (note the leading space):

color-

TO:

pick-a-color-

FROM

"color-

TO

"pick-a-color-

FROM

'pick-a-color-

TO

'color-

FROM

color-preview

TO

pick-a-color-preview

THEN UNDO FROM:

pick-a-color-stop

TO
color-stop

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

No branches or pull requests

1 participant