OneGraph / graphiql-explorer

Explorer plugin for GraphiQL
MIT License
904 stars 97 forks source link

[UX] Move add operation buttons to top #38

Closed FluorescentHallucinogen closed 4 years ago

FluorescentHallucinogen commented 4 years ago

Move "Add new query", "Add new mutation" and "Add new subscription" buttons from the bottom to the top and add operations to the top of code to avoid annoying scrolling to them (buttons).

Before:

before

After:

after

This change requested by one of the Hasura community member and should improve usability.

sgrove commented 4 years ago

This was handled differently in #41

FluorescentHallucinogen commented 4 years ago

@sgrove In your implementation (https://www.dropbox.com/s/b9nrgxl87h0xhly/graphiql_polish.mp4), you still need to scroll down to the mutation you just created. :confused:

sgrove commented 4 years ago

It would be a nice bit of polish if you wanted to scroll down to the newly-created operation via some setTimeout mechanism!

On Mon, Jan 20, 2020 at 10:52 PM Alexey Rodionov notifications@github.com wrote:

@sgrove https://github.com/sgrove In your implementation ( https://www.dropbox.com/s/b9nrgxl87h0xhly/graphiql_polish.mp4), you still need to scroll down to the mutation you just created. 😕

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/OneGraph/graphiql-explorer/pull/38?email_source=notifications&email_token=AAAITYBBOL7IXGZMNVMNS6TQ62LUJA5CNFSM4J4S44U2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEJOVS4Q#issuecomment-576543090, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAAITYDMUSXESW66G3WBZA3Q62LUJANCNFSM4J4S44UQ .

sgrove commented 4 years ago

@FluorescentHallucinogen the scroll issue was minor, but should generally be addressed in #42

sgrove commented 4 years ago

Fix published in 0.5.1