ThePalaceProject / web-patron / 27372724281

88%

push

github

Update visual prominence of external reader buttons (PP-4355) (#145)

## Description

<!--- Describe your changes -->
Buttons that open external web readers are now considered as primary
actions and are also surfaced on the "My Books" page. The new styling
and visual prominence matches that of the iOS and Android apps, further
bringing the CPW into branded parity. All other buttons and select other
surfaces have also been updated to mirror the visual styling of the
apps.

Before
<img width="313" height="175" alt="image"
src="https://github.com/user-attachments/assets/9ac93f88-0cae-4f2e-a09b-d79884135d29"
/>

After
<img width="300" height="150" alt="image"
src="https://github.com/user-attachments/assets/60efe54b-efc2-41e7-9638-988348478e18"
/>

## Motivation and Context

<!--- Why is this change required? What problem does it solve? -->
<!--- If it fixes an open issue, please link to the issue here. -->
Initial testing highlighted a difficult transition from borrowing to
reading. Testers noted that the "Read Online" and "Listen Online"
buttons had low visual priority on the page and thus were easily missed
by users, especially K-12 students.

[Jira PP-4355](https://ebce-lyrasis.atlassian.net/browse/PP-4355)

## How Has This Been Tested?

<!--- Please describe in detail how you tested your changes. -->
<!--- Include details of your testing environment, and the tests you ran
to -->
<!--- see how your change affects other areas of the code, etc. -->
- Manually tested in Firefox, Chrome, and Safari
- Unit and integration tests have been added to test the new verbiage

## Checklist:

<!--- Go over all the following points, and put an `x` in all the boxes
that apply. -->
<!--- If you're unsure about any of these, don't hesitate to ask. We're
here to help! -->

- [ ] I have updated the documentation accordingly.
- [x] All new and existing tests passed.

223 of 258 branches covered (86.43%)

Branch coverage included in aggregate %.

357 of 401 relevant lines covered (89.03%)

31.59 hits per line