DOCSP-47211-default-sort-order-setting #724
Conversation
✅ Deploy Preview for docs-compass ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
There was a problem hiding this comment.
Thanks @ltran-mdb2, I left a few non-blocking suggestions for your consideration. LGTM!
source/settings/set-default-sort.txt
Outdated
| Set Default Sort Order | ||
| ====================== | ||
|
|
||
| .. default-domain:: mongodb |
There was a problem hiding this comment.
I believe this directive is no longer functional. It was being removed in some tickets earlier this week.
source/settings/set-default-sort.txt
Outdated
| * - Sort Key | ||
| - Description | ||
|
|
||
| * - $natural: 1 |
There was a problem hiding this comment.
Should these be literals?
There was a problem hiding this comment.
They're shown in normal text on the Compass UI (shown in screenshot) but I will check with tech review
There was a problem hiding this comment.
@djechlin-mongodb what do you think here?
There was a problem hiding this comment.
Discussed with @djechlin-mongodb and we decided to leave these as non literals to match the Compass UI
|
|
||
| .. list-table:: | ||
| :header-rows: 1 | ||
| :widths: 20 80 |
There was a problem hiding this comment.
Suggestion: We don't use it widely, but with these sort of lexical reference tables the :stub-columns: 1 option sometimes helps with formatting for clarity.
I added a line about how these defaults are not saved as part of saved queries! |
DESCRIPTION
Adds new page on default sort order setting
STAGING
https://deploy-preview-724--docs-compass.netlify.app/settings/set-default-sort/#std-label-compass-sort-query-bar
https://deploy-preview-724--docs-compass.netlify.app/settings/settings-reference/#settings
https://deploy-preview-724--docs-compass.netlify.app/query/sort/
JIRA
https://jira.mongodb.org/browse/DOCSP-47211
Self-Review Checklist
External Review Requirements
What's expected of an external reviewer?