asah / footprint2009dev

original dev repo for AllForGood.org
http://AllForGood.org/
0 stars 1 forks source link

We should strip "http://" in search results #290

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
In the search results, we show the domain url with "http://".

Executive Director
Seattle Washington - Present - April 9, 2011
AHAC Volunteer Executive Director Part-Time Wage and Hours: Volunteer, 15-
20 hours per week Purpose The mission of the Alternative Health Care Access 
Campaign (AHAC) is to increase the accessibility of quality alternative 
healthcare for homeless and low-income adults and ...
http://www.idealist.org - Like

We should show "www.idealist.org" vs "http://www.idealist.org"

Original issue reported on code.google.com by kar...@gmail.com on 19 May 2009 at 1:47

GoogleCodeExporter commented 9 years ago

Original comment by adam.sah on 19 May 2009 at 6:48

GoogleCodeExporter commented 9 years ago

Original comment by adam.sah on 20 May 2009 at 5:14

GoogleCodeExporter commented 9 years ago
Should be easy... Keeping in 1.1

Original comment by kar...@gmail.com on 20 May 2009 at 5:48

GoogleCodeExporter commented 9 years ago
This is pretty trivial to change, just modifying the line in searchresults.py:
    self.url_short = '%s://%s' % (parsed_url.scheme, parsed_url.netloc)
to
    self.url_short = parsed_url.netloc

But when I look at the results I think they look a little strange. The http://
clearly points out that the text is a link, and without that context it 
definitely
loses something. I'll post some screenshots in a bit. I think we may want to 
run this
by our UX person.

Original comment by block...@gmail.com on 21 May 2009 at 3:44

GoogleCodeExporter commented 9 years ago
It's common to not show "http://", since we do this on Google web search, Image 
Search, etc.  However, lowering the priority on this one.  

Original comment by kar...@gmail.com on 28 May 2009 at 12:46

GoogleCodeExporter commented 9 years ago
Fixed in r711

Original comment by ehysen on 5 Jun 2009 at 5:52