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

Quiver Tab "Projects" selected although Tab "Workflow" is opened #65

Merged
merged 2 commits into from
Apr 10, 2024

Conversation

noornoorie
Copy link
Contributor

Closes #57

function onTabChange({ index }) {
router.push(items.value[index].to)
}
const activeRouteIndex = computed({
Copy link
Contributor

Choose a reason for hiding this comment

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

We also have to consider that the URL stays the same which shouldn't be the case. I think the solution should be to redirect the route to /workflows before loading the tabs. Then the tabs would pick up the right path automatically.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I'm not really sure about the issue, could you please elaborate further?

Copy link
Contributor

Choose a reason for hiding this comment

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

When you visit the app initially it stays on /. But in fact it is /workflows, so it should redirect the URL to /workflows. After that you should pass the index to the TabMenu. So the getter here becomes simpler. That way the tabs would always stay in sync with the URL.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Okay, I've updated it. Can you check it out?

@noornoorie
Copy link
Contributor Author

@paulpestov Can you check this pull request and merge if there's no issues? I still don't have write access to the repository

@paulpestov paulpestov merged commit 4a339ec into OCR-D:master Apr 10, 2024
1 check failed
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.

Quiver Tab "Projects" selected although Tab "Workflow" is opened
2 participants