Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Mango query UX improvements #1402

Merged

Conversation

Antonio-Maranhao
Copy link
Contributor

Overview

A few adjustments that should improve UX when coding Mango queries in Fauxton:

  • Set code editor max height so buttons and stats are not hidden when the code spans several lines.

Before:
image

After:
image

  • Add cheatsheet modal that shows query operators.
image
  • Display exec stats and warning directly below the editor instead of in a popover to give easier access to that information, particularly when trying to optimize queries.

Before:
image

After:
image

Testing recommendations

In the 'Run a Query with Mango' page:

  • Run queries and verify exec stats and warning are displayed
  • Verify cheatsheet modal opens when clicking on the new icon added

GitHub issue number

n/a

Related Pull Requests

n/a

Checklist

  • Code is written and works correctly;
  • Changes are covered by tests;
  • Documentation reflects the changes;
  • Update rebar.config.script with the correct tag once a new Fauxton release is made

- Set editor max height so buttons and stats are not hidden when the
query code is longer than the page height.
- Add cheatsheet modal that shows query operators.
- Display exec stats and warning directly below the editor instead of
in a popover to give easier access to that information, particularly
when trying to optimize queries.
<tr>
<td><code>$or</code></td>
<td>Array</td>
<td>Matches if ANY the selectors in the array match</td>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
<td>Matches if ANY the selectors in the array match</td>
<td>Matches if ANY of the selectors in the array match</td>

@Antonio-Maranhao Antonio-Maranhao merged commit 613a056 into apache:main Jul 17, 2023
3 checks passed
@Antonio-Maranhao Antonio-Maranhao deleted the query-editor-improvements branch July 17, 2023 19:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants