-
Notifications
You must be signed in to change notification settings - Fork 45
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
COM-707 Raise signifier link above headline link #2326
COM-707 Raise signifier link above headline link #2326
Conversation
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 confirm this looks correct on Safari and Chrome but I am not seeing it work properly on Firefox.
@mikemai2awesome Looks good! I am seeing the fix on the Listing Teaser Search (/pattern-lab/?p=components-listing-teaser-use-case-search-result-list) page but I am not Community Search Page (/pattern-lab/?p=pages-community-search-layout). Not sure if that matters or not but I figured I would call it out. |
@colbytcook Good catch. This should do it. |
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.
Confirmed bug fixed, solution makes sense.
Jira
https://pegadigitalit.atlassian.net/browse/COM-707
Summary
Updates the Listing Teaser to allow signifier link to have a higher stacking order than the main headline link.
Details
How to test
Run the branch locally and view docs under Listing Teaser. In the Search Results demo page, go to the Related Videos section, the video thumbnail should open a video modal, the headline link should lead to a page.