search

thecity / thecity-login-wordpress

A WordPress plugin that allows users to login direclty to The City from your WordPress website.
1 stars 0 forks source link

Added style based on The City's Developer Appkit #1

Closed ghost closed 11 years ago

ghost commented 11 years ago

screen shot 2013-07-05 at 5 41 18 pm

ghost commented 11 years ago

Hi @claykitchens !

If you're doing a code replace, I would make sure you replace the code for thecity_login_load.js, thecity_login.js, and replace the css in thecity_login.css with the css I've provided in thecity_appkit.css. It is most likely the css file that is giving you trouble, as the original css was limited in styling and the css I've included has been pulled directly from The City's Appkit page.

If you're merging in from a sha, I would just ask to make sure you're using the most recent one (9dfab21...).

As for the checkbox (I'm presuming that you are referring to the "Keep me logged in" box), I have no idea. As far as I can tell, even before I made the style updates, the checkbox wasn't working. That said, if some of my code has broken it, I'd love to take it out to restore functionality.

I hope this helps.

claykitchens commented 11 years ago

Thank you for the quick reply, @markjflores . I'm new to GitHub and I understood a little of what you said. I tried to download your .php files along with the .js and .css files. As soon as I uploaded them and tried to view the page, with the login widget, it displays references to your GitHub repository and not the page on our website. I'm sorry for my confusion, but I do appreciate your assistance!

ghost commented 11 years ago

@claykitchens NP. Keep in mind my suggestions are also not official; The City team has yet to approve my pull request.

weshays commented 11 years ago

I merged this branch in and then refactored class to extract out design additions. It still needs some work but this is worthy of a 0.3.