-
Notifications
You must be signed in to change notification settings - Fork 17
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
Styleguide and Language for the UI's #7
Comments
Some inspiration from Mozilla: Photon Design System |
|
@olizilla About the Design System Guidelines - Can we create together a list (a small one - first MVP) with what we need in first? In my opinion it should be: Logo(options and do/dont's) |
For sure! My main focus on this is to make sure that that whatever the design guidelines are, that I make them easy for devs to implement, by capturing them in code and publishing it as a repo and website. I made a start on that here: https://github.com/ipfs-shipyard/ipfs-ui-style-guide and I'd love to chat about it when you have time. |
Also! Everything in https://github.com/ipfs-shipyard/ipfs-ui-style-guide is TBC as I haven't checked it with you yet @akrych 😄 |
Hi all, be sure to check (first sketch of) IPFS Brand Book Guidelines at ipfs-inactive/ipfs-ui-style-guide#3 :) |
Hello Guys, I start working on IPFS Web GUI Kit and and I would like to show you how the work is going :)
You can also see it and CSS inspect on Zeplin: https://zpl.io/awyMrgj |
sure :) 👍 |
@hacdias - Sure thing:) - all parts will have the dimensions and attributes described (btw. You can se almost everything is in Zeplin now - I guess that the "focus" one have missing shadow because I made it by the "glow" not "shadow" function in Photoshop:) |
Thanks! |
Just a thought: we should consider using Github Pages for hosting things like UI Kit (after conversion to HTML page) directly from github repo. (It is possible to provide custom index.html) |
But IPFS tho!?
Seriously though, I agree we should keep the process super simple.
…On 21 Jan 2018 18:36, "Marcin Rataj" ***@***.***> wrote:
Just a thought: we should consider using Github Pages
<https://pages.github.com/> for hosting things like UI Kit directly from
github repo. (It is possible to provide custom index.html
<https://user-images.githubusercontent.com/157609/35197610-08178e00-fee2-11e7-8ac7-64300a2f79e0.png>
)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#7 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AADl98sdVK65TuLHWvjIaUVFt7OEPJ_mks5tM4OqgaJpZM4RChRk>
.
|
@akrych that looks HAWT 🔥 👏🏽👏🏽
👍 for simplicity, 👎 for not dogfooding our stuff. Having a publish script on merge to master should not be complicated. |
@akrych this is super cool! One issue I noticed - the selected radio - I was expecting a solid turquoise circle inside, I think the turquoise hoop could be misconstrued as a deselected. |
@alanshaw Sure, we can change it:) |
Link to better quality: https://c1.staticflickr.com/5/4701/28390704879_54074deeec_o.png |
Very nice! I made a start on css-ing the previous version over here: and extracted out a minimal re-usable set of css utility classes over here: |
Hey @akrych! I really like what you've been doing 😄 I'll probably start to implement it on IPFS Desktop soon! @malikpiara also liked it very much! He also has some suggestions: maybe use the grey color on the 'Cancel' buttons and a green color on 'Create'. Btw, do you have it on Zeplin? |
The default button style has low contrast, making white text in the first column hard to read: (we switched away from that style in Web UI and Companion, switching to version from the second or third column) |
Closing in favor of #29 (and mentioning this issue there for the sake of not losing these design sketches) |
@olizilla and @alanshaw brought up a ton of valuable points to the discussion, some of them were focused on the lack of Style Guide and Language.
@olizilla @alanshaw, mind sharing a recap here?
The text was updated successfully, but these errors were encountered: