-
Notifications
You must be signed in to change notification settings - Fork 24
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
Conversation
✅ Deploy Preview for neo4j-docs-ui ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
src/css/research.css
Outdated
border-bottom: 1px solid #ccc; | ||
} | ||
|
||
.faq-item .arrow { |
There was a problem hiding this comment.
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. :/
There was a problem hiding this comment.
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.
====
There was a problem hiding this comment.
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.
There was a problem hiding this comment.
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
There was a problem hiding this comment.
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
There was a problem hiding this comment.
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
Reference https://www.figma.com/design/5aibTxQGAMllJ0Bc4gSMAI/UX-Research-page?node-id=1-1968&t=3yw3HyJ2ubWsvNe2-0