Remove view docs dropdown #5400
Conversation
This removes it from the: * User profile page * Homepage This caused a significant DB load when a project has a large number of versions, and is almost certainly never used by users. They can click the project to get a specific version.
There was a problem hiding this comment.
I don't think we need to address here, but we should return to this and potentially remove the link to the build list page when a project doesn't have a build. This seems like a strange place to send a user that is on a profile page -- like, they probably don't have the rights to build the project anyways, so way are we pointing users there?
It matches the current implementation, so this is fine for now.
| <a href="{{ project.get_docs_url }}">{% trans "View Docs" %} | ||
| {% else %} | ||
| <a href="{{ project.get_builds_url }}">{% trans "No Docs" %}</a> | ||
| {% endif %} |
There was a problem hiding this comment.
nit: else and endif indentation is wrong.
There was a problem hiding this comment.
Actually it was missing a </a> :/
What would be the solution here? Removing the "No docs" button completely and only show "View docs" when there is docs to show? |
I guess we can address that in the site redesign |
I'll continue @ericholscher's work here
It looks like this