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 the possibility to search for a path. #1154

Merged
merged 5 commits into from
Nov 13, 2023
Merged

Conversation

max-debug022
Copy link
Contributor

@max-debug022 max-debug022 commented Jul 4, 2023

This merge request adds the possibility to search for page-paths.

The matches are highlighted in the same way as RegEx-Textsearch matches, which means there is possibly more than one match per page in the page search.

In addition, when searching for a path, the page-tree-row shows the matching path.

Screen.Recording.2023-07-19.at.13.31.52.mov

@max-debug022 max-debug022 self-assigned this Jul 4, 2023
@johnnyomair
Copy link
Collaborator

Please provide screencasts of all cases you're handling in this change

@thomasdax98
Copy link
Member

I tested the search for a page with the name "Vivid" and the path "/planet". I encountered some strange behaviors.
Bildschirm­foto 2023-07-14 um 13 49 50

  1. When searching for a path, the page name is highlighted partially. It would be better to display the path and highlight it (as you suggested). Currently, it's quite confusing.
3.mov
  1. When searching for a path, the subpages of the first page aren't highlighted properly.
Bildschirmaufnahme.2023-07-14.um.13.51.01.mov

@netlify
Copy link

netlify bot commented Jul 19, 2023

Deploy Preview for comet-dxp-docs ready!

Name Link
🔨 Latest commit a77dad0
🔍 Latest deploy log https://app.netlify.com/sites/comet-dxp-docs/deploys/653667a5e5ca9500085f8a33
😎 Deploy Preview https://deploy-preview-1154--comet-dxp-docs.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.

@thomasdax98
Copy link
Member

@johnnyomair Should there be a changeset for this change?

@johnnyomair
Copy link
Collaborator

@johnnyomair Should there be a changeset for this change?

Yes, we also want changesets for user-facing features 🙂

Copy link
Collaborator

@johnnyomair johnnyomair left a comment

Choose a reason for hiding this comment

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

Please add a changeset

@netlify
Copy link

netlify bot commented Oct 23, 2023

Deploy Preview for comet-admin ready!

Name Link
🔨 Latest commit a77dad0
🔍 Latest deploy log https://app.netlify.com/sites/comet-admin/deploys/653667a59788fc0007986b42
😎 Deploy Preview https://deploy-preview-1154--comet-admin.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.

johnnyomair
johnnyomair previously approved these changes Oct 31, 2023
@johnnyomair johnnyomair dismissed their stale review November 6, 2023 11:46

Description is not up to date

@johnnyomair
Copy link
Collaborator

@max-debug022 please update the PR's description so it can be used as the commit message

@johnnyomair johnnyomair merged commit 17f977a into main Nov 13, 2023
10 checks passed
@johnnyomair johnnyomair deleted the allow-path-search-in-admin branch November 13, 2023 16:05
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.

3 participants