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

Wallet Manager Style #31

Merged
merged 3 commits into from
Mar 25, 2017
Merged

Wallet Manager Style #31

merged 3 commits into from
Mar 25, 2017

Conversation

hugbed
Copy link
Contributor

@hugbed hugbed commented Mar 25, 2017

Do some CSS work on the wallet manager. (tested on chrome, firefox, safari)
There might be some dead code (hideable and stuff) but the new stuff works.

Here are some snapshots:

screen shot 2017-03-24 at 10 46 17 pm

screen shot 2017-03-24 at 10 46 23 pm

screen shot 2017-03-24 at 10 46 26 pm

screen shot 2017-03-24 at 10 46 30 pm

screen shot 2017-03-24 at 10 46 33 pm

screen shot 2017-03-24 at 10 46 38 pm

@hugbed
Copy link
Contributor Author

hugbed commented Mar 25, 2017

I just realized that the transactions for your current wallet are not blue anymore.
I don't have time to fix this now and will fix this soon.

Something with:

	if(txn.source === coinsClient.wallet_id)
	{
		txnElement.find('#source-address').attr('class', 'my-wallet-id'); // <--- HERE
	}
	
	txnElement.find('#recipient-address').text(txn.recipient.substring(0, 15) + "...");
	txnElement.find('#recipient-address').attr('title', txn.recipient);
	
	if(txn.recipient === coinsClient.wallet_id)
	{
		txnElement.find('#recipient-address').attr('class', 'my-wallet-id'); // <--- HERE
	}

@jordsti
Copy link
Collaborator

jordsti commented Mar 25, 2017

Nice work ! Which University ?

@hugbed
Copy link
Contributor Author

hugbed commented Mar 25, 2017

Sherbrooke, team Gordon!

@DrPandemic
Copy link

🥇

@jordsti jordsti merged commit 102bb02 into csgames:master Mar 25, 2017
@jordsti
Copy link
Collaborator

jordsti commented Mar 25, 2017

You've got 25 coins for that. But there's a bugs with the transaction listing. They aren't shown at the moment. When this will be fix, I'll update the wallet manager running on cscoins.2017.csgames.org.

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.

3 participants