View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0016619 | MMW 5 | Track Browser | public | 2020-05-15 03:12 | 2022-09-14 14:32 |
| Reporter | peke | Assigned To | |||
| Priority | low | Severity | minor | Reproducibility | always |
| Status | feedback | Resolution | reopened | ||
| Product Version | 5.0 | ||||
| Target Version | 5.2 | Fixed in Version | 5.0 | ||
| Summary | 0016619: Info panel: multi-column text is cropped before the final column | ||||
| Description | If Some Artist/Album have enough information, a non expanded version is cropped/cut incorrectly so it is not readable unless user expand it using MORE >> as text is missing. This is most likely because fetched text is not wrapped to visible text area. | ||||
| Tags | No tags attached. | ||||
| Attached Files | 16619_CSS Text Dynamic Span 2 Columns.txt (3,987 bytes)
<!DOCTYPE html>
<html>
<head>
<style>
.wikiDescription {
max-width: 100%;
height: 100%;
column-count: 2;
column-span: all;
}
.ArtistViewHeader {
//height: 427px;
height: 254px;
background: AliceBlue;
}
.OtherTextMoreLess {
line-height: 2em;
vertical-align: middle;
background: DarkOrchid;
}
</style>
</head>
<body>
<div class="ArtistViewHeader"><div class="wikiDescription"><p>Laura Pausini OMRI (Italian: [ˈlaura pauˈziːni]; born 16 May 1974) is an Italian singer-songwriter, record producer and television personality. As a child, she was encouraged by her father to join him during his performances in local piano bars. After competing in local singing contests, Pausini signed her first recording contract. She rose to fame in 1993, winning the newcomer artists' section of the 43rd Sanremo Music Festival with her debut single "La solitudine", which became an Italian standard and an international hit, topping the charts in Italy, the Netherlands, and Belgium. Her self titled debut album was released in Italy on 23 April 1993 and later became an international success, selling two million copies worldwide.</p>
<p>Its follow-up, Laura, was released in 1994 and confirmed her international success, selling three million copies worldwide. During the same year, she released her first Spanish-language album, Laura Pausini, composed of ten adapted songs originally included in her previous works. The album was certified diamond by the Association of Phonographic and Videographic of Spain, making her the first non-Spanish artist to sell more than one million copies in Spain.</p>
<p>As of 2017, Pausini has released twelve studio albums, two international greatest hits albums and two compilation albums for the Hispanic and Anglophone market only, respectively. She mostly performs in Italian and Spanish, but she has also recorded songs in English, French, Portuguese and more recently, Catalan. Her sole English-language album of original songs, From the Inside, was released in 2002, but failed to satisfy commercial expectations both in North America and in the rest of the world. In 2006, her record Io canto was the best-selling album of the year in Italy. </p>
<p>In 2004, Allmusic's Jason Birchmeier considered Pausini's sales "an impressive feat for someone who'd never really broken into the lucrative English–language market". FIMI certified Pausini's sales of more than 70 million records with a FIMI Icon Award.</p>
<p>Pausini appeared as a coach on both the Mexican and Spanish versions of international reality television singing competition franchise The Voice, was a judge on the first and second series of La banda, and is likewise a judge on the Spanish version of international franchise The X Factor. In 2016, she also debuted as a variety show presenter, hosting the television show Laura & Paola, with actress Paola Cortellesi.</p>
<p>Throughout her career, she has won numerous music awards in Italy and internationally. She has earned the first and third place at the Sanremo Music Festival, two "Lunezia Awards", ten Wind Music Awards, four awards at the Festivalbar, seven Telegatti, and an MTV Italian Music Award. Internationally, she has won four awards at the Viña del Mar International Song Festival, three Lo Nuestro Awards, four Latin Grammy Awards, a Billboard Latin Music Award, and six World Music Awards. In 2006, she also became the first Italian female artist to win a Grammy Award, receiving the accolade for Best Latin Pop Album for the record Escucha. She has been honoured as a Commander Order of Merit of the Italian Republic by President Carlo Azeglio Ciampi and as a World Ambassador of Emilia Romagna.</p>
</div>
</div>
<div class="OtherTextMoreLess" vertical-align: middle;>
<p style="text-align:right">
<span style="float:left"> Continue reading at Wikipedia...</span>
...More ≫ / Less ≪
</p></div>
</body>
</html> | ||||
| Fixed in build | 2254 | ||||
|
|
Possible regression of fixing 0016602 |
|
|
I also don't understand what the issue is, i.e. MORE >> is there to show all the text. I don't see an issue, resolving... |
|
|
In attached picture highlighted text should be in second column and not cut off and below track listing. Attaching two new pictures. One is Same text but in Single column and second one is correct 2 column cut with more >>. Compare those two pictures and you will see that WIKI space is not resized and with that text is not shown/rendered in order but simply cropped. Read the text from my first picture with more >> and same text in second picture with more >> and you will see that highlighted text in first picture is missing. |
|
|
The text is not missing, the text is hidden by the cropping of the header. It is hidden with some gradient, to visually show, that the text is cropped and continues below too. There is no direct end for wrapping, the text is not wrapped there, the text continues. This way it was designed. Clicking "More >>" shows the hidden part. Even with wrapping some part of the line would be missing, because of the gradient. I really do not see any problem in this. |
|
|
I checked/inspected the source and It looks like problem is in relations as data-control-class="ArtistViewHeader" which is dynamically expanded and have Dynamic Flag so it expands if MORE>>/LESS<< thru style="height: 254px;" while "headerContainer" class="noOverflow paddingColumn" Is not dynamically resized along with its content. As a result when Artist view is in collapsed mode ArtistViewHeader is 254px high, but noOverflow paddingColumn is 427px high and data is hidden. Once you expand (MORE>>) then ArtistViewHeader is 427px high and all data is there. |
|
|
As explained by Michal, it is by design. Maybe just the gradient on this skin is not visible enough making this not clear. So expanding the gradient (or making it more prominent) should be enough? Nevertheless sounds as a minor issue. |
|
|
I do not see any issue with gradient, problem is Text is not scaled correctly in viewed area because viewed area is not scaled dynamically at all. but it is just hidden/overlapped. It is not just problem with Artist, It is same with other similar views that show online data. Album, TV Series, Podcast, ... It is design flaw that should be fixed and that area and its content shoul dbe resized and scaled OK. Less/More is made to show that there is more test below and it is OK to have it but shown text should be usable/readable in order. This is not a problem ATM, but Skin/Plugin developers can use that data to show their own info and text data should be scalable dynamically liek any 2 column text (Like in Word/OpenOffice/LibreOffice/HTML. If it is not corrected now it will be much harder later. |
|
|
I think that the issue that Peke was trying to communicate is that for any view displaying an 'Info panel' element where: - the screen is sized such that there are multiple columns and - the text doesn't fit in the amount of space available that MM crops the text in columns other than the final column (it shouldn't). What should happen is that text from the first column should flow into the second (and third column, etc.) and the text should only be cropped in the final column. As to the idea of showing 'More>>>' _and_ 'Continue reading at Wikipedia [license]', I suppose that we should always show the license whenever content that falls under another license is displayed (I'm not sure about showing 'Continue reading...' since i'd rather the user stay in MM unless they really need to go to the browser to access additional data that 'More>>>' doesn't show). |
|
|
I understand what Peke wanted, I was trying to say, that current behavior is really how it works by design. In case we would wrap first column to the second, the header element would not know its full size and expanding would not work correctly. Also hiding the end of all columns by the gradient would not make sense for all but last column. I.e. the issue wants completely different behavior of the component and would lead to significant reworking, i.e. I really do not recommend to change this for 5.0. |
|
|
I still believe that it works fine currenty, by the gradient it is self evident that the text of the first column is cropped (see gradient.png ): - so for 5.0 just the gradient should be expanded (as on the first Peke's screenshots the gradient was almost invisible), but is really a minor issue IMO |
|
|
Fixed in 2254. i.e. expaded the gradient height from 1.5 lines to 2.5 lines (to make it more prominent). |
|
|
Verified the gradient change in 2254 and re-opened for the improved design in a future build. |
