search

duckduckgo / zeroclickinfo-longtail

DuckDuckGo Instant Answers based on full-text data
http://duckduckhack.com/
Other
87 stars 113 forks source link

MetroLyrics: IA shows a fraction of the lyrics #42

Closed EpicWolverine closed 8 years ago

EpicWolverine commented 8 years ago

I love the lyrics IA, but it's incredibly frustrating when I have to click on the "More at MetroLyrics" button to see the entire song. Why is the whole song not displayed when "+ Show More" is clicked? Would doing so violate DDG's agreement with MetroLyrics or is it simply an old design decision that needs to be fixed?

IA Page: http://duck.co/ia/view/lyrics

jdorweiler commented 8 years ago

@EpicWolverine we can't show the full song lyrics. We might be able to at least show the box expanded so you don't need to click the (+) show more to see one or two additional lines. Maybe @bsstoner or @andrey-p can comment on that?

bsstoner commented 8 years ago

The issue with expanding by default is that the amount of data we can show is wildly inconsistent from one song to the next. If we expanded by default there would be a lot of cases where the height of the IA would be way to big.

There's some other options we could explore: 1) Just chop off the text, and redirect the show more to metrolyrics? 2) Explore other UI's, like horizontally chunking up the lyrics? 3) Play with a more dynamic threshold for the expand logic?

jdorweiler commented 8 years ago

I could generate a distribution of the length of the lyrics. My guess is that a majority will fall between 5-10 lines long.

jdorweiler commented 8 years ago

Looks like increasing the height to 7-8 lines would cover the majority of results. Right now we only show 3 lines so even going up to 4 lines would be a big improvement. selection_087

bsstoner commented 8 years ago

Awesome, data!

bsstoner commented 8 years ago

@jdorweiler can you create a task in general front-end to make that change?

jdorweiler commented 8 years ago

Thanks for the issue @EpicWolverine. @bbraithwaite and @bsstoner fixed this internally.

EpicWolverine commented 8 years ago

You're welcome, and thank you for handling it.