Skip to content
This repository has been archived by the owner on May 13, 2022. It is now read-only.

Changed appearance and sortablility of table and website #115

Merged
merged 2 commits into from
Jul 4, 2015

Conversation

btcspry
Copy link
Contributor

@btcspry btcspry commented Jun 30, 2015

  • Converted whole site to bootstrap
  • Added "GitHub" and "Getting Started" links in the menubar
  • Included "sortable.js" for sorting the table, client side
    - Removed server-side table sorting

I hope you accept this pull request because it adds a user front-end to view the inner workings of this project, and it's way more presentable.

- Converted whole site to bootstrap
- Added "GitHub" and "Getting Started" links in the menubar
- Included "sortable.js" for sorting the table, client side
        - Removed server-side table sorting
@chris-belcher
Copy link
Collaborator

Looks great.

@btcspry
Copy link
Contributor Author

btcspry commented Jun 30, 2015

Thanks! Can you merge it into your master, after your review of course. You may want to change the background color to white, although it's up to you.

@chris-belcher
Copy link
Collaborator

I don't have much of an opinion right now of the color scheme, or anything gui user-facing. Will merge soon.

@btcspry
Copy link
Contributor Author

btcspry commented Jun 30, 2015

Could you just accept the pull request then! With that, I'll contact the owner of JoinMarket.io and get that updated.

ul li a {
padding 0.2em 1em;
}
</style>-->
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you remove these commented out styles?

@btcspry
Copy link
Contributor Author

btcspry commented Jul 1, 2015

They can be removed when the pull request is accepted.

@chris-belcher
Copy link
Collaborator

I would prefer the commented out parts be removed before merging.

I usually leave PRs open for a while so others can comment.

@btcspry
Copy link
Contributor Author

btcspry commented Jul 1, 2015

I have removed the commented out parts, it is added to this pull request!

@chris-belcher
Copy link
Collaborator

You seem quite eager to contribute code, have you seen this wiki page about small projects I think are good https://github.com/chris-belcher/joinmarket/wiki/What-can-I-do-for-JoinMarket%3F

chris-belcher added a commit that referenced this pull request Jul 4, 2015
Changed appearance and sortablility of table and website
@chris-belcher chris-belcher merged commit 44ce8ce into JoinMarket-Org:master Jul 4, 2015
@btcspry
Copy link
Contributor Author

btcspry commented Jul 4, 2015

I have, I'll work on one or two today actually! Thanks

ghtdak pushed a commit to ghtdak/joinmarket that referenced this pull request Oct 1, 2015
Changed appearance and sortablility of table and website
ghtdak pushed a commit to ghtdak/joinmarket that referenced this pull request Oct 4, 2015
Changed appearance and sortablility of table and website
ghtdak pushed a commit to ghtdak/joinmarket that referenced this pull request Dec 4, 2015
Changed appearance and sortablility of table and website
[gitreformat yapf-ify (github/ghtdak) on Fri Dec  4 04:49:29 2015]
[from commit: 44ce8ce]
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants