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.

Sign up for GitHub

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

Jump to bottom

MHV-66099 Images page (image download, image layout and format, download alerts) #34706

Open

Wyzardsleeves wants to merge 1 commit into main from MHV-66099_images_page_image_download_,_image_layout_,_format_and_download_alerts

+52 −11

Contributor

Wyzardsleeves commented Feb 14, 2025 •

edited

Loading

Summary

Updated content regarding the Image Page.
- Changed "col-4" to "col-6" to create two columns
- Images were already responsive.
- Used math to calculate the # to # of ## images label
- I added the "download started" success banner after the Dicom link was clicked. The failure banner was not mentioned in the ticket, so I'm guessing there's another ticket for it.
- Removed the extra "to" in a "p" tag
This is not a bug.
Updated the JSX and already existing functions.
MHV Medical Records.

Related issue(s)

MHV-66099 Images page (image download, image layout and format, download alerts)

Screenshot

Testing done

Tests are passing.

What areas of the site does it impact?

MHV Medical Records


          MHV-66099

b9f07c4

Images page (image download, image layout and format, download alerts)

Wyzardsleeves requested review from a team as code owners

February 14, 2025 17:38

rmessina1010 approved these changes

View reviewed changes

Contributor

rmessina1010 left a comment

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet