----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/100670/#review1508 -----------------------------------------------------------
Thanks again Martin. I've committed it to Amarok master. - Rainer On Feb. 16, 2011, 8:19 p.m., Martin Kunkel wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/100670/ > ----------------------------------------------------------- > > (Updated Feb. 16, 2011, 8:19 p.m.) > > > Review request for Amarok. > > > Summary > ------- > > TabsEngine displays the text from ultimate-guitar.com between the > <pre>...</pre> tags in a browser. Thus, multiple spaces are reduced to one > space which yields chords like > > Bm F# > On a dark desert highway, cool wind in my hair > > (the F# chord should be over "cool") > > Solution: the <pre> tag should be added. > > > This addresses bug 266171. > https://bugs.kde.org/show_bug.cgi?id=266171 > > > Diffs > ----- > > src/context/engines/tabs/TabsEngine.cpp 7c893c6 > > Diff: http://git.reviewboard.kde.org/r/100670/diff > > > Testing > ------- > > Tested with various songs from my library. > > > Thanks, > > Martin > >
_______________________________________________ Amarok-devel mailing list Amarok-devel@kde.org https://mail.kde.org/mailman/listinfo/amarok-devel