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
fix: navigate to page from athena when result is for a block #2025
Conversation
This pull request is being automatically deployed with Vercel (learn more). 🔍 Inspect: https://vercel.com/athens-research/athens/CfW82xM6uWmtjs2kPjeaqefg1Q7L |
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.
- Code is correct, but I think we should actually
navigate-uid
for blocks.- Make sure this works for both
enter
andclick
handlers.
- Make sure this works for both
- Can you try writing e2e tests for this?
- Let me think about what
:editing/uid
should do going forward. I think it is being called twice right now. - Deleting a page leads to an error:
I was thinking about this too but it was not clear to me what happens when we navigate to a child block whose parent is closed.
I tested click handlers are working for athena navigation
Yup |
If you go directly to the child block as a |
Oh ok so the intention is to open as |
This was introduced by monitoring I think we should add this there and track there |
That would be cool, but I think that requires more UX thought, similar to |
await page.fill('[placeholder="Find or Create Page"]', 'welcome'); | ||
// Press ArrowDown | ||
await page.press('[placeholder="Find or Create Page"]', 'ArrowDown'); | ||
// Press ArrowDown | ||
await page.press('[placeholder="Find or Create Page"]', 'ArrowDown'); | ||
// Press Enter | ||
await page.press('[placeholder="Find or Create Page"]', 'Enter'); |
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.
Seems like common operations, good place to create util
functions.
No description provided.