Skip to content

XNAT-8789 Empty header resize function causing logo overlap - #30

Open
willhorton-xw wants to merge 1 commit into
developfrom
fixes/XNAT-8789
Open

XNAT-8789 Empty header resize function causing logo overlap#30
willhorton-xw wants to merge 1 commit into
developfrom
fixes/XNAT-8789

Conversation

@willhorton-xw

Copy link
Copy Markdown
Contributor

This fixes a logo display issue that appears whenever a site-wide notification is active on the home page. There are three approaches:

  1. Remove the call to XNAT.app.adjustHeaderAndNavForLogoSize() from QuickSearch.vm
  2. Empty the contents of that function -- which were mostly commented out already -- in case a plugin replaces QuickSearch.vm with its own version
  3. Add CSS to ignore inline height settings on the #header object

// only show the header logo on the 'home' page
$('#header_logo').show();
XNAT.app.adjustHeaderAndNavForLogoSize();
// XNAT.app.adjustHeaderAndNavForLogoSize();

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I think you should just delete the line rather than comment it out.

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.

2 participants