Skip to content

fix: prevent showCollection button from disappearing#4154

Closed
fstoe wants to merge 3 commits intoProjectMirador:mainfrom
fstoe:4152
Closed

fix: prevent showCollection button from disappearing#4154
fstoe wants to merge 3 commits intoProjectMirador:mainfrom
fstoe:4152

Conversation

@fstoe
Copy link
Contributor

@fstoe fstoe commented Jun 4, 2025

Hey,
this is my idea to fix issue 4152 .

@codecov
Copy link

codecov bot commented Jun 5, 2025

Codecov Report

Attention: Patch coverage is 80.00000% with 1 line in your changes missing coverage. Please review.

Project coverage is 94.91%. Comparing base (470cf46) to head (06f7d7f).
Report is 17 commits behind head on main.

Files with missing lines Patch % Lines
src/state/actions/window.js 50.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #4154      +/-   ##
==========================================
- Coverage   94.91%   94.91%   -0.01%     
==========================================
  Files         324      324              
  Lines       16150    16152       +2     
  Branches     2533     2533              
==========================================
+ Hits        15329    15330       +1     
- Misses        816      817       +1     
  Partials        5        5              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Copy link
Member

@marlo-longley marlo-longley left a comment

Choose a reason for hiding this comment

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

This fix works for me.

@marlo-longley
Copy link
Member

@fstoe I actually noticed something I have a question about -- the name of the Collection displayed in the sidebar and dialog

In this image, the sidebar Collection name is "Aarau, Aargauer Kantonsbibliothek"
Screenshot 2025-06-06 at 10 48 53 AM

After clicking the "Show Collection" button and opening the dialog, the name is "E-codices..." in the sidebar, but the Collection name is "Aarau, Aargauer Kantonsbibliothek" in the dialog.
Screenshot 2025-06-06 at 10 48 59 AM

After closing the dialog, the Collection name in the sidebar returns to being "Aarau, Aargauer Kantonsbibliothek"

Is this expected? I honestly do not know the intended behavior. Thanks.

@lutzhelm
Copy link
Contributor

Is this expected?

I doubt it. If you navigate to the parent collection, the collection info vanishes from the info panel. At this point, if you close the collection modal, the collection info keeps missing. The collectionPath in the state is used for two different things that have weird interactions. I'll look into it.

@lutzhelm
Copy link
Contributor

lutzhelm commented Jul 3, 2025

Closed in favor of #4155.

@lutzhelm lutzhelm closed this Jul 3, 2025
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