sciserver / scidrive-ui

Web UI for scidrive
http://www.scidrive.org
Apache License 2.0
0 stars 0 forks source link

Enhancement: add address bar for second panel. #33

Closed mtaghiza closed 7 years ago

mtaghiza commented 9 years ago

When the second panel is displayed, it does not show an address bar. We could add one as an enhancement.

jkim485 commented 9 years ago

In particular if the second panel displays an empty folder there is no way to figure it out which folder it is.

dimm0 commented 9 years ago

Second address bar will break current design. Fixed by making path change when mouse cursor is entering a panel - now it always shows the path of panel under the cursor.

dimm0 commented 9 years ago

Actually, that makes it harder to choose a folder for second column, as cursor usually goes on top of the first column... Ideas are appreciated

jkim485 commented 8 years ago

I don't have good idea. The solution you mentioned above seems to me a reasonable one for now: Fixed by making path change when mouse cursor is entering a panel - now it always shows the path of panel under the cursor.

I have tried but the path did not show up as you described.

dimm0 commented 8 years ago

Made the header of inactive panel greyed out, so that it's visible which panel is active and which path is shown. That seems like the best solution for now

jkim485 commented 8 years ago

Sounds good.

Where is the active path shown? On the panel itself or on the address bar? Is it already available on the scitest09?

dimm0 commented 8 years ago

It is available Path is shown as before

jkim485 commented 8 years ago

tested on scitest09 and confirmed the fix.

jkim485 commented 8 years ago

Tested on pre-prod and confirmed the fix.

jkim485 commented 8 years ago

Leave a comment

jkim485 commented 8 years ago

Add second panel address bar. If this is hard to achieve, highlight the selected panel.

dimm0 commented 8 years ago

Highlight the selected panel should work already

jkim485 commented 8 years ago

It does but user doesn't recognize it easily. So we want to make 'highlight' more noticeable.

mtaghiza commented 8 years ago

fixed. Highlighted active panel and greyed-out inactive panel.