search

ros-infrastructure / rosindex

The source code for generating index.ros.org
https://index.ros.org
GNU General Public License v3.0
13 stars 24 forks source link

Redunce dependency list to be package link and arrow for walking up or down #423

Closed tfoote closed 1 month ago

tfoote commented 2 months ago

This fixes #388

It fixes the broken links by removing them as the extra column's info has been reduced. And while here make it symmetric so you navigate packages both up and down the dependency list.

rkent commented 2 months ago

BTW I implemented my suggested changes here and they are merged into my development build at index.rosdabbler.com. I switched the gray to purple so it does not look like disabled on the arrows.

tfoote commented 1 month ago

Thanks for the feedback. If you'd like to break out your changes and replace this PR with the more complete changes you proposoed I'd be happy to do that instead.

rkent commented 1 month ago

"If you'd like to break out your changes and replace this PR with the more complete changes you proposed I'd be happy to do that instead."

I've pretty much already done that, and it is running in https://devindex.rosdabbler.com See https://github.com/rkent/rosindex/tree/tfoote-rkent-simplify-dependency-table I added one commit on top of your PR. I'll PR it, then you can decide what to do with it.

tfoote commented 1 month ago

Replaced by #427