Skip to content

c0de's rewrite#1

Open
c0de-fox wants to merge 12 commits intoaviginsberg:masterfrom
c0de-archive:master
Open

c0de's rewrite#1
c0de-fox wants to merge 12 commits intoaviginsberg:masterfrom
c0de-archive:master

Conversation

@c0de-fox
Copy link
Copy Markdown

I rewrote a large portion of the code and showed it to avi. He seems to like it, but hasn't dissected it yet.
I'm making this pull request just because fox :3

c0de-fox added 12 commits March 13, 2015 20:15
Signed-off-by: David Todd <c0de@unps.us>
Signed-off-by: David Todd <c0de@unps.us>
This makes it look a little bit nicer instead of rendering everything into a bland popup

Signed-off-by: David Todd <c0de@unps.us>
Signed-off-by: David Todd <c0de@unps.us>
This javascript file uses chrome's API to add one or two context menu entries.
Most of the time, the shortenFile will not be called, but it is available

Signed-off-by: David Todd <c0de@unps.us>
This contained jQuery, popup.js, browser_action.html
We don't need a large library like jQuery when native functions work
popups are a thing of the past :3
browser_action was just popup.html renamed, not needed either

Signed-off-by: David Todd <c0de@unps.us>
For some reason chrome.tabs.get is failing with an undefined method error,
this should fix the permissions... hopefully

Signed-off-by: David Todd <c0de@unps.us>
This is the main code of the extension. Most of the main functions are in here.
This should also be close to the final result of what the code will look like

Signed-off-by: David Todd <c0de@unps.us>
Define the API method as GET
JSON.parse is now parsing the response child of the request instead of the whole request object - This kept failing the script after getting response from server
Add a fail scenario when a link fails testURL
encode URIs to prevent "interesting" server-side URI mangling

Signed-off-by: David Todd <c0de@unps.us>
Builds/ and Website/ are lame :3

Signed-off-by: David Todd <c0de@unps.us>
Avi can add them again using the power of git :3

Signed-off-by: David Todd <c0de@unps.us>
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.

1 participant