2 minor UI enhancements

goldbergyoni / node-chaos-monkey

Extremly naughty chaos monkey for Node.js

MIT License

228 stars 10 forks source link

Open goldbergyoni opened 5 years ago

goldbergyoni commented 5 years ago

@TITAN9389 2 minor UI enhancements to finalize this stage:

Enlarge the header in 10%, put a large site title "Node Chaos Monkey" with the monkey enlarged. Move the breadcrumb below the header. Note that there is much space in the header to enlarge the fonts and the monkey
On create new prank, when selecting an item from the list (bottom dropdown) it's not being shown as selected, the backcolor should change

sebmaz93 commented 5 years ago

@i0natan you mean below the header out of that header bar ? or inside it but just other line ?

goldbergyoni commented 5 years ago

@amaliaman Appreciate if you have ideas how to take the overall look&feel one level up

goldbergyoni commented 5 years ago

@TITAN9389

@i0natan you mean below the header out of that header bar ? or inside it but just other line ?

I mean:

Header should grow in 10%
Header should include only a big title and a bigger monkey nearby. No new line or outside
Below the header, there should be a breadcrumb

Cool?

sebmaz93 commented 5 years ago

Done ! opened pull request.

goldbergyoni commented 5 years ago

@TITAN9389 Looks fine ✌️

You mentioned something about a bug in email, what did you mean?

sebmaz93 commented 5 years ago

there was bug in github but its solved now i think :)

goldbergyoni commented 5 years ago

@TITAN9389 Cool. Missed your scipe (:]) message...

I'm thinking now which improvements to apply over the existing state, will bounce back shortly