search

readthedocs / ext-theme

Read the Docs drop in replacement site templates
2 stars 2 forks source link

Use target="_blank" for external links #440

Closed humitos closed 1 month ago

humitos commented 1 month ago

This is based on a feedback we received from a user. As a general rule, we should use target=_blank for all the links that go outside the dashboard.

There is a lot of lint changes. I split the work into different commits to make it easier to review.

humitos commented 1 month ago

There is some inline style lint issues still, but I'm going to merge this as-is for now because I don't know how to skip them. Also, I'm a little tired of fixing such amount of lint issues in this PR. It touched a lot of files. We can continue fixing them in future iterations.