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

Remove #fff from Job Type #2

Closed
wants to merge 5 commits into from
Closed

Remove #fff from Job Type #2

wants to merge 5 commits into from

Conversation

BronsonQuick
Copy link

Hey Mike,

Firstly, thanks so much for making this plugin. It's freaking awesome :) I'm testing it out at the moment for a community site we have.

I noticed that the job type on the preview listing was white so you couldn't see it:

job type white

I've also altered the padding on .job_listing_preview as well so there is 1em around all sides as seen below:

fix padding

Hope that's helpful! Now time to get back to MOAR testing 😀

@BronsonQuick
Copy link
Author

I just noticed that there was an issue with the Job Type in the backend as seen below so I've fixed that up too.

job type backend

@mikejolley
Copy link
Contributor

@BronsonQuick thats odd, those job types should have a background color. http://cl.ly/image/2W3M0s3U3G1X

Ideas why that rule isn't matching up?

@BronsonQuick
Copy link
Author

Hey Mike,

I just had a look then and I realised that the rule isn't being applied because I have custom job types. e.g. Contract and Freelance. If you made orange the default color for custom job types in the CSS then this will fix it. I've just made a new branch that fixes this so i'll close this issue and open up another Pull Request for your consideration :)

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

Successfully merging this pull request may close these issues.

2 participants