-
Notifications
You must be signed in to change notification settings - Fork 7
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
Better visual desing #8
Comments
We can try Material design, but i dont like material design :) |
Yeap, I agree that Metarial is to mainstream :). Hm, maybe font problem is already closed. |
We can try, and see -> good Material design or not good :) In our case |
I like it. And it doesn't look too material, which is cool. Links to documentation is nice idea. We need to return info badge to plugins. There is only one UX issue right now: we don't show that plugins are inside opened file. Maybe we should make files aside like app left menu panel with full height from header to bottom of the page? |
@andywer I mean, that we don't show clicking on file will reload plugin sections. That plugins is a content of current file. |
@Gaserd Interesting, but I see two problems... First, the executed plugins will be the same for all files, so showing there should be no need to show them for every file separately. Secondly, showing some description text for the plugin is way more complicated then just linking to it. Not impossible, but a lot of effort. |
@andywer hm hm hm. |
@andywer back (?) for link npm ? :) But,user may be use different plugins for different files |
@Gaserd You run postcss on your files with using some set of plugins, but you cannot change the plugins array during execution, so in practice there cannot be different plugins for different files. At least I cannot see a way to accomplish that. Problem with the description is that we would need to fetch the package.json (or the github landing page) of each plugin used to get the description. Creating the link was simple: |
@andywer Okay 👍 |
We could try to take plugin description by:
|
@ai I fear that's not going to work, since this would be resolved to |
@andywer you don’t need to resolve, npm will do it for you. |
@andywer if you don’t start path from |
@ai I am speaking of how node resolves when requiring like Or am I missing something else? |
@andywer node.js will look at So it will find plugin in most of cases. Especially if plugin is in project Also in npm@3 all modules are in flat |
@ai Ahh, you know what? I think we had two different use cases in mind... |
@andywer global npm install is evil :) |
@ai Haha! Fair enough ;) |
Looks cool, but I think we should stick to one theme. Maybe open a twitter poll on which theme to use? |
@andywer I think about switch theme. Pressed Ctrl+D and switched theme, but you can open twitter poll on which theme :) |
@Gaserd Sounds nice, but it's still a small debugging utility. Let's not over-engineer :) |
@andywer it is now a small debugging utillity, tommorow it is a big project about it all speaking 👍 |
Start twitter poll about themes. This is will fun 👍 |
@Gaserd Haha :'D |
@andywer you could insert PostCSS logo in white color ? |
@Gaserd it is hard to understand that click on file block change plugin list. They looks like similar links. People will ask: “why nothing changed after I clicked on file block” |
@ai Okay. |
@Gaserd You are doing great work here! I wasn't sure if anyone really uses a debugging utility on their phone, but... It will be pretty cool for demoing! We should put the inspector with some sample code in |
@andywer Thanks and you are right! |
@andywer are you life ? |
@Gaserd Hey! Sorry for the delay. On one hand I have have been very busy the past days, on the other hand I guess I planned too much for the next release. Here is the current roadmap:
I should probably cut it down in favor of releasing soon. |
@andywer Awesome! I can you help ? |
@Gaserd I wish I could share some work on this, but I think this kind of refactoring requires too much insight into the code. |
@andywer okay :) |
@Gaserd I just recognized we still need a solution to prevent accidentally clicking the plugin documentation link when you want to expand a snapshot. This happens to me permanently when using the latest (not yet released) UI. |
@andywer a4d140e...44980a1#diff-66d5b211acafc61a9b202fc438dc503eR46 see it. Delete "ы" |
@andywer are you live ? :) |
Hey @Gaserd. Yes, I am, but I was on holidays last week ;) There are currently still two issues left:
Sorry for the long delay! |
@andywer Ow! |
@Gaserd Not sure if I got you right... Do you mean you can clean up the CSS? The problem is that I want to fix stuff before merging to master, since the master branch should always remain "clean(ish)". And... we still have to come up with a solution for
Because that will annoy users a lot, I think. |
Looks good 😊 |
@Gaserd Yeah, I like it to :) |
Just published version 0.4.0 with the updated UI. Will close this issue ☀️ |
Thanks @Gaserd! |
Out current UX is nice, but we need better visual design:
We could always try Material design, its visual langauge pretty nice and better then nothing (but custom visual design will be nicer).
With better visual design this tool will look more professional.
The text was updated successfully, but these errors were encountered: