Skip to content

Add useful-forks to archived repos#5767

Merged
fregante merged 2 commits intomainfrom
useful-forks-on-archive
Jul 3, 2022
Merged

Add useful-forks to archived repos#5767
fregante merged 2 commits intomainfrom
useful-forks-on-archive

Conversation

@fregante
Copy link
Copy Markdown
Member

@fregante fregante commented Jul 3, 2022

@payne911
Copy link
Copy Markdown
Contributor

payne911 commented Jul 4, 2022

I would have thought it would be nicer to maybe have it closer to

"You can find Useful Forks."

@fregante
Copy link
Copy Markdown
Member Author

fregante commented Jul 4, 2022

This is not my first choice, but it's a compromise for:

  • keeping it short so it doesn't push the content down
  • giving a heads-up to tell that the user is about to go to a third party
  • avoiding an elementReady that would negatively affect every single page load (to detect isArchivedRepo) while only being useful on a fraction of them

I suppose "You can find useful forks [link-icon]" could also work, but it's not as clear that it's a third party. This is particularly a pain point when the users find themselves on an unfamiliar site: useful-forks/useful-forks.github.io#42

@payne911
Copy link
Copy Markdown
Contributor

payne911 commented Jul 4, 2022

I think it would generally be assumed that the "link-icon" would open up an external website.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

Show useful-forks on archived repos

2 participants