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

Add a warning to expand docs #68

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Add a warning to expand docs #68

wants to merge 2 commits into from

Conversation

ljhaywar
Copy link
Collaborator

@ljhaywar ljhaywar commented Oct 17, 2024

I was working through the Search lab and thought I had made a mistake when creating the index. I completely missed the arrow button on the right side of the document. Harshit said this is a common mistake that devs make when working through the lab.

@ljhaywar ljhaywar self-assigned this Oct 17, 2024
@@ -18,6 +20,11 @@ and look at the results you get.

<Screenshot src="img/screenshots/2-search/3-test-search/2-search-tester.png" alt="The Search Tester" url="https://cloud.mongodb.com" />

:::info
Some of the fields in the documents (like `title`) may be hidden. You can expand a document to see all of its fields by clicking the arrow button on the right side of the document.
<img src={useBaseUrl("img/screenshots/2-search/3-test-search/4-expand-document.png")} alt="The expand document button" />
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I tried using the Screenshot component here, but it wasn't displaying correctly inside of the info widget. I went with a regular image instead. If there is a way to get it to display correctly, I'm good with using the Screenshot component.

@ljhaywar ljhaywar requested a review from dfreniche October 17, 2024 12:35
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