Remove jquery #215

Merged
merged 15 commits into from Jan 20, 2016

Conversation

3 participants
@dignifiedquire
Contributor

dignifiedquire commented Jan 20, 2016

No description provided.

Travis Person and others added some commits Jan 3, 2016

Travis Person
Remove jquery from views/page.js
Removes the use of jquery to get the value of the dag path input box.

Addresses part of #133
Travis Person
Remove jquery from views/filelist.js
Removes the use of jquery to get the files hash from the elements
attrs. Instead we will create a callback for the click that will handle
each files unpin.

Also cleaned up things a bit, more will be required.

Addresses part of #133
Travis Person
Editable is no longer used
Removed editable as it's not used anywhere in the code.

Addresses part of #133
Travis Person
Remove jquery from views/config.js
Removed the use of jquery to dynamically set the height of the config
textarea.

Addresses part of #133
Travis Person
Remove jquery from pages/object.js
Use regular old javascript to retrieve the value from the event and
trim off the extra.

Addresses part of #133
Travis Person
Remove jquery from pages/files.js
Removed the hover class addition/removal as it appeared not to be doing
anything, also used good-ol JavaScript to perform click operation.

Addresses part of #133
Travis Person
Move FileItem component to its own file
Moved the FileItem to a seperate file.
Switched to using Glyphicon instead of a div for block element.
Travis Person
Move map out to separate function
Changes location of key to remove error.

@jbenet jbenet added the in progress label Jan 20, 2016

dignifiedquire added a commit that referenced this pull request Jan 20, 2016

@dignifiedquire dignifiedquire merged commit d6e71e1 into master Jan 20, 2016

1 of 3 checks passed

continuous-integration/travis-ci/pr The Travis CI build is in progress
Details
continuous-integration/travis-ci/push The Travis CI build is in progress
Details
ci/circleci Your tests passed on CircleCI!
Details

@dignifiedquire dignifiedquire deleted the remove-jquery branch Jan 20, 2016

@jbenet jbenet removed the in progress label Jan 20, 2016

@RichardLitt

This comment has been minimized.

Show comment
Hide comment
@RichardLitt

RichardLitt Jan 20, 2016

Contributor

Woo!

Contributor

RichardLitt commented Jan 20, 2016

Woo!

@dignifiedquire dignifiedquire referenced this pull request in ipfs/pm Jan 25, 2016

Closed

Sprint January 19th #83

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment