search

nul800sebastiaan / Cultiv.Tabify

MIT License
8 stars 4 forks source link

Update UI to align more with Umbraco #3

Closed FransdeJong closed 5 years ago

FransdeJong commented 5 years ago

On twitter there have been some ideas on improving the UI of this package https://twitter.com/cultiv/status/1147788199361765377

Personaly I like the style Matt proposed: https://twitter.com/mattbrailsford/status/1148166359588777986

I'd like to discuss the layout here so we can start on getting this package more aligned with the rest of the backoffice.

nul800sebastiaan commented 5 years ago

Just go for it, I'm not a designer, Matt is, so trust his judgment. 😅

FransdeJong commented 5 years ago

Just making sure we are all on the same page ;-)

dalyUSN commented 5 years ago

Hey. I already made a PR for an update to the UI. You can see it here: https://github.com/nul800sebastiaan/Cultiv.Tabify/pull/1

Happy to help you out further.

Thanks Paul

FransdeJong commented 5 years ago

Looks nice!

erikjanwestendorp commented 5 years ago

Hi,

I'm also not a designer but I like the style proposed by Matt as well. Maybe the background color should be white to be more consistent with other sections like settings? See options below.

option1

option2

FransdeJong commented 5 years ago

I think the solution of @dalyUSN is best of both worlds?

nul800sebastiaan commented 5 years ago

Fixed in https://github.com/nul800sebastiaan/Cultiv.Tabify/pull/1