Skip to content

Commit

Permalink
change download-pdf-button => file_download
Browse files Browse the repository at this point in the history
ID must be changed so to be consistent with google analytics tracking.

Issue:
- scientist-softserv/palni-palci#1020
  • Loading branch information
ShanaLMoore committed Sep 3, 2024
1 parent f09b786 commit 54168b7
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
4 changes: 2 additions & 2 deletions app/assets/stylesheets/hyku.scss
Original file line number Diff line number Diff line change
Expand Up @@ -428,7 +428,7 @@ div#collapse-citations {
}

@media (min-width: 576px) {
.citations-button .download-pdf-button {
.citations-button .file_download {
max-width: 300px;
}
}
Expand Down Expand Up @@ -552,7 +552,7 @@ span.constraint-value p, .facet-values p {
}
}

#download-pdf-button {
#file_download {
margin: 10px 0 10px 0;
}

Expand Down
4 changes: 2 additions & 2 deletions app/views/hyrax/base/_download_pdf.html.erb
Original file line number Diff line number Diff line change
Expand Up @@ -2,10 +2,10 @@
<div class="download-pdf-controls">
<% if @presenter.representative_presenter.present? && presenter.file_set_presenters.any?(&:pdf?) %>
<%= button_tag type: 'button',
id: "download-pdf-button",
id: "file_download",
data: { label: @presenter.representative_presenter.id,
path: hyrax.download_path(presenter.representative_presenter) },
class: "btn btn-success btn-block download-pdf-button center-block",
class: "btn btn-success btn-block file_download center-block",
onclick: "window.open(this.dataset.path, '_blank');" do %>
Download PDF
<% end %>
Expand Down

0 comments on commit 54168b7

Please sign in to comment.