-
Notifications
You must be signed in to change notification settings - Fork 5
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.
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
Remove tooltip. Remove warning icon and reading room icon from search results. Remove tooltip title #3908
Conversation
the branch is deployed on catalog-staging |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, @christinach ! It will be good to get all of these out of the catalog. My only question is: do we want to get rid of some of the tests as well, rather than setting them to not_to
? Not a blocker! 🐬
spec/system/searching_spec.rb
Outdated
@@ -68,9 +68,9 @@ | |||
|
|||
# We don't have the on-site icon at the moment. It appears as available. We might add it back in the future. | |||
context 'with items which are from aeon locations' do | |||
it 'renders an accessible warning icon for requesting an item in a reading room' do | |||
it 'does not render a warning icon for requesting an item in a reading room' do |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we still need this test?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I can remove them. I kept them to make sure the icon is not there anymore. But it can be deleted now
it 'renders the typical title tooltip' do | ||
expect(subject.css('a').attribute('title').text).to eq('Request to view in Reading Room') | ||
it 'does not render a tooltip' do | ||
expect(subject.css('a').attribute('title')).to be_falsey |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍🏻
a36fc2c
to
3979032
Compare
closes #3885
closes #3906
closes #3907
part of #3878