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

Clicking "Import -> [ANY OPTION]" keeps navbar header disabled #3189

Closed
kshepherd opened this issue Jul 17, 2024 · 3 comments · Fixed by #3220
Closed

Clicking "Import -> [ANY OPTION]" keeps navbar header disabled #3189

kshepherd opened this issue Jul 17, 2024 · 3 comments · Fixed by #3220
Assignees
Labels
bug good first issue This is a smaller ticket/bug that may be easier for new contributors help wanted Needs a volunteer to claim to move forward tools: import
Milestone

Comments

@kshepherd
Copy link
Member

Describe the bug

In DSpace 8.0, if you are logged in as an administrator and use the left-hand sidebar to click "Import" and then either of the two options (Metadata, Batch Import ZIP), the navbar / header links and buttons become disabled and are unable to be interacted with. Even a full page refresh will not re-enable these links. To clear it you must either proceed through the metadata import process, or try some other admin sidebar functions which load different pages and will reset the navbar enabled/disabled state.

It seems like something going wrong with responsiveness detection, perhaps the navbar thinks it is collapsed? The sidebar also doesn't auto-hide for some items, typically those that just go to a new dspace page without also popping up a selector modal (which do auto hide), but I'm not sure if that's related and so far, "Import" items are the only ones I have seen that cause the navbar / header to break. It happens whether or not the sidebar is pinned.

To Reproduce

Steps to reproduce the

  1. Log into a DSpace 8.0 site (e.g. demo.dspace.org) as an administrator
  2. Reveal the left-hand sidebar menu, click Import and then click either Metadata or Batch Import (ZIP)
  3. Try to click links in the breadcrumbs, header, login / profile dropdowns, etc. The cursor will remain as a 'pointer' and nothing can be interacted with.

Expected behavior

The DSpace navigation and header links should all be active and working after clicking Import -> Metadata, just like other admin pages

@kshepherd kshepherd added bug good first issue This is a smaller ticket/bug that may be easier for new contributors needs triage New issue needs triage and/or scheduling labels Jul 17, 2024
@github-project-automation github-project-automation bot moved this to 🆕 Triage in DSpace Backlog Jul 17, 2024
@tdonohue
Copy link
Member

Thanks @kshepherd . I can verify I see the same behavior on demo.dspace.org. If you leave the page (e.g. by clicking the "Back" button), the header is usable again.

So, I suspect it might just be that those pages (or more accurately a component on those pages) are somehow disabling the header or "drawing" something (invisible) over the header, so the header links/buttons can no longer be hovered over.

Moving this to our 8.x/7.x maintenance board. Needs a volunteer.

@tdonohue tdonohue added help wanted Needs a volunteer to claim to move forward tools: import and removed needs triage New issue needs triage and/or scheduling labels Jul 17, 2024
@Andrea-Guevara
Copy link
Contributor

Good afternoon @tdonohue! My name is Andrea and I'm part of the Neki-it company. We have analyzed and seen that in the component “ds-file-dropzone-no-uploader” the classes “position-fixed h-100 w-100” are affecting the behavior of the navbar. We would like to contribute a solution to this problem.

@Andrea-Guevara
Copy link
Contributor

Good morning @tdonohue! We've provided a PR with the cause and solution to this problem.
#3220

We look forward to your feedback!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug good first issue This is a smaller ticket/bug that may be easier for new contributors help wanted Needs a volunteer to claim to move forward tools: import
Projects
Development

Successfully merging a pull request may close this issue.

3 participants