vkuznet / transfer2go

Distributed, loosely couple agent-based transferring system
MIT License
8 stars 2 forks source link

Modify html pages #32

Closed rishiloyola closed 7 years ago

rishiloyola commented 7 years ago

Demo: screen shot 2017-07-16 at 11 21 25 pm screen shot 2017-07-16 at 11 21 56 pm

vkuznet commented 7 years ago

Rishi, please put Source/Dest/Block/Dataset/File as single row. Also I don't understand what colored boxes in top-right corner stands for. My understanding that you should show Status value as such colored box. But there is no need for all of them to show up in top-right corner. Valentin.

On 0, Rishi notifications@github.com wrote:

Demo: screen shot 2017-07-16 at 11 21 25 pm screen shot 2017-07-16 at 11 21 56 pm

You can view, comment on, or merge this pull request online at:

https://github.com/vkuznet/transfer2go/pull/32

-- Commit Summary --

  • Modify html pages

-- File Changes --

M core/catalog.go (3)
D html/index.html (131)
A html/main.css (134)
A html/main.html (38)
A html/main.js (98)
M server/handlers.go (1)
M server/server.go (4)

-- Patch Links --

https://github.com/vkuznet/transfer2go/pull/32.patch https://github.com/vkuznet/transfer2go/pull/32.diff

-- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/vkuznet/transfer2go/pull/32

rishiloyola commented 7 years ago

@vkuznet Those are buttons to render a different kind of requests. If you want to see pending requests click on pending buttons. If you want to see deleted, finished, processing, error requests click according to that.

rishiloyola commented 7 years ago

Demo: screen shot 2017-07-17 at 10 42 26 pm Is it fine now?

vkuznet commented 7 years ago

Regarding buttons, I'll replace them with simple top-level drop-down menu with prefix saying: Select status. The block, dataset and file names should be displayed on individual row, i.e. separate them with </br>. As I said they're very long strings and as currently presented they will not fit into a single line.

rishiloyola commented 7 years ago

@vkuznet I made the necessary changes. For now, keep those buttons. I am planning to redesign this interface after the test.

rishiloyola commented 7 years ago

@vkuznet Can you merge this so that I can deploy the master branch and can test it?

vkuznet commented 7 years ago

Merged.