bblfsh / web

Web client for Babelfish server
http://dashboard.bblf.sh
GNU General Public License v3.0
23 stars 21 forks source link

Auto lang #124

Closed dpordomingo closed 6 years ago

dpordomingo commented 6 years ago

current: image

Poblem to solve:

dropdown contains:

What is (imho) missleading


alternative: image

Proposal:

Change the auto by:

Why? In my opinion this way is clearer that the driver is not enforced by the dashboard, and the used by the server, in this case, was Java.

bzz commented 6 years ago

Was always frustrated by this label in dropdown. 👍

Wonder, why not taking it a step further, and keep just ["Autodetect", "Python", ...] and then, add another "label", a non-interactive UI element, with the current language somewhere?

Suggesting this, as right now the dropdown serves 2 purposes - allowing user to choose a mode of language detection: auto/manual AND it's also a feedback to the user about the language, detected in Auto mode. Thus confusion.

May be, as this is not super urgent, if @ricardobaeta have some spare cycles he could chime in and provide some feedback/suggestions/mocks with different options from the product side?

dpordomingo commented 6 years ago

I agree with you @bzz : it could be improved a lot the UX of the playground (especially the "auto" feature, that has weird behaviors when you change examples and languages)

If I understood you, you're suggesting something like this (but better designed/thought :wink: ) image

What I tried to do, is just to improve the UX of the dropdown with a small effort, so we can:

Whatever you choose, is ok for me :D

bzz commented 6 years ago

@dpordomingo thank you for a picture, illustrating the idea! Yes, that was exactly what I meant, sans the position of language label, which you are right, could be a subject of further improvement/investigation.

👍 for plan b for now

dpordomingo commented 6 years ago

PR merged, and new issue https://github.com/bblfsh/dashboard/issues/128 opened as agreed