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

Hide certain Bibliographic Details from detail record display #19

Open
jameswilson opened this issue Aug 13, 2018 · 0 comments
Open

Hide certain Bibliographic Details from detail record display #19

jameswilson opened this issue Aug 13, 2018 · 0 comments

Comments

@jameswilson
Copy link

We need a way to hide the 'Availability' and 'Access URL' entries from Bibliographic Details table of each detail record output.

The two possible solutions I can think of to fix this would both require coping the entire preprocess to our theme in order to add an extra if statement., however the template_preprocess_ebsco_result() is 160 lines long, which would almost amount to a complete fork of the entire codebase for this section.

A better solution would let us hide these lines without overriding any code. Two possible solutions:

  • expose a setting that lets admins pick which Bibliographic Detail items should be hidden from end users.
  • add a CSS class on each table row in the Bibliographic Details section, so in my theme I can hide the rows with CSS.
jameswilson added a commit to BluesparkLabs/edsapi-drupal8-plugin that referenced this issue Aug 28, 2018
@jameswilson jameswilson mentioned this issue Aug 28, 2018
5 tasks
jameswilson added a commit to BluesparkLabs/edsapi-drupal8-plugin that referenced this issue Aug 28, 2018
jameswilson added a commit to BluesparkLabs/edsapi-drupal8-plugin that referenced this issue Aug 28, 2018
jameswilson added a commit to BluesparkLabs/edsapi-drupal8-plugin that referenced this issue Aug 28, 2018
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 a pull request may close this issue.

1 participant