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

Required: browser extensions #6

Open
remy opened this issue Feb 10, 2012 · 15 comments
Open

Required: browser extensions #6

remy opened this issue Feb 10, 2012 · 15 comments

Comments

@remy
Copy link
Owner

remy commented Feb 10, 2012

Using the green/red idea for the pass or fail, make a request to

"http://cssprefix.com/check.json?url=" + window.location

And the JSON returned contains:

{ pass: true }

(or false) - which will drive the icon.

Can this be done for all browsers?

@sambenne
Copy link

I am having a go at this using the crossrider site. It is pretty simple to use and creates a extension for Internet Explorer 7+, Firefox 3.5+ and Chrome.

Might have something for you tomorrow.

@nikcorg
Copy link

nikcorg commented Feb 10, 2012

How about a bookmarklet? Could work with a JSONP response.

@sambenne
Copy link

@remy Can you add a JSONP response to this. Should be a simple thing to check whether there is a callback in the url.

@remy
Copy link
Owner Author

remy commented Feb 11, 2012

Yep, easy.

Sent from my iPhone

On 11 Feb 2012, at 11:31, Sam Bennett reply@reply.github.com wrote:

@remy Can you add a JSONP response to this. Should be a simple thing to check whether there is a callback in the url.


Reply to this email directly or view it on GitHub:
#6 (comment)

@sambenne
Copy link

Sweet, that should mean my multi browser extension should be ready when you are.

Actually I change the $.getJSON to $.ajax and it works. So I have it changing the icon from red to green. Just need to work out what to do to show the failed content.

@sambenne
Copy link

Have you just broke this? Every site seems to be passing.

@remy
Copy link
Owner Author

remy commented Feb 11, 2012

I think so. I was working last night on using message queues and workers to get the work done, but I suspect it's gone a little tits up. In fact now, I'm not getting anything.

I've got some scaling issues I need to look at - and probably need to go a knocking on hosting company doors :) I'll see if I can't sort it shortly.

@sambenne
Copy link

Doesn't seem to load the page no more :( If I had a decent host I might run a test version so I can keep working.

@remy
Copy link
Owner Author

remy commented Feb 11, 2012

It's back up for the time being. Given the few hours I've worked on it, there's some exceptions I'm hitting (like invalid urls, etc).

If you're testing - best to test with static responders like:

@remy
Copy link
Owner Author

remy commented Feb 11, 2012

With JSONP support, I'll add CORS which will be better as you'll be able to handle timeouts, etc much more cleanly.

@remy
Copy link
Owner Author

remy commented Feb 11, 2012

@sambenne sorry, I should have added - you can host this locally for testing - clone the project, install Node.js then in the project directory run npm install (I think it says this on the README) - and you'll be off and running.

@sambenne
Copy link

Based on the static I can get the JSON to work and it changes the icon from red to green on page load. I am going to get it to show the errors in a popup on a click event.

@sambenne
Copy link

Here is the first test.
http://crossrider.com/plugin/chrome/apps/css-prefix-test.crx

On the page load it will check the site http://jsbin.com/erigos/3/js but on click it will check http://jsbin.com/erigos/4/js so you can see how it looks both ways. I am going to try it on FF.

@remy
Copy link
Owner Author

remy commented Feb 11, 2012

@sambenne extension looks nice as a starter (only quickly checked it out). I saw your tweet too - though I fear you've wasted a bit of your time! The "check your site" part of things is handled entirely by csslint.net. The error when you're doing the .json call is only the first error (and though I've not updated docs) it will only be the first error to basic reduce the work of the server when doing a single yes/no test. So in a way it's not that useful (and I'd probably not show in the icon at all).

Is there source available for the extension at all? Might be useful to let others have a peek around to learn from it.

Couple of bits of feedback, no doubt you've already considered yourself:

  1. Icon is green before anything is done - might suggest it's all good.
  2. Clicking icon runs test - perfect.
  3. Clicking icon after test is run - perhaps offers to get fixed CSS if it's bad.
  4. Status (red/green) seems to carry cross tabs - it shouldn't do I expect.
  5. Hovering over green icon says "this site has 1 error"

I'm also going to update the readme so it's exactly clear what this tool/service does! :)

:: edit ::

Sorry - just re-read - you said it's hitting the static examples so I could see what it looks like - so it's not hooked up to live - that's cool1

@sambenne
Copy link

Ah that image on twitter was actually using your API that I used for the extension. So its data was based of what gets returned. Was thinking the extension could take you to a page that had a more detailed explanation for you.

When I get on my Laptop I will put the code on my github and link you to it.

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

3 participants