wisavalite / gitiles

Automatically exported from code.google.com/p/gitiles
1 stars 0 forks source link

rendering of relative URLs in README.md for directory listing page is wrong #82

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. do a web search for "chromium gn"
2. the top link is to https://chromium.googlesource.com/chromium/src/tools/gn/ 
(this is perhaps less than ideal, 
but a separate issue). 
3. that directory listing lists the change log for that dir, then 
//tools/gn/README.md
4. attempt to click on the links at the bottom of the readme, e.g. "Quick 
Start" 
(https://chromium.googlesource.com/chromium/src/tools/gn/docs/quick_start.md)
5. get a 404 back

What is the expected output? What do you see instead?

ideally the link would resolve to the version on master.

Failing that, the relative URL could also be rewritten to

https://chromium.googlesource.com/chromium/src/+/master/tools/gn/docs/quick_star
t.md instead.

Failing that, it might be better to just rewrite README.md to point to the full 
URL, but I'd be sad to do that since it would hard-code the branch.

Original issue reported on code.google.com by dpranke@chromium.org on 17 Aug 2015 at 7:47

GoogleCodeExporter commented 9 years ago

Original comment by dpranke@chromium.org on 17 Aug 2015 at 7:48

GoogleCodeExporter commented 9 years ago

Original comment by dpranke@chromium.org on 17 Aug 2015 at 7:48

GoogleCodeExporter commented 9 years ago
bump ... this is increasingly causing confusion for our users, since this link 
is the first thing that shows up when you search for "chromium gn" on Google.

Any thoughts? Should I do the workaround I described in the bug description (to 
rewrite to use full URLs)?

Original comment by dpranke@chromium.org on 3 Sep 2015 at 3:42