search

powa-team / powa-web

PoWA user interface
http://powa.readthedocs.io/
74 stars 31 forks source link

UI enhancements #33

Closed pgiraud closed 6 years ago

pgiraud commented 7 years ago

Following the notes I took several weeks ago (https://github.com/pgiraud/powa/wiki/Notes-on-UI) on things I noticed on the UI/UX of PoWA.

I took almost every single comment I made into account. Here are the main enhancements:

Here are some before / after screenshots:

All databases

screenshot from 2017-07-10 14-50-15 screenshot from 2017-07-10 14-50-06

Database

0 0 0 0_8888_database_powa_overview_ 0 0 0 0_8888_database_powa_overview_ 2

Configuration

0 0 0 0_8888_config_ 0 0 0 0_8888_config_ 1

Databases list

0 0 0 0_8888_overview_ 1 0 0 0 0_8888_overview_

pgiraud commented 6 years ago

This PR is now ready for review.

rjuju commented 6 years ago

Thanks a lot @pgiraud! I'm quite sorry it took me so long to review this PR.

I rebased this branch against current head, and pushed everything as a single commit: https://github.com/powa-team/powa-web/commit/9ac03bcc3637561e62ddf2881fa72ae6212e520f. I hope I didn't mess anything during the rebase, nothing looked wrong when I tested it.

Ideally, I'd have split the modifications in 3 commits:

However, there were too many dependencies in the different commits to easily split them like this, so I gave up :(

The UI is way better now, so thanks a lot again!!