harvard-lil / perma

Indelible links
417 stars 71 forks source link

Add dismissable perma tools info #3515

Closed tinykite closed 4 months ago

tinykite commented 4 months ago

What this does

Screenshots

Paragraph is visible by default, unless a user has previously clicked the close icon. Screenshot 2024-05-09 at 3 27 12 PM

The position of the message shifts slightly for individual users: Screenshot 2024-05-09 at 3 39 21 PM

The visibility of the content depends on the "perma_tools_reminder" key, which follows the naming pattern of the local storage object, "perma_selection" Screenshot 2024-05-09 at 3 27 30 PM

Notes

codecov[bot] commented 4 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 70.11%. Comparing base (183aff7) to head (fc69dde). Report is 25 commits behind head on develop.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## develop #3515 +/- ## ======================================== Coverage 70.11% 70.11% ======================================== Files 48 48 Lines 6642 6642 ======================================== Hits 4657 4657 Misses 1985 1985 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

tinykite commented 4 months ago

Only thing is that means all users will have this preference unset when we switch over. If we're indeed okay with that: we should probably make sure that the linked "browser tools" page is up to date and is ready for a bunch of fresh views. Something to bring up with the product team, methinks.

That's a reasonable ask — I'll touch base with Clare.