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

Website Enhancements #287

Closed
wants to merge 4 commits into from
Closed

Website Enhancements #287

wants to merge 4 commits into from

Conversation

comountainclimber
Copy link
Member

@comountainclimber comountainclimber commented Oct 30, 2017

What current issue(s) from Trello/Github does this address?
#247

Is there anything else we should know?
There is a small development build behind this leveraging webpack here is the repo:
https://github.com/comountainclimber/neon-wallet-website

view the deployed version here: https://comountainclimber.github.io/neon-wallet-website/

@coveralls
Copy link

Coverage Status

Changes Unknown when pulling 62a8043 on comountainclimber:master into ** on CityOfZion:master**.

@comountainclimber
Copy link
Member Author

😺

@slipo
Copy link
Contributor

slipo commented Nov 1, 2017

Big improvement in my opinion! Nice work.

@comountainclimber
Copy link
Member Author

@slipo my pleasure... Looking forward to contributing more to this project. Is there anything I need to do on my end to get this thing deployed?

@evgenyboxer
Copy link
Collaborator

@comountainclimber Thank you for your contribution. We are looking into your PR!

@comountainclimber
Copy link
Member Author

@evgenyboxer If you have any questions let me know... Its basically a very simple jQuery app. Using webpack to hash the CSS and JS files (having these hashed should ensure that clients will always get updated code) as well as for vendor prefixing all of the CSS. I mostly wanted to use webpack because I am so used to reloading dev server :D

@Nat1988
Copy link
Contributor

Nat1988 commented Nov 2, 2017

@comountainclimber Hi buddy. First off, thank you for your contribution! It is very much appreciated and a great start!

So, from a design and user-experience perspective this needs a little work from my eye just to help consolidate some of the information and to bring it in line with the branding of NEON/CoZ. I have attached a quick mock-up that I created last night to give an example of where we see the NEON wallet website in the future. This design will evolve as the wallet evolves but for now I feel a quick landing page would serve best.
concept_02

If you like, we can collaborate together on this, or I could provide designs + assets for build.

~Velocihamster

@comountainclimber
Copy link
Member Author

I asked in the issue thread if you guys had mock ups or something specific and even went through the trouble of creating one to generate feedback. After all the work I put in and after receiving no feedback until now I will not be starting from scratch to redo everything. Good luck on the new site.

@dvdschwrtz-zz
Copy link
Collaborator

@comountainclimber sorry about that. I know everyone is really busy, but it seems you do have a point about posting the mockups and then getting feedback to change it later. This is an open source project so you have lots of people working on it. I think Velocihamster is the only designer or one of the only designers at the moment, so he probably never saw it til now and found a way to improve it. I personally like the new design he posted a lot and I suspect you would agree that it looks pretty good. We can take care of changing the site if you are not interested, but I hope that you keep contributing. NEO is a great project if you are interested in learning the blockchain and working with smart people. I suspect if you asked a code related question there would have been more feedback.

@Nat1988
Copy link
Contributor

Nat1988 commented Nov 2, 2017

@comountainclimber Sorry, this is the first chance I have had to reply. @dvdschwrtz is correct in the fact that the first time I was made aware of this was about 16 hours ago so I had not previously seen your mock-up and comments otherwise a design would have been put forward before any dev work happened - I do apologise for this. I would also like to re-iterate David's point above that if you're not interested in continuing the website development that I hope you keep contributing towards NEO as we need eager and talented people like yourself to make that difference in the community and future projects.

@Ejhfast
Copy link
Member

Ejhfast commented Nov 2, 2017

@comountainclimber your effort was not wasted, it helped inspire @Nat1988's revised design! there was no mockup at the time, many things were/are on backlog. we really appreciate your effort, and CoZ council will certainly reward you for the work you put in

we would love if you continued to help with this, perhaps working with @Nat1988 to iterate on a final design

@Ejhfast Ejhfast reopened this Nov 2, 2017
@comountainclimber
Copy link
Member Author

I appreciate the kind words and apologize for my frustration earlier... I understand that contributors are very busy. If you guys are psyched on this mock up than I can code it over the next day or two. @Nat1988 I definitely like what you have going on here. These are all assets I currently have access to so no need to upload anything.

@Nat1988 @Ejhfast Please let me know if you guys have any other major concerns/design features you would like incorporated into the new site

@Nat1988
Copy link
Contributor

Nat1988 commented Nov 3, 2017

@comountainclimber No worries buddy. I know myself the frustration of designing something only to have to start again so please accept my sincerest apologies again for jumping in late on this - it's great to have you back on board 💯!

So RE the designs: tomorrow I'll jump back on this to tweak/refine the proposed design as this was a quick mockup (it won't change much - just a few alignments here and there), and then I'll create tablet/mobile designs as well. I think the only asset you'll need will be the laptop with the wallet mockup. However I am currently sourcing a better screenshot for this as it would be good to have a wallet UI that has transactions/assets rather than the current empty one.

Regarding anything extra, I think for now we won't need any other design features other than a modal state for the QR codes which I will also provide a design for. @Ejhfast can you confirm please? :)

@Ejhfast
Copy link
Member

Ejhfast commented Nov 3, 2017

@Nat1988 @comountainclimber yes, that all sounds great :)

(i provided @Nat1988 with a better screenshot, so now should have everything you need)

@comountainclimber
Copy link
Member Author

@Nat1988 there is one tiny other detail - the linux version has two builds. Can you be sure to incorporate that into your mock up. Including the modal qr code mock up would help a ton as well.
Let me know when you have finished the laptop/screenshot asset and where I can download the file. thanks guys

@comountainclimber
Copy link
Member Author

comountainclimber commented Nov 3, 2017

https://comountainclimber.github.io/neon-website/ using a placeholder for the screen shot asset until I get one from @Nat1988 will also build out the modals and responsive layouts when I get those mock ups

The repo for this thing is here and I decided to just go with create-react-app as a scaffold so that we could all be on the same page in terms of development deployment etc. https://github.com/comountainclimber/neon-website

@Nat1988
Copy link
Contributor

Nat1988 commented Nov 3, 2017

@comountainclimber Excellent work so far bud!

So for the Linux download, what is the difference between the 2 builds so I can ensure this is communicated in the design?

I'm starting on the tablet/mobile/modal designs now so I should have these to you in the next hour or 2.

~Hamster

@comountainclimber
Copy link
Member Author

@Nat1988 it's just the file extension. Check out the current neonwallet.com

@canesin
Copy link
Contributor

canesin commented Nov 3, 2017

@comountainclimber please join our slack clicking here we have a design channel that is always looking for talent.

@comountainclimber
Copy link
Member Author

@canesin will join the slack in a terrible designer though 😂 Mostly a react/JavaScript developer

@comountainclimber
Copy link
Member Author

#300

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.

None yet

8 participants