department-of-veterans-affairs / va.gov-cms

Editor-centered management for Veteran-centered content.
https://prod.cms.va.gov
GNU General Public License v2.0
96 stars 69 forks source link

Update Download PDF link on Form detail pages to <button> element #17314

Open laflannery opened 6 months ago

laflannery commented 6 months ago

Status

[2024-08-19] [Fran] Blocked by the linked Spike.

Description

We recently removed the find_forms_show_pdf_modal flipper so that now we always show the modal when the user clicks to Download the PDF on the Find forms search results. When this was done, the element for "Download PDF" was switched to a <button> because it now activates the modal consistently.

Screenshot 2024-02-22 at 7 58 25 PM


Screenshot 2024-02-22 at 7 58 52 PM

This element is also present on the forms detail page but is currently still an <a> tag. For consistency should also change this to the same <button> behavior as on the search results: image

Acceptance Criteria

FranECross commented 6 months ago

Fran to create Spike ticket to refresh our memories on what's happening. Daniel might have some background on this.

laflannery commented 5 months ago

There is an epic specifically to get the listing and detail behavior of this modal experience the same, this ticket would probably belong in that epic because that's ultimately all I'm asking for here - update the detail page to match the new behavior that's now on the listing