Skip to content

Change target selector to get correct ::before#8

Open
Thenakedgun wants to merge 1 commit into5t3ph:mainfrom
Thenakedgun:patch-1
Open

Change target selector to get correct ::before#8
Thenakedgun wants to merge 1 commit into5t3ph:mainfrom
Thenakedgun:patch-1

Conversation

@Thenakedgun
Copy link
Copy Markdown

When I trialed this example, I didn't see the text "Is it me you're looking for?" as the example would suggest, but instead "Aww, you came to see me!" This second string seems to come from the normal "style.css" file & didn't have anything to do with the example. I think to make this work, you'd need to change .smol-article-anchor:target::before to #smol-article-anchor-title:target::before.

I don't use GitHub much, so please forgive me if the way I've suggested this is abnormal; just tried to suggest the change via their UI. I got a link to your site from Kevin Powell's newsletter.

When I trialed this example, I didn't see the text "Is it me you're looking for?" as the example would suggest, but instead "Aww, you came to see me!" This second string seems to come from the normal "style.css" file & didn't have anything to do with the example. I think to make this work, you'd need to change .smol-article-anchor:target::before to #smol-article-anchor-title:target::before. 

I don't use GitHub much, so please forgive me if the way I've suggested this is abnormal; just tried to suggest the change via their UI. I got a link to your site from Kevin Powell's newsletter.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant