Skip to content

UX research page layout #305

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

Merged
merged 7 commits into from
Apr 2, 2025
Merged

Conversation

lidiazuin
Copy link
Collaborator

@lidiazuin lidiazuin requested a review from recrwplay March 14, 2025 14:06
Copy link

netlify bot commented Mar 14, 2025

Deploy Preview for neo4j-docs-ui ready!

Name Link
🔨 Latest commit 2568547
🔍 Latest deploy log https://app.netlify.com/sites/neo4j-docs-ui/deploys/67dbeed1c4abb00008424694
😎 Deploy Preview https://deploy-preview-305--neo4j-docs-ui.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

border-bottom: 1px solid #ccc;
}

.faq-item .arrow {
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@recrwplay I couldn't get the collapsible drawer to work or use the chevron-up.svg as an icon instead. :/

Copy link
Collaborator

Choose a reason for hiding this comment

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

I think [%collapsible] not working is a quirk of how the ui-bundle preview is built. If I build this page as part of the docs-home repo I can get the standard collapsible behavior with eg this format to show a list of collapsible entries:

.What is a user research panel?
[%collapsible%open]
====
A user research panel is a group of users who have signed up to participate in research studies. 
These studies help us understand how people use our products so we can improve them.
====

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Do you know how we can use an image for the collapsible icon? The instructions I read suggested using the icon in the asciidoc, not in the CSS, but then it would be a special character and not an image.

Copy link
Collaborator

Choose a reason for hiding this comment

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

I think if we use it we just have to take the default for now

Copy link
Collaborator

Choose a reason for hiding this comment

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

IOW I don't think it's worth the time for a one-off usage

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Anu also mentioned we don't really need to make the items collapsible, so we can skip that

@recrwplay recrwplay self-requested a review April 2, 2025 11:55
@lidiazuin lidiazuin merged commit 3f7a390 into neo4j-documentation:master Apr 2, 2025
4 of 5 checks passed
@lidiazuin lidiazuin deleted the ux-research branch April 2, 2025 12:08
lidiazuin added a commit that referenced this pull request Apr 3, 2025
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.

2 participants